Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Side by Side Diff: chrome/browser/gtk/browser_window_gtk.cc

Issue 348015: Fix and re-enable a test I broke with the find bar changes I made last night.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/gtk/browser_window_gtk.h" 5 #include "chrome/browser/gtk/browser_window_gtk.h"
6 6
7 #include <gdk/gdkkeysyms.h> 7 #include <gdk/gdkkeysyms.h>
8 #include <X11/XF86keysym.h> 8 #include <X11/XF86keysym.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 1238 matching lines...) Expand 10 before | Expand all | Expand 10 after
1249 infobar_container_->ChangeTabContents(new_contents); 1249 infobar_container_->ChangeTabContents(new_contents);
1250 contents_container_->SetTabContents(new_contents); 1250 contents_container_->SetTabContents(new_contents);
1251 UpdateDevToolsForContents(new_contents); 1251 UpdateDevToolsForContents(new_contents);
1252 1252
1253 new_contents->DidBecomeSelected(); 1253 new_contents->DidBecomeSelected();
1254 // TODO(estade): after we manage browser activation, add a check to make sure 1254 // TODO(estade): after we manage browser activation, add a check to make sure
1255 // we are the active browser before calling RestoreFocus(). 1255 // we are the active browser before calling RestoreFocus().
1256 if (!browser_->tabstrip_model()->closing_all()) { 1256 if (!browser_->tabstrip_model()->closing_all()) {
1257 new_contents->view()->RestoreFocus(); 1257 new_contents->view()->RestoreFocus();
1258 if (new_contents->find_ui_active()) 1258 if (new_contents->find_ui_active())
1259 browser_->find_bar()->find_bar()->SetFocusAndSelection(); 1259 browser_->GetFindBarController()->find_bar()->SetFocusAndSelection();
1260 } 1260 }
1261 1261
1262 // Update all the UI bits. 1262 // Update all the UI bits.
1263 UpdateTitleBar(); 1263 UpdateTitleBar();
1264 toolbar_->SetProfile(new_contents->profile()); 1264 toolbar_->SetProfile(new_contents->profile());
1265 UpdateToolbar(new_contents, true); 1265 UpdateToolbar(new_contents, true);
1266 UpdateUIForContents(new_contents); 1266 UpdateUIForContents(new_contents);
1267 } 1267 }
1268 1268
1269 void BrowserWindowGtk::TabStripEmpty() { 1269 void BrowserWindowGtk::TabStripEmpty() {
(...skipping 1032 matching lines...) Expand 10 before | Expand all | Expand 10 after
2302 // are taken from the WMs' source code. 2302 // are taken from the WMs' source code.
2303 return (wm_name == "Blackbox" || 2303 return (wm_name == "Blackbox" ||
2304 wm_name == "compiz" || 2304 wm_name == "compiz" ||
2305 wm_name == "e16" || // Enlightenment DR16 2305 wm_name == "e16" || // Enlightenment DR16
2306 wm_name == "KWin" || 2306 wm_name == "KWin" ||
2307 wm_name == "Metacity" || 2307 wm_name == "Metacity" ||
2308 wm_name == "Mutter" || 2308 wm_name == "Mutter" ||
2309 wm_name == "Openbox" || 2309 wm_name == "Openbox" ||
2310 wm_name == "Xfwm4"); 2310 wm_name == "Xfwm4");
2311 } 2311 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698