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

Unified Diff: chrome/browser/android/vr_shell/textures/url_bar_texture.cc

Issue 2926643003: [VrShell] Centralize color handling and enable close button on fullscreen (Closed)
Patch Set: Created 3 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/vr_shell/textures/url_bar_texture.cc
diff --git a/chrome/browser/android/vr_shell/textures/url_bar_texture.cc b/chrome/browser/android/vr_shell/textures/url_bar_texture.cc
index 5ad3fc2c29481aa199b8f8c0cb1c04e874755630..91a0665467f7c1eae0027bceb68c91a17a36389a 100644
--- a/chrome/browser/android/vr_shell/textures/url_bar_texture.cc
+++ b/chrome/browser/android/vr_shell/textures/url_bar_texture.cc
@@ -65,12 +65,12 @@ SkColor getSchemeColor(SecurityLevel level, const ColorScheme& color_scheme) {
switch (level) {
case SecurityLevel::NONE:
case SecurityLevel::HTTP_SHOW_WARNING:
- return color_scheme.deemphasized;
+ return color_scheme.url_deemphasized;
case SecurityLevel::EV_SECURE:
case SecurityLevel::SECURE:
return color_scheme.secure;
case SecurityLevel::SECURITY_WARNING:
- return color_scheme.deemphasized;
+ return color_scheme.url_deemphasized;
case SecurityLevel::SECURE_WITH_POLICY_INSTALLED_CERT: // ChromeOS only.
return color_scheme.insecure;
case SecurityLevel::DANGEROUS:
@@ -86,7 +86,7 @@ void setEmphasis(vr_shell::RenderTextWrapper* render_text,
const gfx::Range& range,
const ColorScheme& color_scheme) {
SkColor color =
- emphasis ? color_scheme.emphasized : color_scheme.deemphasized;
+ emphasis ? color_scheme.url_emphasized : color_scheme.url_deemphasized;
if (range.IsValid()) {
render_text->ApplyColor(color, range);
} else {
@@ -169,10 +169,6 @@ void UrlBarTexture::OnSetMode() {
set_dirty();
}
-const ColorScheme& UrlBarTexture::color_scheme() const {
- return ColorScheme::GetColorScheme(mode());
-}
-
void UrlBarTexture::Draw(SkCanvas* canvas, const gfx::Size& texture_size) {
size_.set_height(texture_size.height());
size_.set_width(texture_size.width());
@@ -189,19 +185,19 @@ void UrlBarTexture::Draw(SkCanvas* canvas, const gfx::Size& texture_size) {
SkVector rounded_corner = {kHeight / 2, kHeight / 2};
SkVector left_corners[4] = {rounded_corner, {0, 0}, {0, 0}, rounded_corner};
round_rect.setRectRadii({0, 0, kHeight, kHeight}, left_corners);
- SkColor color = color_scheme().background;
+ SkColor color = color_scheme().element_background;
if (can_go_back_) {
if (pressed_)
- color = color_scheme().background_down;
+ color = color_scheme().element_background_down;
else if (hovered_)
- color = color_scheme().background_hover;
+ color = color_scheme().element_background_hover;
}
SkPaint paint;
paint.setColor(color);
canvas->drawRRect(round_rect, paint);
// URL area.
- paint.setColor(color_scheme().background);
+ paint.setColor(color_scheme().element_background);
SkVector right_corners[4] = {{0, 0}, rounded_corner, rounded_corner, {0, 0}};
round_rect.setRectRadii({kHeight, 0, kWidth, kHeight}, right_corners);
canvas->drawRRect(round_rect, paint);
@@ -218,9 +214,9 @@ void UrlBarTexture::Draw(SkCanvas* canvas, const gfx::Size& texture_size) {
int icon_default_height = GetDefaultSizeOfVectorIcon(ui::kBackArrowIcon);
float icon_scale = kBackIconHeight / icon_default_height;
canvas->scale(icon_scale, icon_scale);
- PaintVectorIcon(
- &gfx_canvas, ui::kBackArrowIcon,
- can_go_back_ ? color_scheme().foreground : color_scheme().disabled);
+ PaintVectorIcon(&gfx_canvas, ui::kBackArrowIcon,
+ can_go_back_ ? color_scheme().element_foreground
+ : color_scheme().disabled);
canvas->restore();
// Site security state icon.

Powered by Google App Engine
This is Rietveld 408576698