Do not calculate composition bounds until IME requests.

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;
@@ skipping 118 matching lines @@
     /**
      * Sets positional information of composing text as an array of character bounds.
      * @param compositionCharacterBounds Array of character bounds in local coordinates.
      */
     public void setCompositionCharacterBounds(float[] compositionCharacterBounds) {
         if (!mIsEditable) return;
 
         if (!Arrays.equals(compositionCharacterBounds, mCompositionCharacterBounds)) {
             mLastCursorAnchorInfo = null;
             mCompositionCharacterBounds = compositionCharacterBounds;
         }

[kinaba, 2016/07/05 08:50:09]

We may have to invoke updateCursorAnchorInfo() from here.

With this change, on the first IMMEDIATE case we don't receive onUpdateFrameInfo
and only setCompositionCharacterBounds comes.

[Seigo Nonaka, 2016/07/06 03:25:56]

Done.

     }
 
     /**
      * Sets coordinates system parameters and selection marker information.
      * @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.
@@ skipping 41 matching lines @@
         }
 
         // Notify to IME if there is a pending request, or if it is in monitor mode and we have
         // some change in the state.
         if (mHasPendingImmediateRequest
                 || (mMonitorModeEnabled && mLastCursorAnchorInfo == null)) {
             updateCursorAnchorInfo(view);
         }
     }
 
-    /**
-     * Resets the current state on update monitoring mode to the default (= do nothing.)
-     */
-    public void resetMonitoringState() {
-        mMonitorModeEnabled = false;
-        mHasPendingImmediateRequest = false;
-    }
-
     public void focusedNodeChanged(boolean isEditable) {
         mIsEditable = isEditable;
         mCompositionCharacterBounds = null;
         mHasCoordinateInfo = false;
         mLastCursorAnchorInfo = null;
     }
 
     public boolean onRequestCursorUpdates(int cursorUpdateMode, View view) {
         if (!mIsEditable) return false;
 
@@ skipping 52 matching lines @@
             }
             mLastCursorAnchorInfo = mCursorAnchorInfoBuilder.build();
         }
 
         if (mInputMethodManagerWrapper != null) {
             mInputMethodManagerWrapper.updateCursorAnchorInfo(view, mLastCursorAnchorInfo);
         }
         mHasPendingImmediateRequest = false;
     }
 }