Refactor Samsung SmartClip implementation.

content/public/android/java/src/org/chromium/content_public/browser/WebContents.java

Index: content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
diff --git a/content/public/android/java/src/org/chromium/content_public/browser/WebContents.java b/content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
index 22bb994a982b845057fc8f3963a109d7e7e9f5ea..f09d934fa8e637da2dccb4ecec4560a94714cf3b 100644
--- a/content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
+++ b/content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
@@ -6,6 +6,7 @@ package org.chromium.content_public.browser;
 
 import android.graphics.Bitmap;
 import android.graphics.Rect;
+import android.os.Handler;
 import android.os.Parcelable;
 
 import org.chromium.base.VisibleForTesting;
@@ -297,6 +298,10 @@ public interface WebContents extends Parcelable {
      */
     int getThemeColor();
 
+    void requestSmartClipExtract(int x, int y, int width, int height);

[boliu, 2017/02/06 13:08:09]

javadoc

[aelias_OOO_until_Jul13, 2017/02/07 20:48:13]

Done.

+
+    void setSmartClipResultHandler(final Handler smartClipHandler);

[boliu, 2017/02/06 13:08:09]

final here is kinda leaking implementation into interface, write it such that it
doesn't need to be declared final in the interface?

[aelias_OOO_until_Jul13, 2017/02/07 20:48:13]

The final is already baked into the """public""" API in
https://cs.chromium.org/chromium/src/content/public/android/java/src/org/chro...
which I can't change, may as well keep the internal API consistent with it as
well.

+
     /**
      * Requests a snapshop of accessibility tree. The result is provided asynchronously
      * using the callback