android_webview: Let AwContents manage TouchHandleDrawable

android_webview/java/src/org/chromium/android_webview/AwContents.java

Index: android_webview/java/src/org/chromium/android_webview/AwContents.java
diff --git a/android_webview/java/src/org/chromium/android_webview/AwContents.java b/android_webview/java/src/org/chromium/android_webview/AwContents.java
index a3dbd95e3576653464aabdc50a83f32b418d5df6..4eceb10ce4ec85a96d08bd566b9674a5c2960e75 100644
--- a/android_webview/java/src/org/chromium/android_webview/AwContents.java
+++ b/android_webview/java/src/org/chromium/android_webview/AwContents.java
@@ -81,7 +81,9 @@ import java.io.File;
 import java.lang.annotation.Annotation;
 import java.net.MalformedURLException;
 import java.net.URL;
+import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -288,6 +290,8 @@ public class AwContents implements SmartClipProvider,
     private final AwSettings mSettings;
     private final ScrollAccessibilityHelper mScrollAccessibilityHelper;
 
+    private final List<PopupTouchHandleDrawable> mTouchHandleDrawables = new ArrayList<>();
+
     private boolean mIsPaused;
     private boolean mIsViewVisible;
     private boolean mIsWindowVisible;
@@ -934,6 +938,9 @@ public class AwContents implements SmartClipProvider,
             mAwPdfExporter.setContainerView(mContainerView);
         }
         mWebContentsDelegate.setContainerView(mContainerView);
+        for (PopupTouchHandleDrawable drawable: mTouchHandleDrawables) {
+            drawable.onContainerViewChanged(newContainerView);
+        }
         onContainerViewChanged();
     }
 
@@ -2681,6 +2688,11 @@ public class AwContents implements SmartClipProvider,
         mFavicon = bitmap;
     }
 
+    @CalledByNative
+    private void onCreatedTouchHandle(PopupTouchHandleDrawable drawable) {
+        mTouchHandleDrawables.add(drawable);

[boliu, 2016/08/24 23:35:30]

when are things ever removed from the list?

[Jinsuk Kim, 2016/08/25 07:32:21]

Good catch. Drawable's is ultimately owned by RWHVA through
TouchSelectionController. I may be wrong but I think it is safe to clear the
list when AwContent is destroyed since the elements will be gone by then anyway.

[boliu, 2016/08/26 04:15:44]

That's not good enough. There can be an unbounded number of selection handles
created in the lifetime of AwContents.

Sounds like it should be removed when RWHVA destroys the native side?

+    }
+
     /** Callback for generateMHTML. */
     @CalledByNative
     private static void generateMHTMLCallback(