Add back Rect parcelable to onSmartClipDataExtracted.

android_webview/java/src/org/chromium/android_webview/AwContents.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.android_webview;
 
 import android.annotation.SuppressLint;
 import android.annotation.TargetApi;
 import android.app.Activity;
 import android.content.ComponentCallbacks2;
@@ skipping 2996 matching lines @@
         }
 
         Log.e(TAG, "Unable to auto generate archive name for path: %s", baseName);
         return null;
     }
 
     @Override
     public void extractSmartClipData(int x, int y, int width, int height) {
         float dpi = mContentViewCore.getRenderCoordinates().getDeviceScaleFactor();
         if (!isDestroyedOrNoOperation(WARN)) {
-            mWebContents.requestSmartClipExtract(
+            mWebContents.requestSmartClipExtract(new Rect(x, y, x + width, y + height),

[David Trainor- moved to gerrit, 2017/04/26 20:58:17]

It looks like rect isn't scaled by dpi but the other values are.  Is this on
purpose/expected?  If it is, can we put something in the API explaining this (or
just make it all non-dpi scaled and do the scaling inside WebContents)?

                     (int) (x / dpi), (int) (y / dpi), (int) (width / dpi), (int) (height / dpi));
         }
     }
 
     @Override
     public void setSmartClipResultHandler(final Handler resultHandler) {
         if (isDestroyedOrNoOperation(WARN)) return;
 
         mWebContents.setSmartClipResultHandler(resultHandler);
     }
@@ skipping 455 matching lines @@
 
     private native void nativePreauthorizePermission(long nativeAwContents, String origin,
             long resources);
 
     private native void nativePostMessageToFrame(long nativeAwContents, String frameId,
             String message, String targetOrigin, MessagePort[] ports);
 
     private native void nativeGrantFileSchemeAccesstoChildProcess(long nativeAwContents);
     private native void nativeResumeLoadingCreatedPopupWebContents(long nativeAwContents);
 }