VR: Update UiSceneManager with screen capturing flag

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 182 matching lines @@
     bool should_be_visible = visible_in_browsing.find(element->debug_id()) !=
                              visible_in_browsing.end();
     EXPECT_EQ(should_be_visible, element->visible());
   }
   {
     SCOPED_TRACE("Exited Fullsceen");
     EXPECT_TRUE(ColorEquals(initial_background, scene_->GetBackgroundColor()));
   }
 }
 
+TEST_F(UiSceneManagerTest, UiUpdatesForWebVR) {
+  MakeManager(kNotInCct, kInWebVr);
+
+  manager_->SetWebVrSecureOrigin(true);
+  manager_->SetAudioCapturingIndicator(true);
+  manager_->SetVideoCapturingIndicator(true);
+  manager_->SetScreenCapturingIndicator(true);
+
+  // All elements should be hidden.
+  for (const auto& element : scene_->GetUiElements()) {
+    SCOPED_TRACE(element->debug_id());
+    EXPECT_FALSE(element->visible());
+  }
+}
+
+TEST_F(UiSceneManagerTest, UiUpdateTransitionToWebVR) {
+  MakeManager(kNotInCct, kNotInWebVr);
+  manager_->SetAudioCapturingIndicator(true);
+  manager_->SetVideoCapturingIndicator(true);
+  manager_->SetScreenCapturingIndicator(true);
+
+  // Transition to WebVR mode
+  manager_->SetWebVrMode(true);
+  manager_->SetWebVrSecureOrigin(true);
+
+  // All elements should be hidden.
+  for (const auto& element : scene_->GetUiElements()) {
+    SCOPED_TRACE(element->debug_id());
+    EXPECT_FALSE(element->visible());
+  }

[amp, 2017/05/26 17:26:01]

Should we test that transitioning out of WebVR makes everything visible here?

This is kind of already verified by the test below (line 254 and after), but
that test starts with WebVR disabled, enables it and then disables it, where
this test starts with it enabled and then disables it.  There is a possibility
that they could be hitting a different path that way.

OTOH feel free to ignore me if you feel this is too pedantic. :)

[cjgrant, 2017/05/26 17:33:35]

I still feel the same about this as before.  I think testing non-WebVR
visibility is a bigger problem, and should be separate tests.  These tests are
just meant to cover "wherever you were before, when you get into WebVR, nothing
should show".  That a different intent that Amir's test below, which verifies
the indicators across multiple modes.

So my opinion is that this is good as is, and when we have time, we should add a
"regular browsing" test that does heavy testing of the browsing elements, goes
into WebVR and back out, reverifies, etc"..

[amp, 2017/05/26 17:55:30]

That makes sense.  I agree a single large test that goes through all the modes
makes more sense that adding those parts piecemeal into each of the other parts.

So yea this is good as is, we'll revisit those bigger tests in a future change.

+}
+
+TEST_F(UiSceneManagerTest, CaptureIndicatorsVisibility) {
+  MakeManager(kNotInCct, kNotInWebVr);
+  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
+
+  manager_->SetAudioCapturingIndicator(true);
+  manager_->SetVideoCapturingIndicator(true);
+  manager_->SetScreenCapturingIndicator(true);
+
+  EXPECT_TRUE(IsVisible(kAudioCaptureIndicator));
+  EXPECT_TRUE(IsVisible(kVideoCaptureIndicator));
+  EXPECT_TRUE(IsVisible(kScreenCaptureIndicator));
+
+  manager_->SetWebVrMode(true);
+  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
+
+  manager_->SetWebVrMode(false);
+  EXPECT_TRUE(IsVisible(kAudioCaptureIndicator));
+  EXPECT_TRUE(IsVisible(kVideoCaptureIndicator));
+  EXPECT_TRUE(IsVisible(kScreenCaptureIndicator));
+
+  manager_->SetAudioCapturingIndicator(false);
+  manager_->SetVideoCapturingIndicator(false);
+  manager_->SetScreenCapturingIndicator(false);
+
+  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
+  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
+}
 }  // namespace vr_shell