Skip to content

Commit

Permalink
Remove TODO for ash::FrameBackButton
Browse files Browse the repository at this point in the history
Bug: 887663
Change-Id: I0e5d3e16d59cf3459fde94f3ed2edb23c2ea9f0e
Reviewed-on: https://chromium-review.googlesource.com/c/1260002
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597593}
  • Loading branch information
Evan Stade authored and Commit Bot committed Oct 8, 2018
1 parent 0fd89f3 commit 0860c04
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions ash/public/cpp/caption_buttons/frame_back_button.cc
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,6 @@ FrameBackButton::~FrameBackButton() = default;

void FrameBackButton::ButtonPressed(Button* sender, const ui::Event& event) {
// Send up event as well as down event as ARC++ clients expect this sequence.
// TODO(estade): this may not actually work in Mash, but thus far isn't used
// there. Consider fixing or moving back to //ash/frame/.
// https://crbug.com/887663
aura::Window* root_window = GetWidget()->GetNativeWindow()->GetRootWindow();
ui::KeyEvent press_key_event(ui::ET_KEY_PRESSED, ui::VKEY_BROWSER_BACK,
ui::EF_NONE);
Expand Down

0 comments on commit 0860c04

Please sign in to comment.