diff options
| author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-29 20:35:37 +0000 |
|---|---|---|
| committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-29 20:35:37 +0000 |
| commit | 24db8a07affa9456337a9b7eb1c22dffff2c0458 (patch) | |
| tree | 41f2a49c5909bf54fb706628956310993c912912 /chrome/browser/cocoa | |
| parent | 9eabdd09a7af3be5341e4ff2627fe4023aaa0ab6 (diff) | |
| download | chromium_src-24db8a07affa9456337a9b7eb1c22dffff2c0458.zip chromium_src-24db8a07affa9456337a9b7eb1c22dffff2c0458.tar.gz chromium_src-24db8a07affa9456337a9b7eb1c22dffff2c0458.tar.bz2 | |
Fix and re-enable a test I broke with the find bar changes I made last night.
Remove Browser::find_bar(), make everyone use either Browser::GetFindBarController() which creates the FindBarController on demand, or a new method HasFindBarController(), which can be used to check for its existence (so as to avoid creating it unnecessarily).
http://crbug.com/26231
TEST=interactive ui tests, standard find tests
Review URL: http://codereview.chromium.org/348015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30507 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa')
| -rw-r--r-- | chrome/browser/cocoa/tab_strip_controller.mm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/cocoa/tab_strip_controller.mm b/chrome/browser/cocoa/tab_strip_controller.mm index e37fabb..75b7f45 100644 --- a/chrome/browser/cocoa/tab_strip_controller.mm +++ b/chrome/browser/cocoa/tab_strip_controller.mm @@ -649,7 +649,7 @@ static const float kIndentLeavingSpaceForControls = 64.0; newContents->view()->RestoreFocus(); if (newContents->find_ui_active()) - browser_->find_bar()->find_bar()->SetFocusAndSelection(); + browser_->GetFindBarController()->find_bar()->SetFocusAndSelection(); } } |