Expose immersive-ar enum value in all cases

Bug: 1025992 Change-Id: If19cde37384656652a3dcbbf5331c13c794241f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1924886 Reviewed-by: Alexander Cooper <alcooper@chromium.org> Reviewed-by: Brandon Jones <bajones@chromium.org> Commit-Queue: Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#716805}
marcosholgado · Nov 19, 2019 · 34bab11 · 34bab11
1 parent e13dde5
commit 34bab11
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/third_party/blink/renderer/modules/xr/xr.cc b/third_party/blink/renderer/modules/xr/xr.cc
@@ -655,9 +655,7 @@ ScriptPromise XR::InternalIsSessionSupported(ScriptState* script_state,
 
   if (session_mode == XRSession::kModeImmersiveAR &&
       !RuntimeEnabledFeatures::WebXRARModuleEnabled(doc)) {
-    query->RejectWithTypeError(
-        String::Format(kImmersiveArModeNotValid, "supportsSession"),
-        &exception_state);
+    query->Resolve(false);
     return promise;
   }
 

diff --git a/third_party/blink/renderer/modules/xr/xr_session.idl b/third_party/blink/renderer/modules/xr/xr_session.idl
@@ -7,7 +7,7 @@
 enum XRSessionMode {
   "inline",
   "immersive-vr",
-  /* [RuntimeEnabled=WebXRARModule] */ "immersive-ar",
+  "immersive-ar",
 };
 
 enum XREnvironmentBlendMode {