diff options
| author | oshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-22 03:19:52 +0000 |
|---|---|---|
| committer | oshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-22 03:19:52 +0000 |
| commit | 88de6c013ddad04c6896f10714c787ce21cf858a (patch) | |
| tree | 58afe864c5335bbf7ead61f8082fb70b5c9dbfe6 /chrome/browser/ui/find_bar | |
| parent | 9f7e241b3649c863e353c724d1117b3f56e5f512 (diff) | |
| download | chromium_src-88de6c013ddad04c6896f10714c787ce21cf858a.zip chromium_src-88de6c013ddad04c6896f10714c787ce21cf858a.tar.gz chromium_src-88de6c013ddad04c6896f10714c787ce21cf858a.tar.bz2 | |
Don't call OnNativeWidgetVisibilityChange in NativeWidgetGtk
BUG=chromium-os:20545
TEST=added a test case in FindInPageControllerTest.AcceleratorRestoring. See bug for repro step.
Review URL: http://codereview.chromium.org/7978015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102233 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/find_bar')
| -rw-r--r-- | chrome/browser/ui/find_bar/find_bar_host_browsertest.cc | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc b/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc index f569772..650951c 100644 --- a/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc +++ b/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc @@ -782,6 +782,13 @@ IN_PROC_BROWSER_TEST_F(FindInPageControllerTest, AcceleratorRestoring) { // The accelerator for Escape should be back to what it was before. EXPECT_EQ(old_target, focus_manager->GetCurrentTargetForAccelerator(escape)); + + // Show find bar again with animation on, and the target should be + // on find bar. + DropdownBarHost::disable_animations_during_testing_ = false; + browser()->ShowFindBar(); + EXPECT_EQ(new_target, + focus_manager->GetCurrentTargetForAccelerator(escape)); } #endif // TOOLKIT_VIEWS |