Let IME frame update bypass ContentViewCore

content/public/android/java/src/org/chromium/content/browser/input/CursorAnchorInfoController.java

 // Copyright 2016 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.content.browser.input;
 
 import android.annotation.TargetApi;
 import android.graphics.Matrix;
 import android.os.Build;
 import android.view.View;
 import android.view.inputmethod.CursorAnchorInfo;
 
 import org.chromium.base.VisibleForTesting;
 import org.chromium.base.annotations.SuppressFBWarnings;
-import org.chromium.content.browser.RenderCoordinates;
 
 import java.util.Arrays;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
 /**
  * A state machine interface which receives Chromium internal events to determines when to call
  * {@link InputMethodManager#updateCursorAnchorInfo(View, CursorAnchorInfo)}. This interface is
  * also used in unit tests to mock out {@link CursorAnchorInfo}, which is available only in
@@ skipping 111 matching lines @@
             mLastCursorAnchorInfo = null;
             mCompositionCharacterBounds = compositionCharacterBounds;
             if (mHasCoordinateInfo) {
                 updateCursorAnchorInfo(view);
             }
         }
     }
 
     /**
      * Sets coordinates system parameters and selection marker information.
+     * @param scale device scale factor.
+     * @param contentOffsetYPix Y offset below the browser controls.
      * @param hasInsertionMarker {@code true} if the insertion marker exists.
      * @param isInsertionMarkerVisible {@code true} if the insertion insertion marker is visible.
      * @param insertionMarkerHorizontal X coordinate of the top of the first selection marker.
      * @param insertionMarkerTop Y coordinate of the top of the first selection marker.
      * @param insertionMarkerBottom Y coordinate of the bottom of the first selection marker.
      * @param view The attached view.
      */
     @SuppressFBWarnings("FE_FLOATING_POINT_EQUALITY")
-    public void onUpdateFrameInfo(@Nonnull RenderCoordinates renderCoordinates,
-            boolean hasInsertionMarker, boolean isInsertionMarkerVisible,
-            float insertionMarkerHorizontal, float insertionMarkerTop,
-            float insertionMarkerBottom, @Nonnull View view) {
+    public void onUpdateFrameInfo(float scale, float contentOffsetYPix, boolean hasInsertionMarker,
+            boolean isInsertionMarkerVisible, float insertionMarkerHorizontal,
+            float insertionMarkerTop, float insertionMarkerBottom, @Nonnull View view) {
         if (!mIsEditable) return;
 
         // Reuse {@param #mViewOrigin} to avoid object creation, as this method is supposed to be
         // called at relatively high rate.
         mViewDelegate.getLocationOnScreen(view, mViewOrigin);
 
         // Character bounds and insertion marker locations come in device independent pixels
         // relative from the top-left corner of the web view content area. (In other words, the
         // effects of various kinds of zooming and scrolling are already taken into account.)
         //
         // We need to prepare parameters that convert such values to physical pixels, in the
         // screen coordinate. Hence the following values are derived.
-        float scale = renderCoordinates.getDeviceScaleFactor();
         float translationX = mViewOrigin[0];
-        float translationY = mViewOrigin[1] + renderCoordinates.getContentOffsetYPix();
-
+        float translationY = mViewOrigin[1] + contentOffsetYPix;
         if (!mHasCoordinateInfo
                 || scale != mScale
                 || translationX != mTranslationX
                 || translationY != mTranslationY
                 || hasInsertionMarker != mHasInsertionMarker
                 || isInsertionMarkerVisible != mIsInsertionMarkerVisible
                 || insertionMarkerHorizontal != mInsertionMarkerHorizontal
                 || insertionMarkerTop != mInsertionMarkerTop
                 || insertionMarkerBottom != mInsertionMarkerBottom) {
             mLastCursorAnchorInfo = null;
@@ skipping 88 matching lines @@
             }
             mLastCursorAnchorInfo = mCursorAnchorInfoBuilder.build();
         }
 
         if (mInputMethodManagerWrapper != null) {
             mInputMethodManagerWrapper.updateCursorAnchorInfo(view, mLastCursorAnchorInfo);
         }
         mHasPendingImmediateRequest = false;
     }
 }