Chromium Code Reviews| Index: chrome/browser/ui/views/frame/browser_command_handler_x11.cc |
| diff --git a/chrome/browser/ui/views/frame/browser_command_handler_x11.cc b/chrome/browser/ui/views/frame/browser_command_handler_x11.cc |
| index ce37fcc47799ad2138f4e943214cead8fd23c3ea..d5ea53ce3d328045361ca0a7c1d5b6324d925ca4 100644 |
| --- a/chrome/browser/ui/views/frame/browser_command_handler_x11.cc |
| +++ b/chrome/browser/ui/views/frame/browser_command_handler_x11.cc |
| @@ -8,15 +8,25 @@ |
| #include "chrome/browser/ui/browser.h" |
| #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| +#include "chrome/browser/ui/views/frame/browser_view.h" |
| #include "content/public/browser/navigation_controller.h" |
| #include "content/public/browser/web_contents.h" |
| +#include "ui/aura/window.h" |
| #include "ui/events/event.h" |
| #include "ui/events/event_utils.h" |
| -BrowserCommandHandlerX11::BrowserCommandHandlerX11(Browser* browser) |
| - : browser_(browser) {} |
| +BrowserCommandHandlerX11::BrowserCommandHandlerX11(BrowserView* browser_view) |
| + : browser_view_(browser_view) { |
| + aura::Window* window = browser_view_->frame()->GetNativeWindow(); |
| + if (window) |
|
tapted
2014/11/07 08:35:58
Should this ever be null? - maybe DCHECK(window)?
Andre
2014/11/07 19:06:44
Done.
|
| + window->AddPreTargetHandler(this); |
| +} |
| -BrowserCommandHandlerX11::~BrowserCommandHandlerX11() {} |
| +BrowserCommandHandlerX11::~BrowserCommandHandlerX11() { |
| + aura::Window* window = browser_view_->frame()->GetNativeWindow(); |
| + if (window) |
| + window->RemovePreTargetHandler(this); |
| +} |
| void BrowserCommandHandlerX11::OnMouseEvent(ui::MouseEvent* event) { |
| if (event->type() != ui::ET_MOUSE_PRESSED) |
| @@ -32,7 +42,7 @@ void BrowserCommandHandlerX11::OnMouseEvent(ui::MouseEvent* event) { |
| const int kForwardMouseButton = 9; |
| if (button == kBackMouseButton || button == kForwardMouseButton) { |
| content::WebContents* contents = |
| - browser_->tab_strip_model()->GetActiveWebContents(); |
| + browser_view_->browser()->tab_strip_model()->GetActiveWebContents(); |
| if (!contents) |
| return; |
| content::NavigationController& controller = contents->GetController(); |