VR: Disable system indicators in full-screen mode.

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/memory/ptr_util.h"
 #include "base/test/scoped_task_environment.h"
 #include "chrome/browser/android/vr_shell/ui_browser_interface.h"
@@ skipping 59 matching lines @@
     scene_ = base::MakeUnique<UiScene>();
     manager_ = base::MakeUnique<UiSceneManager>(
         browser_.get(), scene_.get(), kNotInCct, kInWebVr, kAutopresented);
   }
 
   bool IsVisible(UiElementDebugId debug_id) {
     UiElement* element = scene_->GetUiElementByDebugId(debug_id);
     return element ? element->visible() : false;
   }
 
+  // Verify that only the elements in the set are visible.
   void VerifyElementsVisible(const std::string& debug_name,
-                             const std::set<UiElementDebugId>& elements) {
+                             const std::set<UiElementDebugId>& debug_ids) {
     SCOPED_TRACE(debug_name);
     for (const auto& element : scene_->GetUiElements()) {
       SCOPED_TRACE(element->debug_id());
       bool should_be_visible =
-          elements.find(element->debug_id()) != elements.end();
+          debug_ids.find(element->debug_id()) != debug_ids.end();
       EXPECT_EQ(should_be_visible, element->visible());
     }
   }
 
+  // Return false if not all elements in the set match the specified visibility
+  // state. Other elements are ignored.
+  bool VerifyVisibility(const std::set<UiElementDebugId>& debug_ids,
+                        bool visible) {
+    for (const auto& element : scene_->GetUiElements()) {
+      if (debug_ids.find(element->debug_id()) != debug_ids.end() &&
+          element->visible() != visible) {
+        return false;
+      }
+    }
+    return true;
+  }
+
   base::test::ScopedTaskEnvironment scoped_task_environment_;
   std::unique_ptr<MockBrowserInterface> browser_;
   std::unique_ptr<UiScene> scene_;
   std::unique_ptr<UiSceneManager> manager_;
 };
 
 TEST_F(UiSceneManagerTest, ExitPresentAndFullscreenOnAppButtonClick) {
   MakeManager(kNotInCct, kInWebVr);
 
   // Clicking app button should trigger to exit presentation.
@@ skipping 250 matching lines @@
 
   // Transition to WebVR mode
   manager_->SetWebVrMode(true, false, false);
   manager_->SetWebVrSecureOrigin(true);
 
   // All elements should be hidden.
   VerifyElementsVisible("Elements hidden", std::set<UiElementDebugId>{});
 }
 
 TEST_F(UiSceneManagerTest, CaptureIndicatorsVisibility) {
+  const std::set<UiElementDebugId> indicators = {
+      kAudioCaptureIndicator, kVideoCaptureIndicator, kScreenCaptureIndicator,
+      kLocationAccessIndicator,
+  };
+
   MakeManager(kNotInCct, kNotInWebVr);
-  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kLocationAccessIndicator));
+  EXPECT_TRUE(VerifyVisibility(indicators, false));
 
   manager_->SetAudioCapturingIndicator(true);

[amp, 2017/06/23 18:36:59]

Does anything different happen if some of the indicators are enabled and some
are disabled?

[cjgrant, 2017/06/23 19:38:26]

Nope, but, Amir and I talked about additional testing to make sure that enabling
one at a time shows one at a time.  Basically to make sure the plumbing from API
to individual indicator is correct, but that seems slightly too trivial to be
valuable.  A la "1 + 2 = 3.  Unit test that 1 + 2 really is 3".

   manager_->SetVideoCapturingIndicator(true);
   manager_->SetScreenCapturingIndicator(true);
   manager_->SetLocationAccessIndicator(true);
-
-  EXPECT_TRUE(IsVisible(kAudioCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kVideoCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kScreenCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kLocationAccessIndicator));
+  EXPECT_TRUE(VerifyVisibility(indicators, true));
 
   manager_->SetWebVrMode(true, false, false);
-  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kLocationAccessIndicator));
+  EXPECT_TRUE(VerifyVisibility(indicators, false));
 
   manager_->SetWebVrMode(false, false, false);
-  EXPECT_TRUE(IsVisible(kAudioCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kVideoCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kScreenCaptureIndicator));
-  EXPECT_TRUE(IsVisible(kLocationAccessIndicator));
+  EXPECT_TRUE(VerifyVisibility(indicators, true));
+
+  manager_->SetFullscreen(true);

[asimjour1, 2017/06/23 13:28:41]

We miss the transition from Fullscreen to WebVr.

[cjgrant, 2017/06/23 19:38:26]

Discussed offline.  The intent isn't really to cover all transitions, as much as
transitions that should cause shows/hides.

+  EXPECT_TRUE(VerifyVisibility(indicators, false));
+  manager_->SetFullscreen(false);
+  EXPECT_TRUE(VerifyVisibility(indicators, true));
 
   manager_->SetAudioCapturingIndicator(false);
   manager_->SetVideoCapturingIndicator(false);
   manager_->SetScreenCapturingIndicator(false);
   manager_->SetLocationAccessIndicator(false);
+  EXPECT_TRUE(VerifyVisibility(indicators, false));
+}
 
-  EXPECT_FALSE(IsVisible(kAudioCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kVideoCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kScreenCaptureIndicator));
-  EXPECT_FALSE(IsVisible(kLocationAccessIndicator));
-}
 }  // namespace vr_shell