From: Jakub Gielzak <jgielzak@google.com>
Date: Tue, 5 Feb 2019 12:33:43 +0000 (+0000)
Subject: Restrict reflection access to mLayoutParams
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=79bceebe32fdaddd425a25d4e75eddffa15cee49;p=android-x86%2Fframeworks-base.git

Restrict reflection access to mLayoutParams

Users should rely on the getter / setter. The setter additionally
guarantees internal state correctness.

Bug: 123768937
Test: n/a

Change-Id: Ia2dcbe9db3fdeab8aeac9b80dcfaaa0932724dc2
---

diff --git a/core/java/android/view/View.java b/core/java/android/view/View.java
index 992b99617064..ab18fbdca85f 100644
--- a/core/java/android/view/View.java
+++ b/core/java/android/view/View.java
@@ -4140,9 +4140,13 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
      * The layout parameters associated with this view and used by the parent
      * {@link android.view.ViewGroup} to determine how this view should be
      * laid out.
+     *
+     * The field should not be used directly. Instead {@link #getLayoutParams()} and {@link
+     * #setLayoutParams(ViewGroup.LayoutParams)} should be used. The setter guarantees internal
+     * state correctness of the class.
      * {@hide}
      */
-    @UnsupportedAppUsage
+    @UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P)
     protected ViewGroup.LayoutParams mLayoutParams;
 
     /**