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

Unified Diff: chrome/browser/android/vr_shell/textures/insecure_content_permanent_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/insecure_content_permanent_texture.cc
diff --git a/chrome/browser/android/vr_shell/textures/insecure_content_permanent_texture.cc b/chrome/browser/android/vr_shell/textures/insecure_content_permanent_texture.cc
index ba7d35a29d64969f25d9fdfc852e3fdfc4fbf426..f43c061831e27f556e04fa02af3ed7f25a8f8801 100644
--- a/chrome/browser/android/vr_shell/textures/insecure_content_permanent_texture.cc
+++ b/chrome/browser/android/vr_shell/textures/insecure_content_permanent_texture.cc
@@ -20,8 +20,6 @@ namespace vr_shell {
namespace {
-const SkColor kBackgroundColor = SK_ColorWHITE;
-const SkColor kForegroundColor = 0xFF444444;
constexpr float kBorderFactor = 0.1;
constexpr float kIconSizeFactor = 0.7;
constexpr float kFontSizeFactor = 0.40;
@@ -43,16 +41,16 @@ void InsecureContentPermanentTexture::Draw(SkCanvas* sk_canvas,
DCHECK(texture_size.height() * 4 == texture_size.width());
size_.set_height(texture_size.height());
SkPaint paint;
- paint.setColor(kBackgroundColor);
+ paint.setColor(color_scheme().permanent_warning_background);
auto text =
l10n_util::GetStringUTF16(IDS_PAGE_INFO_INSECURE_WEBVR_CONTENT_PERMANENT);
gfx::FontList fonts;
GetFontList(size_.height() * kFontSizeFactor, text, &fonts);
gfx::Rect text_size(0, kTextHeightFactor * size_.height());
- std::vector<std::unique_ptr<gfx::RenderText>> lines =
- PrepareDrawStringRect(text, fonts, kForegroundColor, &text_size,
- kTextAlignmentNone, kWrappingBehaviorNoWrap);
+ std::vector<std::unique_ptr<gfx::RenderText>> lines = PrepareDrawStringRect(
+ text, fonts, color_scheme().permanent_warning_foreground, &text_size,
+ kTextAlignmentNone, kWrappingBehaviorNoWrap);
DCHECK_LE(text_size.width(), kTextWidthFactor * size_.height());
// Setting background size giving some extra lateral padding to the text.
@@ -68,7 +66,8 @@ void InsecureContentPermanentTexture::Draw(SkCanvas* sk_canvas,
: size_.height() * kBorderFactor,
size_.height() * (1.0 - kIconSizeFactor) / 2.0));
PaintVectorIcon(canvas, ui::kInfoOutlineIcon,
- size_.height() * kIconSizeFactor, kForegroundColor);
+ size_.height() * kIconSizeFactor,
+ color_scheme().permanent_warning_foreground);
canvas->Restore();
canvas->Save();

Powered by Google App Engine
This is Rietveld 408576698