Make WebView work for external displays (over Presentations).

ui/android/java/src/org/chromium/ui/gfx/DeviceDisplayInfo.java

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.ui.gfx;
 
 import android.annotation.TargetApi;
 import android.content.Context;
 import android.graphics.PixelFormat;
 import android.graphics.Point;
 import android.os.Build;
 import android.util.DisplayMetrics;
 import android.view.Display;
 import android.view.Surface;
 import android.view.WindowManager;
 
 import org.chromium.base.CalledByNative;
 import org.chromium.base.JNINamespace;
 
+import java.lang.ref.WeakReference;
+
 /**
  * This class facilitates access to android information typically only
  * available using the Java SDK, including {@link Display} properties.
  *
  * Currently the information consists of very raw display information (height, width, DPI scale)
  * regarding the main display.
  */
 @JNINamespace("gfx")
 public class DeviceDisplayInfo {
-
-    private final Context mAppContext;
+    private final WeakReference<Context> mContextRef;
     private final WindowManager mWinManager;
     private Point mTempPoint = new Point();
     private DisplayMetrics mTempMetrics = new DisplayMetrics();
 
-    private DeviceDisplayInfo(Context context) {
-        mAppContext = context.getApplicationContext();
-        mWinManager = (WindowManager) mAppContext.getSystemService(Context.WINDOW_SERVICE);
+    private DeviceDisplayInfo(WeakReference<Context> contextRef) {
+        mContextRef = contextRef;
+        Context context = mContextRef.get();
+        if (context == null)

[boliu, 2015/07/15 06:38:47]

Pass a strong ref in constructor, then this can't ever happen

[gsennton, 2015/07/15 19:26:21]

Unless the incoming context is null because we retrieved it from WindowAndroid
which only holds a weak reference to its display context (and therefore might
pass null)?

[boliu, 2015/07/16 14:36:56]

You can check for null before passing it into DeviceDisplayInfo then. And if it
is null, you'll have to handle that correctly too. Make a dummy
DeviceDisplayInfo, or check caller can handle null?

[gsennton, 2015/10/20 13:27:12]

Right, will check this in WindowAndroid::UpdateDeviceDisplayInfo and just use
the default DeviceDisplayInfo ctor if the context is null.

[boliu, 2015/10/20 15:07:22]

"use the default DeviceDisplayInfo" sounds wrong, but then maybe it doesn't
matter much since this can only be null momentarily during shutdown. So add a
comment if you decide to do that.

+            throw new RuntimeException("got null Context in DeviceDisplayInfo constructor");
+        mWinManager = (WindowManager) context.getSystemService(Context.WINDOW_SERVICE);
     }
 
     /**
      * @return Display height in physical pixels.
      */
     @CalledByNative
     public int getDisplayHeight() {
         getDisplay().getSize(mTempPoint);
         return mTempPoint.y;
     }
@@ skipping 99 matching lines @@
         getDisplay().getMetrics(mTempMetrics);
         return mTempMetrics.density;
     }
 
     /**
      * @return Smallest screen size in density-independent pixels that the
      *         application will see, regardless of orientation.
      */
     @CalledByNative
     private int getSmallestDIPWidth() {
-        return mAppContext.getResources().getConfiguration().smallestScreenWidthDp;
+        Context context = mContextRef.get();
+        if (context == null) throw new RuntimeException("null context in getSmallestDIPWidth!!!");
+        return context.getResources().getConfiguration().smallestScreenWidthDp;

[boliu, 2015/07/15 06:38:47]

I wonder if this can ever change. Maybe it's ok to cache the value in the
constructor.

Throwing exception is not ok. That's the same as crashing in a @CalledByNative
method, and an object going away under a weak pointer has to be handled without
crashing.

[jdduke (slow), 2015/07/15 16:03:07]

+1. It might also be worth investigating why this constant is needed, and if
it's absolutely necessary. Regardless, even if the value does change I don't
think we'd react to the change properly anyway, and I'd be OK storing it in the
constructor.

[gsennton, 2015/07/15 19:26:21]

Right, going ahead and storing the value in the constructor.

Checking with skobes@ whether getSmallestDIPWidth can be removed
(it was added in https://codereview.chromium.org/28053002).

[boliu, 2015/07/16 14:36:56]

You no longer need to keep mContextRef in this class then \o/

[gsennton, 2015/10/20 13:27:12]

will remove mContextRef

Regarding getSmallestDIPWidth, according to skobes@ the value is needed but can
probably be calculated in the following way

int smallestDIPWidth = min(info.GetPhysicalDisplayHeight(),
info.GetPhysicalDisplayWidth()) / info.GetDIPScale();

     }
 
     /**
      * @return the screen's rotation angle from its 'natural' orientation.
      * Expected values are one of { 0, 90, 180, 270 }.
      * See http://developer.android.com/reference/android/view/Display.html#getRotation()
      * for more information about Display.getRotation() behavior.
      */
     @CalledByNative
     public int getRotationDegrees() {
         switch (getDisplay().getRotation()) {
             case Surface.ROTATION_0:
                 return 0;
             case Surface.ROTATION_90:
                 return 90;
             case Surface.ROTATION_180:
                 return 180;
             case Surface.ROTATION_270:
                 return 270;
         }
 
         // This should not happen.
         assert false;
         return 0;
     }
 
-    /**
-     * Inform the native implementation to update its cached representation of
-     * the DeviceDisplayInfo values.
-     */
-    public void updateNativeSharedDisplayInfo() {
-        nativeUpdateSharedDeviceDisplayInfo(
-                getDisplayHeight(), getDisplayWidth(),
-                getPhysicalDisplayHeight(), getPhysicalDisplayWidth(),
-                getBitsPerPixel(), getBitsPerComponent(),
-                getDIPScale(), getSmallestDIPWidth(), getRotationDegrees());
-    }
-
     private Display getDisplay() {
         return mWinManager.getDefaultDisplay();
     }
 
     /**
      * Creates DeviceDisplayInfo for a given Context.
      *
      * @param context A context to use.
      * @return DeviceDisplayInfo associated with a given Context.
      */
     @CalledByNative
     public static DeviceDisplayInfo create(Context context) {
-        return new DeviceDisplayInfo(context);
+        return new DeviceDisplayInfo(new WeakReference<Context>(context));
     }
-
-    private native void nativeUpdateSharedDeviceDisplayInfo(
-            int displayHeight, int displayWidth,
-            int physicalDisplayHeight, int physicalDisplayWidth,
-            int bitsPerPixel, int bitsPerComponent, double dipScale,
-            int smallestDIPWidth, int rotationDegrees);
-
 }