diff options
author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-14 03:42:55 +0000 |
---|---|---|
committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-14 03:42:55 +0000 |
commit | 334a105b240c34841c864e8dd50974d90e0105b1 (patch) | |
tree | ed3d36542e3bb7bf1719e0b7268e72ae14698cdb /views/focus/focus_manager.h | |
parent | de23fff1310d85866e800103b50fb04851b6d1bc (diff) | |
download | chromium_src-334a105b240c34841c864e8dd50974d90e0105b1.zip chromium_src-334a105b240c34841c864e8dd50974d90e0105b1.tar.gz chromium_src-334a105b240c34841c864e8dd50974d90e0105b1.tar.bz2 |
Allow TabContentsDelegates to process unhandled keyboard messages coming back from the renderer. This is required
for ExternalTabContainer to be able to pass unhandled keyboard accelerator messages back to the external host.
This worked before as the focus manager would hand off these messages to keystroke listeners like the ExternalTabContainer.
However with the ExternalTabContainer no longer using the focus manager this does not work anymore.
Removed the keystroke listener code from the focus manager as the ExternalTabContainer was the only consumer and it does not use it anymore.
This fixes bug http://b/issue?id=1975749
Bug=1975749
Review URL: http://codereview.chromium.org/149520
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20600 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/focus/focus_manager.h')
-rw-r--r-- | views/focus/focus_manager.h | 24 |
1 files changed, 0 insertions, 24 deletions
diff --git a/views/focus/focus_manager.h b/views/focus/focus_manager.h index 8b4c8c9a..b1c87b6 100644 --- a/views/focus/focus_manager.h +++ b/views/focus/focus_manager.h @@ -128,19 +128,6 @@ class FocusTraversable { virtual View* GetFocusTraversableParentView() = 0; }; -// The KeystrokeListener interface is used by components (such as the -// ExternalTabContainer class) which need a crack at handling all -// keystrokes. -class KeystrokeListener { - public: - // If this returns true, then the component handled the keystroke and ate - // it. -#if defined(OS_WIN) - virtual bool ProcessKeyStroke(HWND window, UINT message, WPARAM wparam, - LPARAM lparam) = 0; -#endif -}; - // This interface should be implemented by classes that want to be notified when // the focus is about to change. See the Add/RemoveFocusChangeListener methods. class FocusChangeListener { @@ -159,10 +146,6 @@ class FocusManager { UINT message, WPARAM wparam, LPARAM lparam); - bool OnKeyUp(HWND window, - UINT message, - WPARAM wparam, - LPARAM lparam); #endif // Returns true is the specified is part of the hierarchy of the window @@ -224,9 +207,6 @@ class FocusManager { // that this FocusManager is attached to the parent Widget of. void ViewRemoved(View* parent, View* removed); - void AddKeystrokeListener(KeystrokeListener* listener); - void RemoveKeystrokeListener(KeystrokeListener* listener); - // Adds/removes a listener. The FocusChangeListener is notified every time // the focused view is about to change. void AddFocusChangeListener(FocusChangeListener* listener); @@ -280,10 +260,6 @@ class FocusManager { typedef std::map<Accelerator, AcceleratorTargetList> AcceleratorMap; AcceleratorMap accelerators_; - // The list of registered keystroke listeners - typedef std::vector<KeystrokeListener*> KeystrokeListenerList; - KeystrokeListenerList keystroke_listeners_; - // The list of registered FocusChange listeners. typedef std::vector<FocusChangeListener*> FocusChangeListenerList; FocusChangeListenerList focus_change_listeners_; |