IPH - connect data saver previews

chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java

Index: chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java b/chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java
index 3a77d626ae97dbbdb6cea30160a659e4af36deb3..cad33b40f6eef674c1f399152c3071b694ee4f42 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/widget/textbubble/ViewAnchoredTextBubble.java
@@ -8,20 +8,28 @@ import android.content.Context;
 import android.graphics.Rect;
 import android.support.annotation.StringRes;
 import android.view.View;
-import android.view.View.OnLayoutChangeListener;
+import android.view.ViewTreeObserver;
 
 import org.chromium.base.ApiCompatibilityUtils;
+import org.chromium.content.browser.PositionObserver;
+import org.chromium.content.browser.ViewPositionObserver;
 
 /**
  * A helper class that anchors a {@link TextBubble} to a particular {@link View}.  The bubble will
  * listen to layout events on the {@link View} and update accordingly.
  */
-public class ViewAnchoredTextBubble extends TextBubble implements OnLayoutChangeListener {
+public class ViewAnchoredTextBubble extends TextBubble

[David Trainor- moved to gerrit, 2017/04/12 08:36:17]

Might be worth pulling changes to this file out to another CL.

+        implements PositionObserver.Listener, ViewTreeObserver.OnGlobalLayoutListener,
+                   View.OnAttachStateChangeListener {
     private final int[] mCachedScreenCoordinates = new int[2];
     private final Rect mAnchorRect = new Rect();
     private final Rect mInsetRect = new Rect();
     private final View mAnchorView;
 
+    private final ViewPositionObserver mViewPositionObserver;
+
+    private ViewTreeObserver mViewTreeObserver;
+
     /**
      * Creates an instance of a {@link ViewAnchoredTextBubble}.
      * @param context    Context to draw resources from.
@@ -31,6 +39,8 @@ public class ViewAnchoredTextBubble extends TextBubble implements OnLayoutChange
     public ViewAnchoredTextBubble(Context context, View anchorView, @StringRes int stringId) {
         super(context, anchorView.getRootView(), stringId);
         mAnchorView = anchorView;
+
+        mViewPositionObserver = new ViewPositionObserver(mAnchorView);
     }
 
     /**
@@ -45,29 +55,50 @@ public class ViewAnchoredTextBubble extends TextBubble implements OnLayoutChange
     // TextBubble implementation.
     @Override
     public void show() {
-        mAnchorView.addOnLayoutChangeListener(this);
+        mViewPositionObserver.addListener(this);
+        mAnchorView.addOnAttachStateChangeListener(this);
+
+        mViewTreeObserver = mAnchorView.getViewTreeObserver();
+        mViewTreeObserver.addOnGlobalLayoutListener(this);
+
         refreshAnchorBounds();
         super.show();
     }
 
     @Override
-    public void dismiss() {
-        super.dismiss();
-        mAnchorView.removeOnLayoutChangeListener(this);
+    public void onDismiss() {
+        super.onDismiss();
+
+        mViewPositionObserver.removeListener(this);
+        mAnchorView.removeOnAttachStateChangeListener(this);
+
+        if (mViewTreeObserver != null && mViewTreeObserver.isAlive()) {
+            mViewTreeObserver.removeOnGlobalLayoutListener(this);
+        }
+        mViewTreeObserver = null;
     }
 
-    // OnLayoutChangeListener implementation.
+    // ViewTreeObserver.OnGlobalLayoutListener implementation.
     @Override
-    public void onLayoutChange(View v, int left, int top, int right, int bottom, int oldLeft,
-            int oldTop, int oldRight, int oldBottom) {
-        if (!mAnchorView.isShown()) {
-            dismiss();
-            return;
-        }
+    public void onGlobalLayout() {
+        if (mAnchorView.getVisibility() != View.VISIBLE) dismiss();
+    }
 
+    // PositionObserver.Listener implementation.
+    @Override
+    public void onPositionChanged(int positionX, int positionY) {
         refreshAnchorBounds();
     }
 
+    // View.OnAttachStateChangedObserver implementation.
+    @Override
+    public void onViewAttachedToWindow(View v) {}
+
+    @Override
+    public void onViewDetachedFromWindow(View v) {
+        dismiss();
+    }
+
     private void refreshAnchorBounds() {
         mAnchorView.getLocationOnScreen(mCachedScreenCoordinates);
         mAnchorRect.left = mCachedScreenCoordinates[0] + mInsetRect.left;