Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant calls to invalidateParentIfNeeded

Browse Source 
Test: CtsViewTestCases

setLeft, setTop, setRight, and setBottom called it once, and then
potentially called it again. The second call is unnecessary.

Change-Id: If92d4b786d4680024614042cd8905ae56ea1c9e9
This commit is contained in:
Leon Scroggins III
2019-04-05 13:15:08 -04:00
parent 39a9df4de8
commit b199568a4a
1 changed files with 0 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
core/java/android/view/View.java
View File

@@ -16815,10 +16815,6 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
mForegroundInfo.mBoundsChanged = true;
}
invalidateParentIfNeeded();
if ((mPrivateFlags2 & PFLAG2_VIEW_QUICK_REJECTED) == PFLAG2_VIEW_QUICK_REJECTED) {
// View was rejected last time it was drawn by its parent; this may have changed
invalidateParentIfNeeded();
}
}
}
@@ -16884,10 +16880,6 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
mForegroundInfo.mBoundsChanged = true;
}
invalidateParentIfNeeded();
if ((mPrivateFlags2 & PFLAG2_VIEW_QUICK_REJECTED) == PFLAG2_VIEW_QUICK_REJECTED) {
// View was rejected last time it was drawn by its parent; this may have changed
invalidateParentIfNeeded();
}
}
}
@@ -16947,10 +16939,6 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
mForegroundInfo.mBoundsChanged = true;
}
invalidateParentIfNeeded();
if ((mPrivateFlags2 & PFLAG2_VIEW_QUICK_REJECTED) == PFLAG2_VIEW_QUICK_REJECTED) {
// View was rejected last time it was drawn by its parent; this may have changed
invalidateParentIfNeeded();
}
}
}
@@ -17007,10 +16995,6 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
mForegroundInfo.mBoundsChanged = true;
}
invalidateParentIfNeeded();
if ((mPrivateFlags2 & PFLAG2_VIEW_QUICK_REJECTED) == PFLAG2_VIEW_QUICK_REJECTED) {
// View was rejected last time it was drawn by its parent; this may have changed
invalidateParentIfNeeded();
}
}
}