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

Unified Diff: chrome/browser/android/compositor/layer/tab_layer.cc

Issue 2045733002: Replace tab switcher borders with single asset (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move var Created 4 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/android/compositor/layer/tab_layer.cc
diff --git a/chrome/browser/android/compositor/layer/tab_layer.cc b/chrome/browser/android/compositor/layer/tab_layer.cc
index fe1d5b27d0af8eb8ae4f7932a24ddbeda92ea48f..dc16d07239bcfdbcf9fde71e229230081aba8db1 100644
--- a/chrome/browser/android/compositor/layer/tab_layer.cc
+++ b/chrome/browser/android/compositor/layer/tab_layer.cc
@@ -134,6 +134,7 @@ void TabLayer::SetProperties(int id,
float view_width,
float view_height,
bool show_toolbar,
+ int default_theme_color,
int toolbar_background_color,
bool anonymize_toolbar,
int toolbar_textbox_resource_id,
@@ -479,15 +480,20 @@ void TabLayer::SetProperties(int id,
front_border_->SetOpacity(border_alpha);
front_border_->SetNearestNeighbor(toolbar_visible);
- // Colorize the tab decoration.
- if (tab_switcher_themes_enabled_ && !incognito_
- && toolbar_background_color != toolbar_background_color_) {
+ int tab_switcher_color = default_theme_color;
+
+ // Colorize the tab decoration if enabled.
+ if (tab_switcher_themes_enabled_) {
+ tab_switcher_color = toolbar_background_color;
+ }
+
+ if (toolbar_background_color != toolbar_background_color_) {
toolbar_background_color_ = toolbar_background_color;
cc::FilterOperations filters;
SkScalar colorMatrix[] = {
- SkColorGetR(toolbar_background_color) / 255.0f, 0, 0, 0, 0,
- 0, SkColorGetG(toolbar_background_color) / 255.0f, 0, 0, 0,
- 0, 0, SkColorGetB(toolbar_background_color) / 255.0f, 0, 0,
+ SkColorGetR(tab_switcher_color) / 255.0f, 0, 0, 0, 0,
+ 0, SkColorGetG(tab_switcher_color) / 255.0f, 0, 0, 0,
+ 0, 0, SkColorGetB(tab_switcher_color) / 255.0f, 0, 0,
0, 0, 0, 1, 0,
};
filters.Append(cc::FilterOperation::CreateColorMatrixFilter(colorMatrix));

Powered by Google App Engine
This is Rietveld 408576698