diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-19 22:57:54 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-19 22:57:54 +0000 |
| commit | a109d02492863aeb537dea09bfb445eb6f5d94c5 (patch) | |
| tree | bd3d2e9a783179eb2582f0055192b97e8d248547 /chrome/browser/external_tab_container.cc | |
| parent | 3c30f4c1e5f2bdb4f5d198c4a9ea18cc8045ddea (diff) | |
| download | chromium_src-a109d02492863aeb537dea09bfb445eb6f5d94c5.zip chromium_src-a109d02492863aeb537dea09bfb445eb6f5d94c5.tar.gz chromium_src-a109d02492863aeb537dea09bfb445eb6f5d94c5.tar.bz2 | |
Removed the last Windows specific part out of the focus manager. HWNDs are not subclassed anymore.The FocusManager is now created and owned by top-level WidgetWins.BUG=NoneTEST=Run the unit tests, UI tests, interactive UI tests. Fully test the focus behavior in the browser: activate/deactivate the browser windows, make sure focus is remembered. Switch tabs, make sure focus is remembered for each tab. make sure accelerators work as expected. Test focus traversal in a web page, in the option dialog.
Review URL: http://codereview.chromium.org/125148
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18872 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/external_tab_container.cc')
| -rw-r--r-- | chrome/browser/external_tab_container.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/external_tab_container.cc b/chrome/browser/external_tab_container.cc index 7da0be0..d7e4836 100644 --- a/chrome/browser/external_tab_container.cc +++ b/chrome/browser/external_tab_container.cc @@ -53,11 +53,12 @@ bool ExternalTabContainer::Init(Profile* profile, } set_window_style(WS_POPUP); - views::WidgetWin::Init(NULL, bounds, true); + views::WidgetWin::Init(NULL, bounds); if (!IsWindow()) { NOTREACHED(); return false; } + // TODO(jcampan): limit focus traversal to contents. // We don't ever remove the prop because the lifetime of this object // is the same as the lifetime of the window |