[Android History] Even more polish

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

 <?xml version="1.0" encoding="utf-8"?>
 <!-- Copyright 2016 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.history.HistoryItemView
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:chrome="http://schemas.android.com/apk/res-auto"
     android:layout_width="match_parent"
     android:layout_height="wrap_content" >
 
+    <!-- TODO(twellington): remove start/end padding when new assets with
+                            correct built in padding are available. -->
     <LinearLayout
         android:id="@+id/content"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:paddingTop="16dp"
         android:paddingBottom="16dp"
+        android:paddingStart="1dp"

[Theresa, 2017/02/15 21:41:04]

This should be moved to HistoryItemView above since this LinearLayout's end
padding is programatically set based on whether the remove icon is present.

[Theresa, 2017/02/15 23:16:15]

Done.

+        android:paddingEnd="1dp"
         android:orientation="horizontal" >
 
         <ImageView
             android:id="@+id/icon_view"
             style="@style/HistoryStartIcon"
             android:contentDescription="@null"
             android:src="@drawable/default_favicon" />
 
         <LinearLayout
           android:layout_width="0dp"
@@ skipping 23 matching lines @@
 
         <org.chromium.chrome.browser.widget.TintedImageButton
             android:id="@+id/remove"
             style="@style/HistoryEndIcon"
             android:contentDescription="@string/remove"
             android:src="@drawable/btn_trash"
             chrome:tint="@color/dark_mode_tint" />
     </LinearLayout>
 
 </org.chromium.chrome.browser.history.HistoryItemView>