[vr] Close exit prompt when clicking on background

chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc

Index: chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
diff --git a/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc b/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
index 81c6c736ecf674c70074ad54919d761fb42af3ee..818ef491c7e43f22bf0f41317f52de2b3d916a9d 100644
--- a/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
+++ b/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
@@ -291,6 +291,42 @@ TEST_F(UiSceneManagerTest, UiUpdatesExitPrompt) {
   }
 }
 
+TEST_F(UiSceneManagerTest, BackplaneClickClosesExitPrompt) {
+  std::set<UiElementDebugId> visible_when_prompting = {kExitPrompt, kBackplane,
+                                                       kCeiling, kFloor};
+  MakeManager(kNotInCct, kNotInWebVr);
+
+  manager_->SetWebVrSecureOrigin(true);
+
+  // Initial state.
+  for (const auto& element : scene_->GetUiElements()) {
+    SCOPED_TRACE(element->debug_id());
+    bool should_be_visible =
+        kElementsVisibleInBrowsing.find(element->debug_id()) !=
+        kElementsVisibleInBrowsing.end();
+    EXPECT_EQ(should_be_visible, element->visible());
+  }
+
+  // Exit prompt visible state.
+  manager_->OnSecurityIconClicked();
+  for (const auto& element : scene_->GetUiElements()) {

[cjgrant, 2017/06/06 19:42:13]

Based on how many of these loops we have throughout tests, and the fact that
SCOPED_TRACE is our new friend, could we break out a module- or class-level
helper method, VerifyElementsVisible(std::set<elements>)?

This could probably be:
VerifyElementsVisible({kExitPrompt, kBackplane, kCeiling, kFloor});

[ymalik, 2017/06/06 21:11:58]

Done. Much cleaner.

+    SCOPED_TRACE(element->debug_id());
+    bool should_be_visible = visible_when_prompting.find(element->debug_id()) !=
+                             visible_when_prompting.end();
+    EXPECT_EQ(should_be_visible, element->visible());
+  }
+
+  // Back to initial state.
+  manager_->OnBackplaneClicked();
+  for (const auto& element : scene_->GetUiElements()) {

[cjgrant, 2017/06/06 19:42:13]

Originally I thought it's weird so many tests check which elements are visible
in normal browsing mode.  But I changed my mind, and I think it does make sense.
 If we used the helper method approach, doing so would be a one-liner. 
Thoughts?

[ymalik, 2017/06/06 21:11:58]

Agreed, we want to check the visibility of these elements with the case that a
unittest covers. 

+    SCOPED_TRACE(element->debug_id());
+    bool should_be_visible =
+        kElementsVisibleInBrowsing.find(element->debug_id()) !=
+        kElementsVisibleInBrowsing.end();
+    EXPECT_EQ(should_be_visible, element->visible());
+  }
+}
+
 TEST_F(UiSceneManagerTest, UiUpdatesForWebVR) {
   MakeManager(kNotInCct, kInWebVr);