[vr] Add incognito coloring

chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/vr_shell/ui_scene_manager.h"
 
 #include "base/macros.h"
 #include "base/test/scoped_task_environment.h"
 #include "chrome/browser/android/vr_shell/ui_elements/ui_element.h"
 #include "chrome/browser/android/vr_shell/ui_elements/ui_element_debug_id.h"
@@ skipping 121 matching lines @@
 
   MakeManager(kNotInCct, kNotInWebVr);
   EXPECT_FALSE(IsVisible(kCloseButton));
 
   MakeManager(kInCct, kInWebVr);
   EXPECT_FALSE(IsVisible(kCloseButton));
   manager_->SetWebVrMode(false);
   EXPECT_TRUE(IsVisible(kCloseButton));
 }
 
+TEST_F(UiSceneManagerTest, UiUpdatesForIncognito) {
+  MakeManager(kNotInCct, kNotInWebVr);
+
+  // Hold onto the background color to make sure it changes.
+  SkColor initial_background = scene_->GetBackgroundColor();
+  manager_->SetFullscreen(true);

[amp, 2017/05/27 00:47:47]

Why do we need to check fullscreen?

Shouldn't we just set incognito and verify the colors between that and normal?

Although, cjgrant and I were discussing having one big test that went through
all the transitions.  I think I agree with him when he prefers that feature
specific tests should be more tightly focused.

[Ian Vollick, 2017/05/27 01:29:12]

I was hoping to show that incognito "wins". That is, if we were both full screen
and incognito, we should be showing the incognito colors.

+
+  {
+    SCOPED_TRACE("Entered Fullsceen");
+    // Make sure background has changed for fullscreen.
+    EXPECT_NE(initial_background, scene_->GetBackgroundColor());
+  }
+
+  SkColor fullscreen_background = scene_->GetBackgroundColor();
+
+  manager_->SetIncognito(true);
+
+  {
+    SCOPED_TRACE("Entered Incognito");
+    // Make sure background has changed for incognito.
+    EXPECT_NE(fullscreen_background, scene_->GetBackgroundColor());
+    EXPECT_NE(initial_background, scene_->GetBackgroundColor());
+  }
+
+  SkColor incognito_background = scene_->GetBackgroundColor();
+
+  manager_->SetIncognito(false);
+
+  {
+    SCOPED_TRACE("Exited Incognito");
+    EXPECT_EQ(fullscreen_background, scene_->GetBackgroundColor());
+  }
+
+  manager_->SetFullscreen(false);
+
+  {
+    SCOPED_TRACE("Exited Fullsceen");
+    EXPECT_EQ(initial_background, scene_->GetBackgroundColor());
+  }
+
+  manager_->SetIncognito(true);
+
+  {
+    SCOPED_TRACE("Entered Incognito");
+    EXPECT_EQ(incognito_background, scene_->GetBackgroundColor());
+  }
+
+  manager_->SetIncognito(false);
+
+  {
+    SCOPED_TRACE("Exited Incognito");
+    EXPECT_EQ(initial_background, scene_->GetBackgroundColor());
+  }
+}
+
 TEST_F(UiSceneManagerTest, UiUpdatesForFullscreenChanges) {
   std::set<UiElementDebugId> visible_in_browsing = {
       UiElementDebugId::kContentQuad,     UiElementDebugId::kBackplane,
       UiElementDebugId::kCeiling,         UiElementDebugId::kFloor,
       UiElementDebugId::kFloorGrid,       UiElementDebugId::kUrlBar,
       UiElementDebugId::kLoadingIndicator};
   std::set<UiElementDebugId> visible_in_fullscreen = {
       UiElementDebugId::kContentQuad, UiElementDebugId::kBackplane,
       UiElementDebugId::kCeiling, UiElementDebugId::kFloor,
       UiElementDebugId::kFloorGrid};
@@ skipping 101 matching lines @@
 
   manager_->SetAudioCapturingIndicator(false);
   manager_->SetVideoCapturingIndicator(false);
   manager_->SetScreenCapturingIndicator(false);
 
   EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
   EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
   EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
 }
 }  // namespace vr_shell