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

Side by Side Diff: chrome/browser/ui/views/toolbar_view.cc

Issue 10702029: Move tab functions off Browser into browser_tabstrip and browser_tabrestore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 months 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/ui/views/toolbar_view.h" 5 #include "chrome/browser/ui/views/toolbar_view.h"
6 6
7 #include "base/i18n/number_formatting.h" 7 #include "base/i18n/number_formatting.h"
8 #include "base/utf_string_conversions.h" 8 #include "base/utf_string_conversions.h"
9 #include "chrome/app/chrome_command_ids.h" 9 #include "chrome/app/chrome_command_ids.h"
10 #include "chrome/browser/command_updater.h" 10 #include "chrome/browser/command_updater.h"
11 #include "chrome/browser/event_disposition.h" 11 #include "chrome/browser/event_disposition.h"
12 #include "chrome/browser/prefs/pref_service.h" 12 #include "chrome/browser/prefs/pref_service.h"
13 #include "chrome/browser/profiles/profile.h" 13 #include "chrome/browser/profiles/profile.h"
14 #include "chrome/browser/themes/theme_service.h" 14 #include "chrome/browser/themes/theme_service.h"
15 #include "chrome/browser/ui/browser.h" 15 #include "chrome/browser/ui/browser.h"
16 #include "chrome/browser/ui/browser_command_controller.h" 16 #include "chrome/browser/ui/browser_command_controller.h"
17 #include "chrome/browser/ui/browser_commands.h" 17 #include "chrome/browser/ui/browser_commands.h"
18 #include "chrome/browser/ui/browser_content_setting_bubble_model_delegate.h" 18 #include "chrome/browser/ui/browser_content_setting_bubble_model_delegate.h"
19 #include "chrome/browser/ui/browser_tabstrip.h"
19 #include "chrome/browser/ui/browser_window.h" 20 #include "chrome/browser/ui/browser_window.h"
20 #include "chrome/browser/ui/global_error/global_error_service.h" 21 #include "chrome/browser/ui/global_error/global_error_service.h"
21 #include "chrome/browser/ui/global_error/global_error_service_factory.h" 22 #include "chrome/browser/ui/global_error/global_error_service_factory.h"
22 #include "chrome/browser/ui/omnibox/omnibox_view.h" 23 #include "chrome/browser/ui/omnibox/omnibox_view.h"
23 #include "chrome/browser/ui/search/search.h" 24 #include "chrome/browser/ui/search/search.h"
24 #include "chrome/browser/ui/search/search_model.h" 25 #include "chrome/browser/ui/search/search_model.h"
25 #include "chrome/browser/ui/search/search_tab_helper.h" 26 #include "chrome/browser/ui/search/search_tab_helper.h"
26 #include "chrome/browser/ui/search/search_types.h" 27 #include "chrome/browser/ui/search/search_types.h"
27 #include "chrome/browser/ui/search/search_ui.h" 28 #include "chrome/browser/ui/search/search_ui.h"
28 #include "chrome/browser/ui/tab_contents/tab_contents.h" 29 #include "chrome/browser/ui/tab_contents/tab_contents.h"
(...skipping 390 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 420
420 FOR_EACH_OBSERVER(views::MenuListener, menu_listeners_, OnMenuOpened()); 421 FOR_EACH_OBSERVER(views::MenuListener, menu_listeners_, OnMenuOpened());
421 422
422 wrench_menu_->RunMenu(app_menu_); 423 wrench_menu_->RunMenu(app_menu_);
423 } 424 }
424 425
425 //////////////////////////////////////////////////////////////////////////////// 426 ////////////////////////////////////////////////////////////////////////////////
426 // ToolbarView, LocationBarView::Delegate implementation: 427 // ToolbarView, LocationBarView::Delegate implementation:
427 428
428 TabContents* ToolbarView::GetTabContents() const { 429 TabContents* ToolbarView::GetTabContents() const {
429 return browser_->GetActiveTabContents(); 430 return chrome::GetActiveTabContents(browser_);
430 } 431 }
431 432
432 InstantController* ToolbarView::GetInstant() { 433 InstantController* ToolbarView::GetInstant() {
433 return browser_->instant(); 434 return browser_->instant();
434 } 435 }
435 436
436 ContentSettingBubbleModelDelegate* 437 ContentSettingBubbleModelDelegate*
437 ToolbarView::GetContentSettingBubbleModelDelegate() { 438 ToolbarView::GetContentSettingBubbleModelDelegate() {
438 return browser_->content_setting_bubble_model_delegate(); 439 return browser_->content_setting_bubble_model_delegate();
439 } 440 }
(...skipping 482 matching lines...) Expand 10 before | Expand all | Expand 10 after
922 923
923 app_menu_->SetIcon(GetAppMenuIcon(views::CustomButton::BS_NORMAL)); 924 app_menu_->SetIcon(GetAppMenuIcon(views::CustomButton::BS_NORMAL));
924 app_menu_->SetHoverIcon(GetAppMenuIcon(views::CustomButton::BS_HOT)); 925 app_menu_->SetHoverIcon(GetAppMenuIcon(views::CustomButton::BS_HOT));
925 app_menu_->SetPushedIcon(GetAppMenuIcon(views::CustomButton::BS_PUSHED)); 926 app_menu_->SetPushedIcon(GetAppMenuIcon(views::CustomButton::BS_PUSHED));
926 SchedulePaint(); 927 SchedulePaint();
927 } 928 }
928 929
929 void ToolbarView::LayoutLocationBarNTP() { 930 void ToolbarView::LayoutLocationBarNTP() {
930 // TODO(kuan): this likely needs to cancel animations. 931 // TODO(kuan): this likely needs to cancel animations.
931 932
932 WebContents* contents = browser_->GetActiveWebContents(); 933 WebContents* contents = chrome::GetActiveWebContents(browser_);
933 #if defined(USE_AURA) 934 #if defined(USE_AURA)
934 // Under aura we can't use WebContentsView::GetContainerBounds since it is 935 // Under aura we can't use WebContentsView::GetContainerBounds since it is
935 // affected by any animations that scale the window (such as during startup). 936 // affected by any animations that scale the window (such as during startup).
936 // Instead we convert coordinates using aura::Window. 937 // Instead we convert coordinates using aura::Window.
937 aura::Window* contents_view = contents && contents->GetView() ? 938 aura::Window* contents_view = contents && contents->GetView() ?
938 contents->GetView()->GetNativeView() : NULL; 939 contents->GetView()->GetNativeView() : NULL;
939 if (!contents_view) 940 if (!contents_view)
940 return; 941 return;
941 942
942 aura::Window* browser_window = GetWidget()->GetNativeView(); 943 aura::Window* browser_window = GetWidget()->GetNativeView();
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
993 // LocationBarContainer is not a child of the ToolbarView. 994 // LocationBarContainer is not a child of the ToolbarView.
994 gfx::Point origin(bounds.origin()); 995 gfx::Point origin(bounds.origin());
995 views::View::ConvertPointToView(this, location_bar_container_->parent(), 996 views::View::ConvertPointToView(this, location_bar_container_->parent(),
996 &origin); 997 &origin);
997 gfx::Rect target_bounds(origin, bounds.size()); 998 gfx::Rect target_bounds(origin, bounds.size());
998 if (location_bar_container_->GetTargetBounds() != target_bounds) { 999 if (location_bar_container_->GetTargetBounds() != target_bounds) {
999 location_bar_container_->SetInToolbar(true); 1000 location_bar_container_->SetInToolbar(true);
1000 location_bar_container_->SetBoundsRect(target_bounds); 1001 location_bar_container_->SetBoundsRect(target_bounds);
1001 } 1002 }
1002 } 1003 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc ('k') | chrome/browser/ui/views/web_dialog_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698