Fix bug #6427629 Clean up layout direction APIs

- rename getResolvedTextDirection() to getTextDirection()

Change-Id: Id2a6025daf5521dcd676e454fc6bb9955fdccf2d

Author: Fabrice Di Meglio

api/current.txt

@@ -24899,7 +24899,6 @@ package android.view {
     method public float getPivotX();
     method public float getPivotY();
     method public int getResolvedTextAlignment();
-    method public int getResolvedTextDirection();
     method public android.content.res.Resources getResources();
     method public final int getRight();
     method protected float getRightFadingEdgeStrength();

core/java/android/view/View.java

@@ -16496,6 +16496,8 @@ protected float getHorizontalScrollFactor() {
      * {@link #TEXT_DIRECTION_LTR},
      * {@link #TEXT_DIRECTION_RTL},
      * {@link #TEXT_DIRECTION_LOCALE}
+     *
+     * @hide
      */
     @ViewDebug.ExportedProperty(category = "text", mapping = {
             @ViewDebug.IntToString(from = TEXT_DIRECTION_INHERIT, to = "INHERIT"),
@@ -16505,7 +16507,7 @@ protected float getHorizontalScrollFactor() {
             @ViewDebug.IntToString(from = TEXT_DIRECTION_RTL, to = "RTL"),
             @ViewDebug.IntToString(from = TEXT_DIRECTION_LOCALE, to = "LOCALE")
     })
-    public int getTextDirection() {
+    public int getRawTextDirection() {
         return (mPrivateFlags2 & PFLAG2_TEXT_DIRECTION_MASK) >> PFLAG2_TEXT_DIRECTION_MASK_SHIFT;
     }
 
@@ -16522,7 +16524,7 @@ public int getTextDirection() {
      * {@link #TEXT_DIRECTION_LOCALE}
      */
     public void setTextDirection(int textDirection) {
-        if (getTextDirection() != textDirection) {
+        if (getRawTextDirection() != textDirection) {
             // Reset the current text direction and the resolved one
             mPrivateFlags2 &= ~PFLAG2_TEXT_DIRECTION_MASK;
             resetResolvedTextDirection();
@@ -16539,10 +16541,10 @@ public void setTextDirection(int textDirection) {
     /**
      * Return the resolved text direction.
      *
-     * This needs resolution if the value is TEXT_DIRECTION_INHERIT. The resolution matches
-     * {@link #getTextDirection()}if it is not TEXT_DIRECTION_INHERIT, otherwise resolution proceeds
-     * up the parent chain of the view. if there is no parent, then it will return the default
-     * {@link #TEXT_DIRECTION_FIRST_STRONG}.
+     * This needs resolution if the value is TEXT_DIRECTION_INHERIT. The resolution matches what has
+     * been set by {@link #setTextDirection(int)} if it is not TEXT_DIRECTION_INHERIT, otherwise the
+     * resolution proceeds up the parent chain of the view. If there is no parent, then it will
+     * return the default {@link #TEXT_DIRECTION_FIRST_STRONG}.
      *
      * @return the resolved text direction. Returns one of:
      *
@@ -16552,7 +16554,7 @@ public void setTextDirection(int textDirection) {
      * {@link #TEXT_DIRECTION_RTL},
      * {@link #TEXT_DIRECTION_LOCALE}
      */
-    public int getResolvedTextDirection() {
+    public int getTextDirection() {
         // The text direction will be resolved only if needed
         if ((mPrivateFlags2 & PFLAG2_TEXT_DIRECTION_RESOLVED) != PFLAG2_TEXT_DIRECTION_RESOLVED) {
             resolveTextDirection();
@@ -16571,14 +16573,14 @@ public void resolveTextDirection() {
 
         if (hasRtlSupport()) {
             // Set resolved text direction flag depending on text direction flag
-            final int textDirection = getTextDirection();
+            final int textDirection = getRawTextDirection();
             switch(textDirection) {
                 case TEXT_DIRECTION_INHERIT:
                     if (canResolveTextDirection()) {
                         ViewGroup viewGroup = ((ViewGroup) mParent);
 
                         // Set current resolved direction to the same value as the parent's one
-                        final int parentResolvedDirection = viewGroup.getResolvedTextDirection();
+                        final int parentResolvedDirection = viewGroup.getTextDirection();
                         switch (parentResolvedDirection) {
                             case TEXT_DIRECTION_FIRST_STRONG:
                             case TEXT_DIRECTION_ANY_RTL:
@@ -16624,7 +16626,7 @@ public void resolveTextDirection() {
      * @return true if text direction resolution can be done otherwise return false.
      */
     private boolean canResolveTextDirection() {
-        switch (getTextDirection()) {
+        switch (getRawTextDirection()) {
             case TEXT_DIRECTION_INHERIT:
                 return (mParent != null) && (mParent instanceof ViewGroup);
             default:
@@ -16634,14 +16636,21 @@ private boolean canResolveTextDirection() {
 
     /**
      * Reset resolved text direction. Text direction can be resolved with a call to
-     * getResolvedTextDirection().
+     * getTextDirection().
      *
      * @hide
      */
     public void resetResolvedTextDirection() {
         mPrivateFlags2 &= ~(PFLAG2_TEXT_DIRECTION_RESOLVED | PFLAG2_TEXT_DIRECTION_RESOLVED_MASK);
     }
 
+    /**
+     * @hide
+     */
+    public boolean isTextDirectionInherited() {
+        return (getRawTextDirection() == TEXT_DIRECTION_INHERIT);
+    }
+
     /**
      * Return the value specifying the text alignment or policy that was set with
      * {@link #setTextAlignment(int)}.

core/java/android/view/ViewGroup.java

@@ -5271,7 +5271,7 @@ public void resetResolvedLayoutDirection() {
             if (child.isLayoutDirectionInherited()) {
                 child.resetResolvedLayoutDirection();
             }
-            if (child.getTextDirection() == TEXT_DIRECTION_INHERIT) {
+            if (child.isTextDirectionInherited()) {
                 child.resetResolvedTextDirection();
             }
             if (child.getTextAlignment() == TEXT_ALIGNMENT_INHERIT) {

core/java/android/widget/TextView.java

@@ -8192,8 +8192,7 @@ TextDirectionHeuristic getTextDirectionHeuristic() {
         final boolean defaultIsRtl = (getLayoutDirection() == LAYOUT_DIRECTION_RTL);
 
         // Now, we can select the heuristic
-        int textDir = getResolvedTextDirection();
-        switch (textDir) {
+        switch (getTextDirection()) {
             default:
             case TEXT_DIRECTION_FIRST_STRONG:
                 return (defaultIsRtl ? TextDirectionHeuristics.FIRSTSTRONG_RTL :