[vr] Clicking on the security icon should prompt the user to bail out of VR

chrome/browser/android/vr_shell/color_scheme.cc

Index: chrome/browser/android/vr_shell/color_scheme.cc
diff --git a/chrome/browser/android/vr_shell/color_scheme.cc b/chrome/browser/android/vr_shell/color_scheme.cc
index 49432770da2703ff5c8e26b695a93b8ad63e2b95..ab068ed7e0c0706886d2fc6de414ada0ce32a998 100644
--- a/chrome/browser/android/vr_shell/color_scheme.cc
+++ b/chrome/browser/android/vr_shell/color_scheme.cc
@@ -28,6 +28,14 @@ void InitializeColorSchemes() {
   normal_scheme.foreground = 0xFF333333;
   normal_scheme.emphasized = 0xFF000000;
   normal_scheme.deemphasized = 0xFF5A5A5A;
+  normal_scheme.prompt_text = 0xCC000000;
+  normal_scheme.primary_button_background = 0xBFFFFFFF;
+  normal_scheme.secondary_button_background = 0x66FFFFFF;
+  normal_scheme.primary_button_text = 0xA6000000;
+  normal_scheme.secondary_button_text = 0xA6000000;
+  normal_scheme.button_background_down = 0xE6FFFFFF;
+  normal_scheme.button_background_hover = 0xFFFFFFFF;
+
   normal_scheme.separator = 0x33000000;
   normal_scheme.secure = gfx::kGoogleGreen700;
   normal_scheme.insecure = gfx::kGoogleRed700;
@@ -54,6 +62,13 @@ void InitializeColorSchemes() {
   incognito_scheme.background = 0xB9454545;
   incognito_scheme.background_hover = 0xD9454545;
   incognito_scheme.background_down = 0xFF454545;
+  incognito_scheme.prompt_text = 0xCC000000;
+  incognito_scheme.primary_button_background = 0xD9FFFFFF;
+  incognito_scheme.secondary_button_background = 0x80FFFFFF;
+  incognito_scheme.primary_button_text = 0xD9000000;
+  incognito_scheme.secondary_button_text = 0xD9000000;
+  incognito_scheme.button_background_hover = 0xFF8C8C8C;

[amp, 2017/06/02 21:53:59]

The normal background_hover and background_down (line 26-27) were supposed to be
for buttons as well (I think, but I'm not sure which buttons), but now we have a
specific button_xxx and I think it could get confusing which is used where.

Should we be even more specific here (ie prompt_button_background_hover)?  Or is
that overkill?

[ymalik, 2017/06/04 04:08:57]

I think being specific in this case definitely adds more clarity, so +1. Done.

+  incognito_scheme.button_background_down = 0xE6FFFFFF;
   incognito_scheme.separator = 0xDD595959;
   incognito_scheme.secure = incognito_scheme.foreground;
   incognito_scheme.insecure = incognito_scheme.foreground;