Add UI texture scaling

MozillaReality · Nov 26, 2018 · ea63530 · ea63530
1 parent b89303c
commit ea63530
Show file tree

Hide file tree

Showing 6 changed files with 34 additions and 10 deletions.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/VRBrowserActivity.java b/app/src/common/shared/org/mozilla/vrbrowser/VRBrowserActivity.java
@@ -441,8 +441,10 @@ public void onFrameAvailable(SurfaceTexture surfaceTexture) {
             }
             widget.setSurfaceTexture(aTexture, aWidth, aHeight);
             // Add widget to a virtual display for invalidation
-            if (((View)widget).getParent() == null) {
-                mWidgetContainer.addView((View) widget, new FrameLayout.LayoutParams(aWidth, aHeight));
+            View view = (View) widget;
+            if (view.getParent() == null) {
+                float scale = widget.getPlacement().textureScale;
+                mWidgetContainer.addView(view, new FrameLayout.LayoutParams((int) Math.ceil(aWidth / scale), (int) Math.ceil(aHeight / scale)));
             }
         });
     }
@@ -473,7 +475,8 @@ void dispatchCreateWidgetLayer(final int aHandle, final Surface aSurface, final
             View view = (View) widget;
             // Add widget to a virtual display for invalidation
             if (aSurface != null && view.getParent() == null) {
-                mWidgetContainer.addView(view, new FrameLayout.LayoutParams(aWidth, aHeight));
+                float scale = widget.getPlacement().textureScale;
+                mWidgetContainer.addView(view, new FrameLayout.LayoutParams((int) Math.ceil(aWidth / scale), (int) Math.ceil(aHeight / scale)));
             } else if (aSurface == null && view.getParent() != null) {
                 mWidgetContainer.removeView(view);
             }
@@ -486,13 +489,17 @@ void dispatchCreateWidgetLayer(final int aHandle, final Surface aSurface, final
     void handleMotionEvent(final int aHandle, final int aDevice, final boolean aPressed, final float aX, final float aY) {
         runOnUiThread(() -> {
             Widget widget = mWidgets.get(aHandle);
+            float scale = widget != null ? widget.getPlacement().textureScale : 1.0f;
+            final float x = aX / scale;
+            final float y = aY / scale;
+
             if (widget == null) {
-                MotionEventGenerator.dispatch(mRootWidget, aDevice, aPressed, aX, aY);
+                MotionEventGenerator.dispatch(mRootWidget, aDevice, aPressed, x, y);
             } else if (widget == mBrowserWidget && mBrowserWidget.getBorderWidth() > 0) {
                 final int border = mBrowserWidget.getBorderWidth();
-                MotionEventGenerator.dispatch(widget, aDevice, aPressed, aX - border, aY - border);
+                MotionEventGenerator.dispatch(widget, aDevice, aPressed, x - border, y - border);
             } else {
-                MotionEventGenerator.dispatch(widget, aDevice, aPressed, aX, aY);
+                MotionEventGenerator.dispatch(widget, aDevice, aPressed, x, y);
             }
         });
     }

diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/BrowserWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/BrowserWidget.java
@@ -87,6 +87,7 @@ protected void initializeWidgetPlacement(WidgetPlacement aPlacement) {
         aPlacement.anchorX = 0.5f;
         aPlacement.anchorY = 0.0f;
         aPlacement.visible = true;
+        aPlacement.textureScale = 1.0f;
     }
 
     @Override

diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/WidgetPlacement.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/WidgetPlacement.java
@@ -39,6 +39,7 @@ public WidgetPlacement(Context aContext) {
     public boolean showPointer = true;
     public boolean firstDraw = false;
     public boolean layer = true;
+    public float textureScale = 0.7f;
 
     public WidgetPlacement clone() {
         WidgetPlacement w = new WidgetPlacement();
@@ -67,6 +68,7 @@ public void copyFrom(WidgetPlacement w) {
         this.opaque = w.opaque;
         this.showPointer = w.showPointer;
         this.firstDraw = w.firstDraw;
+        this.textureScale = w.textureScale;
     }
 
     public int textureWidth() {

diff --git a/app/src/main/cpp/BrowserWorld.cpp b/app/src/main/cpp/BrowserWorld.cpp
@@ -690,8 +690,8 @@ BrowserWorld::AddWidget(int32_t aHandle, const WidgetPlacementPtr& aPlacement) {
     worldWidth = aPlacement->width * kWorldDPIRatio;
   }
 
-  int32_t textureWidth = (int32_t)(ceilf(aPlacement->width * aPlacement->density));
-  int32_t textureHeight = (int32_t)(ceilf(aPlacement->height * aPlacement->density));
+  const int32_t textureWidth = aPlacement->GetTextureWidth();
+  const int32_t textureHeight = aPlacement->GetTextureHeight();
 
   WidgetPtr widget;
   VRLayerQuadPtr layer;
@@ -734,8 +734,7 @@ BrowserWorld::UpdateWidget(int32_t aHandle, const WidgetPlacementPtr& aPlacement
 
   widget->SetPlacement(aPlacement);
   widget->ToggleWidget(aPlacement->visible);
-  widget->SetSurfaceTextureSize((int32_t)(ceilf(aPlacement->width * aPlacement->density)),
-                                (int32_t)(ceilf(aPlacement->height * aPlacement->density)));
+  widget->SetSurfaceTextureSize(aPlacement->GetTextureWidth(), aPlacement->GetTextureHeight());
 
   float worldWidth = 0.0f, worldHeight = 0.0f;
   widget->GetWorldSize(worldWidth, worldHeight);

diff --git a/app/src/main/cpp/WidgetPlacement.cpp b/app/src/main/cpp/WidgetPlacement.cpp
@@ -53,8 +53,19 @@ WidgetPlacement::FromJava(JNIEnv* aEnv, jobject& aObject) {
   GET_BOOLEAN_FIELD(showPointer);
   GET_BOOLEAN_FIELD(firstDraw);
   GET_BOOLEAN_FIELD(layer);
+  GET_FLOAT_FIELD(textureScale, "textureScale");
 
   return result;
 }
 
+int32_t
+WidgetPlacement::GetTextureWidth() const{
+  return (int32_t)ceilf(width * density * textureScale);
+}
+
+int32_t
+WidgetPlacement::GetTextureHeight() const {
+  return (int32_t)ceilf(height * density * textureScale);
+}
+
 }
diff --git a/app/src/main/cpp/WidgetPlacement.h b/app/src/main/cpp/WidgetPlacement.h
@@ -31,6 +31,10 @@ struct WidgetPlacement {
   bool showPointer;
   bool firstDraw;
   bool layer;
+  float textureScale;
+
+  int32_t GetTextureWidth() const;
+  int32_t GetTextureHeight() const;
 
   static WidgetPlacementPtr FromJava(JNIEnv* aEnv, jobject& aObject);
 private: