Add icon to the Data Saver main menu footer and update to spec

chrome/android/java/res/layout/data_reduction_main_menu_footer.xml

 <?xml version="1.0" encoding="utf-8"?>
 <!-- Copyright 2017 The Chromium Authors. All rights reserved.
      Use of this source code is governed by a BSD-style license that can be
      found in the LICENSE file. -->
 
 <org.chromium.chrome.browser.preferences.datareduction.DataReductionMainMenuFooter
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:chrome="http://schemas.android.com/apk/res-auto"
     android:id="@+id/data_reduction_footer"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:layout_gravity="start"
-    android:paddingTop="12dp"
-    android:paddingBottom="12dp"
     android:orientation="vertical"
-    style="@style/AppMenuItem" >
+    android:background="#EDFAFAFA" >
 
-    <TextView
-        android:id="@+id/menu_item_text"
-        android:layout_width="wrap_content"
+    <View style="@style/Divider" />
+
+    <LinearLayout
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:text="@string/data_reduction_title"
-        android:textAppearance="?android:attr/textAppearanceLargePopupMenu" />
+        android:layout_gravity="start"
+        android:paddingTop="12dp"
+        android:paddingBottom="12dp"
+        android:orientation="horizontal"
+        style="@style/AppMenuItem" >

[Ted C, 2017/04/07 18:19:10]

I think even if we want to keep AppMenuItem, we should still make
DataReductionMainMenuFooter and this LinearLayout into FrameLayouts.

Here you would set a marginTop to 1dp which is the divider height.  We could
pull out a size dimen for this and use it in styles.xml for Divider and allow it
to be reused here.

Then if we just change layout_width of the image view to be a specific value,
set the marginStart of the inner linearlayout to match, you should be able to
reduce from 3 nested LinearLayouts to having just one as the child.  The number
of layout passes required to do that should be fewer and it shouldn't make it
too hard to grok IMO.

[megjablon, 2017/04/07 20:03:26]

Done. PTAL to make sure I understood correctly.

 
-    <TextView
-        android:id="@+id/menu_item_summary"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:textSize="14sp"
-        android:textColor="@color/descriptive_text_color" />
+        <ImageView
+            android:id="@+id/chart_icon"
+            android:src="@+drawable/data_reduction_main_menu_icon"
+            android:layout_width="wrap_content"
+            android:layout_height="match_parent"
+            android:contentDescription="@null" />
+
+        <LinearLayout
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:orientation="vertical"
+            android:paddingStart="16dp" >
+
+            <TextView
+                android:id="@+id/menu_item_text"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:textSize="14sp"
+                android:textAppearance="@style/RobotoMediumStyle" />
+
+            <TextView
+                android:id="@+id/menu_item_summary"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:textSize="14sp"
+                android:textColor="@color/descriptive_text_color" />
+        </LinearLayout>
+
+    </LinearLayout>
 
 </org.chromium.chrome.browser.preferences.datareduction.DataReductionMainMenuFooter>