Fix bug #7651552 resetRtlProperties() is called way too often

- remove unnecessary calls to resetRtlProperties().
- now reset of RTL properties will only be done when adding a View
(and no more when removing it)

Change-Id: I0d42128c9f7df6085fb92bb5af5c9bd4d1ba88a3

Author: Fabrice Di Meglio

core/java/android/view/View.java

@@ -11854,8 +11854,6 @@ protected void onDetachedFromWindow() {
 
         mCurrentAnimation = null;
 
-        resetRtlProperties();
-        onRtlPropertiesChanged(LAYOUT_DIRECTION_DEFAULT);
         resetAccessibilityStateChanged();
     }
 

core/java/android/view/ViewGroup.java

@@ -3620,8 +3620,6 @@ private void removeViewInternal(int index, View view) {
             childHasTransientStateChanged(view, false);
         }
 
-        view.resetRtlProperties();
-
         onViewRemoved(view);
 
         needGlobalAttributesUpdate(false);
@@ -5368,21 +5366,6 @@ public void resolveLayoutParams() {
         }
     }
 
-    /**
-     * @hide
-     */
-    @Override
-    public void resetRtlProperties() {
-        super.resetRtlProperties();
-        int count = getChildCount();
-        for (int i = 0; i < count; i++) {
-            final View child = getChildAt(i);
-            if (child.isLayoutDirectionInherited()) {
-                child.resetRtlProperties();
-            }
-        }
-    }
-
     /**
      * @hide
      */