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

Unified Diff: chrome/browser/ui/views/frame/browser_view.cc

Issue 10662032: alternate ntp (cros/partial-win): add tab-related stuff and toolbar/tab background change (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressed scott's comments Created 8 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index 615f495fbc005063132269b66a9afeacd9ddb053..7bc3954301add464c0c19800d801eebf43710c0c 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -40,6 +40,7 @@
#include "chrome/browser/ui/omnibox/omnibox_popup_view.h"
#include "chrome/browser/ui/omnibox/omnibox_view.h"
#include "chrome/browser/ui/search/search.h"
+#include "chrome/browser/ui/search/search_model.h"
#include "chrome/browser/ui/tab_contents/tab_contents.h"
#include "chrome/browser/ui/tabs/tab_menu_model.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ -242,7 +243,11 @@ void BookmarkExtensionBackground::Paint(gfx::Canvas* canvas,
} else {
DetachableToolbarView::PaintBackgroundAttachedMode(canvas, host_view_,
browser_view_->OffsetPointForToolbarBackgroundImage(
- gfx::Point(host_view_->GetMirroredX(), host_view_->y())));
+ gfx::Point(host_view_->GetMirroredX(), host_view_->y())),
+ browser_view_->GetToolbarBackgroundColor(
+ browser_view_->browser()->search_model()->mode().mode),
+ browser_view_->GetToolbarBackgroundImage(
+ browser_view_->browser()->search_model()->mode().mode));
if (host_view_->height() >= toolbar_overlap)
DetachableToolbarView::PaintHorizontalBorder(canvas, host_view_);
}
@@ -856,6 +861,48 @@ void BrowserView::ToolbarSizeChanged(bool is_animating) {
}
}
+SkColor BrowserView::GetToolbarBackgroundColor(
+ chrome::search::Mode::Type mode) {
+ ui::ThemeProvider* theme_provider = GetThemeProvider();
+ DCHECK(theme_provider);
+
+ if (!chrome::search::IsInstantExtendedAPIEnabled(browser()->profile()))
+ return theme_provider->GetColor(ThemeService::COLOR_TOOLBAR);
+
+ switch (mode) {
+ case chrome::search::Mode::MODE_NTP:
+ return theme_provider->GetColor(
+ ThemeService::COLOR_SEARCH_NTP_BACKGROUND);
+
+ case chrome::search::Mode::MODE_SEARCH:
+ return theme_provider->GetColor(
+ ThemeService::COLOR_SEARCH_SEARCH_BACKGROUND);
+
+ case chrome::search::Mode::MODE_DEFAULT:
+ default:
+ return theme_provider->GetColor(
+ ThemeService::COLOR_SEARCH_DEFAULT_BACKGROUND);
+ }
+}
+
+gfx::ImageSkia* BrowserView::GetToolbarBackgroundImage(
+ chrome::search::Mode::Type mode) {
+ ui::ThemeProvider* theme_provider = GetThemeProvider();
+ DCHECK(theme_provider);
+ if (!chrome::search::IsInstantExtendedAPIEnabled(browser()->profile()))
+ return theme_provider->GetImageSkiaNamed(IDR_THEME_TOOLBAR);
+
+ switch (mode) {
+ case chrome::search::Mode::MODE_NTP:
+ return theme_provider->GetImageSkiaNamed(IDR_THEME_NTP_BACKGROUND);
+
+ case chrome::search::Mode::MODE_SEARCH:
+ case chrome::search::Mode::MODE_DEFAULT:
+ default:
+ return theme_provider->GetImageSkiaNamed(IDR_THEME_TOOLBAR_SEARCH);
+ }
+}
+
LocationBar* BrowserView::GetLocationBar() const {
return GetLocationBarView();
}
@@ -1859,7 +1906,9 @@ void BrowserView::Init() {
BrowserTabStripController* tabstrip_controller =
new BrowserTabStripController(browser_.get(),
browser_->tab_strip_model());
- tabstrip_ = new TabStrip(tabstrip_controller);
+ tabstrip_ = new TabStrip(tabstrip_controller,
+ chrome::search::IsInstantExtendedAPIEnabled(
+ browser_->profile()));
AddChildView(tabstrip_);
tabstrip_controller->InitFromModel(tabstrip_);
« no previous file with comments | « chrome/browser/ui/views/frame/browser_view.h ('k') | chrome/browser/ui/views/frame/glass_browser_frame_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698