✨ [Story Preview] Allow amp-story-360 in preview mode (#38229)

coreymasanto · web-flow · commit b7954b6b839b · 2022-06-08T20:21:08.000Z
* Enable amp-story-360 and amp-story-captions in preview mode

* Create skeleton code for AmpStory360's previewAllowed()

* Update AmpStory360.previewAllowed() to allow preview mode as long as video is not being used as the 360 media source

* Revert allowing amp-story-captions because ensuring its tracks use cached sources is likely complex
diff --git a/extensions/amp-story-360/0.1/amp-story-360.js b/extensions/amp-story-360/0.1/amp-story-360.js
@@ -228,6 +228,16 @@ class CameraAnimation {
 }
 
 export class AmpStory360 extends AMP.BaseElement {
+  /** @override  */
+  static previewAllowed(element) {
+    // We can assume that images are cached, but the same is not necessarily
+    // true for videos. We only allow preview mode for `AmpStory360` when it
+    // uses cached sources, because requests for origin sources cannot be made
+    // due to privacy concerns.
+    const usesVideo = element.querySelector('amp-video');
+    return !usesVideo;
+  }
+
   /** @param {!AmpElement} element */
   constructor(element) {
     super(element);
