commit 989ae759f8037fc5427ce8c289a3f49151080df4
author Svetoslav Ganov <svetoslavganov@google.com> Tue Sep 25 15:16:48 2012 -0700
committer Svetoslav Ganov <svetoslavganov@google.com> Tue Sep 25 15:16:54 2012 -0700
tree 9b97710d273ebf79e69a664196a976f69facbe54
parent 1ab8a08a9b85aa62045a6a78fd93859382c88b0f

Fixing argument expectations of View.requestRectangleOnScreen

1. In a previous patch the implementation of the method was fixed
   in terms of end result but this broke some assumptions about the
   method arguments. This change updates the argument expectations.

bug:7172035

Change-Id: I76d738b1c74dfb1dd45fc667f3217911f1c10a5f

core/java/android/view/View.java

diff --git a/core/java/android/view/View.java b/core/java/android/view/View.java
index 30b8b85..ecacaca 100644
--- a/core/java/android/view/View.java
+++ b/core/java/android/view/View.java
@@ -4348,13 +4348,13 @@
      * @return Whether any parent scrolled.
      */
     public boolean requestRectangleOnScreen(Rect rectangle, boolean immediate) {
-        if (mAttachInfo == null) {
+        if (mParent == null) {
             return false;
         }
 
         View child = this;
 
-        RectF position = mAttachInfo.mTmpTransformRect;
+        RectF position = (mAttachInfo != null) ? mAttachInfo.mTmpTransformRect : new RectF();
         position.set(rectangle);
 
         ViewParent parent = mParent;