Fix parsing of mozVideoProjection parameter (#1412)

MozillaReality · Jul 17, 2019 · a22cc27 · a22cc27
1 parent ee5aba1
commit a22cc27
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 7 deletions.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/VideoProjectionMenuWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/VideoProjectionMenuWidget.java
@@ -145,16 +145,16 @@ public void setSelectedProjection(@VideoProjectionFlags int aProjection) {
 
         autoEnter.set(projection.endsWith("_auto"));
 
-        if (projection.startsWith("360")) {
-            return VIDEO_PROJECTION_360;
-        } else if (projection.startsWith("360s")) {
+        if (projection.startsWith("360s")) {
             return VIDEO_PROJECTION_360_STEREO;
-        } else if (projection.startsWith("180")) {
-            return VIDEO_PROJECTION_180;
+        } else if (projection.startsWith("360")) {
+            return VIDEO_PROJECTION_360;
         } else if (projection.startsWith("180lr")) {
             return VIDEO_PROJECTION_180_STEREO_LEFT_RIGHT;
         } else if (projection.startsWith("180tb")) {
             return VIDEO_PROJECTION_180_STEREO_TOP_BOTTOM;
+        } else if (projection.startsWith("180")) {
+            return VIDEO_PROJECTION_180;
         } else if (projection.startsWith("3d")) {
             return VIDEO_PROJECTION_3D_SIDE_BY_SIDE;
         }

diff --git a/app/src/main/assets/web_extensions/webcompat_youtube/main.js b/app/src/main/assets/web_extensions/webcompat_youtube/main.js
@@ -99,9 +99,17 @@ try {
     qs.delete('mozVideoProjection');
     switch (currentProjection) {
       case '360':
-        qs.set('mozVideoProjection', '360');
-        break;
       case '360_auto':
+      case '360s':
+      case '360s_auto':
+      case '180':
+      case '180_auto':
+      case '180lr':
+      case '180lr_auto':
+      case '180tb':
+      case '180tb_auto':
+        qs.set('mozVideoProjection', currentProjection);
+        break;
       default:
         qs.set('mozVideoProjection', auto ? '360_auto' : '360');
         break;