summaryrefslogtreecommitdiffstats
path: root/chrome/browser/views
diff options
context:
space:
mode:
Diffstat (limited to 'chrome/browser/views')
-rw-r--r--chrome/browser/views/frame/browser_frame_win.cc5
-rw-r--r--chrome/browser/views/frame/browser_frame_win.h1
-rw-r--r--chrome/browser/views/info_bubble.cc2
-rw-r--r--chrome/browser/views/info_bubble.h8
4 files changed, 5 insertions, 11 deletions
diff --git a/chrome/browser/views/frame/browser_frame_win.cc b/chrome/browser/views/frame/browser_frame_win.cc
index 4c7cd02..4315346 100644
--- a/chrome/browser/views/frame/browser_frame_win.cc
+++ b/chrome/browser/views/frame/browser_frame_win.cc
@@ -121,11 +121,6 @@ bool BrowserFrameWin::AlwaysUseNativeFrame() const {
///////////////////////////////////////////////////////////////////////////////
// BrowserFrame, views::WidgetWin overrides:
-bool BrowserFrameWin::AcceleratorPressed(
- const views::Accelerator& accelerator) {
- return browser_view_->AcceleratorPressed(accelerator);
-}
-
bool BrowserFrameWin::GetAccelerator(int cmd_id,
views::Accelerator* accelerator) {
return browser_view_->GetAccelerator(cmd_id, accelerator);
diff --git a/chrome/browser/views/frame/browser_frame_win.h b/chrome/browser/views/frame/browser_frame_win.h
index 7bde95f..dc23eb7 100644
--- a/chrome/browser/views/frame/browser_frame_win.h
+++ b/chrome/browser/views/frame/browser_frame_win.h
@@ -48,7 +48,6 @@ class BrowserFrameWin : public BrowserFrame, public views::WindowWin {
protected:
// Overridden from views::WidgetWin:
- virtual bool AcceleratorPressed(const views::Accelerator& accelerator);
virtual bool GetAccelerator(int cmd_id, views::Accelerator* accelerator);
virtual void OnEndSession(BOOL ending, UINT logoff);
virtual void OnEnterSizeMove();
diff --git a/chrome/browser/views/info_bubble.cc b/chrome/browser/views/info_bubble.cc
index bba77a9f3..1218f46 100644
--- a/chrome/browser/views/info_bubble.cc
+++ b/chrome/browser/views/info_bubble.cc
@@ -277,11 +277,9 @@ void InfoBubble::Init(views::Window* parent,
#endif
SetBounds(window_bounds);
-#if defined(OS_WIN)
// Register the Escape accelerator for closing.
GetFocusManager()->RegisterAccelerator(
views::Accelerator(base::VKEY_ESCAPE, false, false, false), this);
-#endif
// Done creating the bubble.
NotificationService::current()->Notify(NotificationType::INFO_BUBBLE_CREATED,
diff --git a/chrome/browser/views/info_bubble.h b/chrome/browser/views/info_bubble.h
index 5ee23f5..996413a 100644
--- a/chrome/browser/views/info_bubble.h
+++ b/chrome/browser/views/info_bubble.h
@@ -79,11 +79,13 @@ class InfoBubbleDelegate {
// TODO: this code is ifdef-tastic. It might be cleaner to refactor the
// WidgetFoo subclass into a separate class that calls into InfoBubble.
// That way InfoBubble has no (or very few) ifdefs.
+class InfoBubble
#if defined(OS_WIN)
-class InfoBubble : public views::WidgetWin {
+ : public views::WidgetWin,
#elif defined(OS_LINUX)
-class InfoBubble : public views::WidgetGtk {
+ : public views::WidgetGtk,
#endif
+ public views::AcceleratorTarget {
public:
// Shows the InfoBubble. |parent| is set as the parent window, |contents| are
// the contents shown in the bubble, and |position_relative_to| is a rect in
@@ -126,7 +128,7 @@ class InfoBubble : public views::WidgetGtk {
// the close is the result of pressing escape.
void Close(bool closed_by_escape);
- // Overridden from WidgetWin/WidgetGtk:
+ // Overridden from AcceleratorTarget:
virtual bool AcceleratorPressed(const views::Accelerator& accelerator);
// The delegate, if any.