Fix the issue that horizontal margin is ignored on devices with API (g…

…oogle#354)

level < 17.
Because the FlexboxHelper expected
ViewGroup.MarginLayoutParams#getMarginStart (getMarginEnd) methods when
calculating the wrap condition.

No devices are available on API level < 17  on Firebase Test Lab. So
manually ran the tests on an emulator with API level 16.

flexbox/src/androidTest/res/layout/activity_wrap_content_child_bottom_margin_column_grow.xml

@@ -29,6 +29,7 @@
         android:layout_width="24dp"
         android:layout_height="0dp"
         android:layout_marginEnd="12dp"
+        android:layout_marginRight="12dp"
         android:text="1"
         app:layout_flexBasisPercent="50%"
         app:layout_flexGrow="1"

flexbox/src/androidTest/res/layout/activity_wrap_content_child_bottom_margin_column_shrink.xml

@@ -29,6 +29,7 @@
         android:layout_width="24dp"
         android:layout_height="150dp"
         android:layout_marginEnd="12dp"
+        android:layout_marginRight="12dp"
         android:text="1"
         app:layout_flexShrink="1" />
 

flexbox/src/main/java/com/google/android/flexbox/FlexboxHelper.java

@@ -738,36 +738,49 @@ private int getFlexItemSizeCross(FlexItem flexItem, boolean isMainHorizontal) {
 
     /**
      * Returns the flexItem's start margin in the main axis. Either start or top.
+     * For the backward compatibility for API level < 17, the horizontal margin is returned using
+     * {@link FlexItem#getMarginLeft} (ViewGroup.MarginLayoutParams#getMarginStart isn't available
+     * in API level < 17). Thus this method needs to be used with {@link #getFlexItemMarginEndMain}
+     * not to misuse the margin in RTL.
+     *
      *
      * @param flexItem         the flexItem
      * @param isMainHorizontal is the main axis horizontal
      * @return the flexItem's start margin in the main axis
      */
     private int getFlexItemMarginStartMain(FlexItem flexItem, boolean isMainHorizontal) {
         if (isMainHorizontal) {
-            return flexItem.getMarginStart();
+            return flexItem.getMarginLeft();
         }
 
         return flexItem.getMarginTop();
     }
 
     /**
      * Returns the flexItem's end margin in the main axis. Either end or bottom.
+     * For the backward compatibility for API level < 17, the horizontal margin is returned using
+     * {@link FlexItem#getMarginRight} (ViewGroup.MarginLayoutParams#getMarginEnd isn't available
+     * in API level < 17). Thus this method needs to be used with
+     * {@link #getFlexItemMarginStartMain} not to misuse the margin in RTL.
      *
      * @param flexItem         the flexItem
      * @param isMainHorizontal is the main axis horizontal
      * @return the flexItem's end margin in the main axis
      */
     private int getFlexItemMarginEndMain(FlexItem flexItem, boolean isMainHorizontal) {
         if (isMainHorizontal) {
-            return flexItem.getMarginEnd();
+            return flexItem.getMarginRight();
         }
 
         return flexItem.getMarginBottom();
     }
 
     /**
      * Returns the flexItem's start margin in the cross axis. Either start or top.
+     * For the backward compatibility for API level < 17, the horizontal margin is returned using
+     * {@link FlexItem#getMarginLeft} (ViewGroup.MarginLayoutParams#getMarginStart isn't available
+     * in API level < 17). Thus this method needs to be used with
+     * {@link #getFlexItemMarginEndCross} to not to misuse the margin in RTL.
      *
      * @param flexItem         the flexItem
      * @param isMainHorizontal is the main axis horizontal
@@ -778,11 +791,15 @@ private int getFlexItemMarginStartCross(FlexItem flexItem, boolean isMainHorizon
             return flexItem.getMarginTop();
         }
 
-        return flexItem.getMarginStart();
+        return flexItem.getMarginLeft();
     }
 
     /**
      * Returns the flexItem's end margin in the cross axis. Either end or bottom.
+     * For the backward compatibility for API level < 17, the horizontal margin is returned using
+     * {@link FlexItem#getMarginRight} (ViewGroup.MarginLayoutParams#getMarginEnd isn't available
+     * in API level < 17). Thus this method needs to be used with
+     * {@link #getFlexItemMarginStartCross} to not to misuse the margin in RTL.
      *
      * @param flexItem         the flexItem
      * @param isMainHorizontal is the main axis horizontal
@@ -793,7 +810,7 @@ private int getFlexItemMarginEndCross(FlexItem flexItem, boolean isMainHorizonta
             return flexItem.getMarginBottom();
         }
 
-        return flexItem.getMarginEnd();
+        return flexItem.getMarginRight();
     }
 
     /**