VR: Make webVR insecure content warning look more like M58 warning.

chrome/browser/android/vr_shell/ui_scene_manager.cc

Index: chrome/browser/android/vr_shell/ui_scene_manager.cc
diff --git a/chrome/browser/android/vr_shell/ui_scene_manager.cc b/chrome/browser/android/vr_shell/ui_scene_manager.cc
index 4cbeae41985937e6e5b8f066a9e8f03afad5c1d7..3ec21acdec72b97b25cbce208044374b644d87a8 100644
--- a/chrome/browser/android/vr_shell/ui_scene_manager.cc
+++ b/chrome/browser/android/vr_shell/ui_scene_manager.cc
@@ -19,10 +19,10 @@ namespace {
 static constexpr int kWarningTimeoutSeconds = 30;
 static constexpr float kWarningDistance = 0.7;
 static constexpr float kWarningAngleRadians = 16.3 * M_PI / 180.0;
-static constexpr float kPermanentWarningHeight = 0.226;
-static constexpr float kPermanentWarningWidth = 0.078;
-static constexpr float kTransientWarningHeight = 0.512;
-static constexpr float kTransientWarningWidth = 0.160;
+static constexpr float kPermanentWarningHeight = 0.070f;
+static constexpr float kPermanentWarningWidth = 0.224f;
+static constexpr float kTransientWarningHeight = 0.160;
+static constexpr float kTransientWarningWidth = 0.512;
 
 static constexpr float kContentWidth = 2.4;
 static constexpr float kContentHeight = 1.6;
@@ -185,7 +185,7 @@ void UiSceneManager::SetWebVRSecureOrigin(bool secure) {
 }
 
 void UiSceneManager::ConfigureSecurityWarnings() {
-  bool enabled = web_vr_mode_ && !secure_origin_;
+  bool enabled = web_vr_mode_ && secure_origin_;

[amp, 2017/04/28 19:36:50]

Wait, why are we changing the logic here?  It looks to be backwards now?

[mthiesse, 2017/04/28 19:42:15]

oooops. For debugging, good catch.

   permanent_security_warning_->visible = enabled;
   transient_security_warning_->visible = enabled;
   if (enabled) {