Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Unified Diff: chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc

Issue 2955483003: Show splash screen when entering VR from a deep-link (Closed)
Patch Set: fix compile error on bots Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/android/vr_shell/ui_scene_manager.cc ('k') | chrome/browser/android/vr_shell/vr_gl_thread.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 38b82414712b3d60651a222fd83c09b15bff338c..b9f81288f7bf4e72ab39374c838c9fe5ba38e771 100644
--- a/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
+++ b/chrome/browser/android/vr_shell/ui_scene_manager_unittest.cc
@@ -37,7 +37,6 @@ std::set<UiElementDebugId> kElementsVisibleInBrowsing = {
kContentQuad, kBackplane, kCeiling, kFloor, kUrlBar};
std::set<UiElementDebugId> kElementsVisibleWithExitPrompt = {
kExitPrompt, kExitPromptBackplane, kCeiling, kFloor};
-
} // namespace
class UiSceneManagerTest : public testing::Test {
@@ -69,7 +68,7 @@ class UiSceneManagerTest : public testing::Test {
void MakeAutoPresentedManager() {
scene_ = base::MakeUnique<UiScene>();
manager_ = base::MakeUnique<UiSceneManager>(
- browser_.get(), scene_.get(), kNotInCct, kInWebVr, kAutopresented);
+ browser_.get(), scene_.get(), kNotInCct, kNotInWebVr, kAutopresented);
}
bool IsVisible(UiElementDebugId debug_id) {
@@ -132,7 +131,7 @@ TEST_F(UiSceneManagerTest, WebVrWarningsShowWhenInitiallyInWebVr) {
EXPECT_TRUE(IsVisible(kWebVrPermanentHttpSecurityWarning));
EXPECT_TRUE(IsVisible(kWebVrTransientHttpSecurityWarning));
- manager_->SetWebVrMode(false, false, false);
+ manager_->SetWebVrMode(false, false);
EXPECT_FALSE(IsVisible(kWebVrPermanentHttpSecurityWarning));
EXPECT_FALSE(IsVisible(kWebVrTransientHttpSecurityWarning));
}
@@ -143,7 +142,7 @@ TEST_F(UiSceneManagerTest, WebVrWarningsDoNotShowWhenInitiallyOutsideWebVr) {
EXPECT_FALSE(IsVisible(kWebVrPermanentHttpSecurityWarning));
EXPECT_FALSE(IsVisible(kWebVrTransientHttpSecurityWarning));
- manager_->SetWebVrMode(true, false, false);
+ manager_->SetWebVrMode(true, false);
EXPECT_TRUE(IsVisible(kWebVrPermanentHttpSecurityWarning));
EXPECT_TRUE(IsVisible(kWebVrTransientHttpSecurityWarning));
}
@@ -160,19 +159,19 @@ TEST_F(UiSceneManagerTest, ToastVisibility) {
manager_->SetFullscreen(true);
EXPECT_TRUE(IsVisible(kPresentationToast));
- manager_->SetWebVrMode(true, false, true);
+ manager_->SetWebVrMode(true, true);
EXPECT_TRUE(IsVisible(kPresentationToast));
- manager_->SetWebVrMode(false, false, false);
+ manager_->SetWebVrMode(false, false);
EXPECT_FALSE(IsVisible(kPresentationToast));
manager_->SetFullscreen(false);
EXPECT_FALSE(IsVisible(kPresentationToast));
- manager_->SetWebVrMode(true, false, false);
+ manager_->SetWebVrMode(true, false);
EXPECT_FALSE(IsVisible(kPresentationToast));
- manager_->SetWebVrMode(false, false, true);
+ manager_->SetWebVrMode(false, true);
EXPECT_TRUE(IsVisible(kPresentationToast));
}
@@ -194,7 +193,7 @@ TEST_F(UiSceneManagerTest, CloseButtonVisibleInCctFullscreen) {
// Button should not be visible when in WebVR.
MakeManager(kInCct, kInWebVr);
EXPECT_FALSE(IsVisible(kCloseButton));
- manager_->SetWebVrMode(false, false, false);
+ manager_->SetWebVrMode(false, false);
EXPECT_TRUE(IsVisible(kCloseButton));
// Button should be visible in Cct across transistions in fullscreen.
@@ -261,23 +260,17 @@ TEST_F(UiSceneManagerTest, UiUpdatesForIncognito) {
}
}
-TEST_F(UiSceneManagerTest, WebVrAutopresentedInitially) {
- MakeAutoPresentedManager();
- manager_->SetWebVrSecureOrigin(true);
- VerifyElementsVisible("Autopresented",
- std::set<UiElementDebugId>{kTransientUrlBar});
-}
-
TEST_F(UiSceneManagerTest, WebVrAutopresented) {
- MakeManager(kNotInCct, kNotInWebVr);
+ MakeAutoPresentedManager();
manager_->SetWebVrSecureOrigin(true);
- // Initial state.
- VerifyElementsVisible("Initial", kElementsVisibleInBrowsing);
+ // Initially, we should only show the splash screen.
+ VerifyElementsVisible("Initial",
+ std::set<UiElementDebugId>{kSplashScreenIcon});
// Enter WebVR with autopresentation.
- manager_->SetWebVrMode(true, true, false);
+ manager_->SetWebVrMode(true, false);
VerifyElementsVisible("Autopresented",
std::set<UiElementDebugId>{kTransientUrlBar});
@@ -370,7 +363,7 @@ TEST_F(UiSceneManagerTest, UiUpdateTransitionToWebVR) {
manager_->SetLocationAccessIndicator(true);
// Transition to WebVR mode
- manager_->SetWebVrMode(true, false, false);
+ manager_->SetWebVrMode(true, false);
manager_->SetWebVrSecureOrigin(true);
// All elements should be hidden.
@@ -393,9 +386,9 @@ TEST_F(UiSceneManagerTest, CaptureIndicatorsVisibility) {
EXPECT_TRUE(VerifyVisibility(indicators, true));
// Go into non-browser modes and make sure all indicators are hidden.
- manager_->SetWebVrMode(true, false, false);
+ manager_->SetWebVrMode(true, false);
EXPECT_TRUE(VerifyVisibility(indicators, false));
- manager_->SetWebVrMode(false, false, false);
+ manager_->SetWebVrMode(false, false);
manager_->SetFullscreen(true);
EXPECT_TRUE(VerifyVisibility(indicators, false));
manager_->SetFullscreen(false);
« no previous file with comments | « chrome/browser/android/vr_shell/ui_scene_manager.cc ('k') | chrome/browser/android/vr_shell/vr_gl_thread.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698