Notification custom layouts: move settings button to the footer.

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

 <?xml version="1.0" encoding="utf-8"?>
 <!-- Copyright 2015 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. -->
 
 <!--
   Web notification custom layout, big state.
   _______________________________________________________
   |        |                                            |
   |  Icon  |  Title text                        1:23 pm |
   |        |                                            |
   |        |  Body text which may run to multiple lines |
   |________|  if there is a lot of it.                  |
   |           __________________________________________|
   |                                                     |
-  |           Button       Button       Settings        |
+  |           Button       Button                       |
   |           __________________________________________|
   |                                                     |
-  |           www.example.com                           |
+  |           ⛭ www.example.com                         |
   |_____________________________________________________|
 -->
 <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content">
 
     <include layout="@layout/web_notification_icon_frame"
         android:id="@+id/icon_frame"/>
 
     <!--
@@ skipping 62 matching lines @@
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:orientation="horizontal"        
             android:visibility="gone"
             style="@style/WebNotificationButtons"/>
 
         <ImageView
             android:id="@+id/footer_divider"
             android:layout_width="match_parent"
             android:layout_height="1dp"
-            android:layout_marginBottom="8dp"
             android:contentDescription="@null"
             style="@style/WebNotificationDivider"/>
 
         <RelativeLayout
             android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:layout_marginBottom="10dp">
+            android:layout_height="wrap_content">
 
-            <TextView
+            <ImageView
+                android:id="@+id/origin_settings_icon"
+                android:layout_width="wrap_content"
+                android:layout_height="40dp"
+                android:layout_alignParentStart="true"
+                android:layout_alignParentTop="true"
+                android:contentDescription="@null"
+                android:src="@drawable/settings_cog"
+                android:scaleType="centerInside"/>
+
+            <Button
                 android:id="@+id/origin"
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content"
+                android:layout_height="40dp"

[newt (away), 2016/01/08 23:44:16]

it's safer to use minHeight and wrap_content for TextViews; that'll ensure the
text doesn't get clipped if the user's system font size is set to Huge.

[Michael van Ouwerkerk, 2016/01/11 16:33:17]

Done.

                 android:layout_marginEnd="8dp"
                 android:layout_alignParentStart="true"
                 android:layout_alignParentTop="true"

[newt (away), 2016/01/08 23:44:16]

remove alignParentTop since this is centered using gravity

[Michael van Ouwerkerk, 2016/01/11 16:33:17]

I'm not sure I understood this one. This button is in a RelativeLayout, does
that use gravity? I thought it didn't. My understanding is that the gravity
attribute on this button determines where the text inside the button goes, in
case the button is bigger than the text which is usually the case here. The
layout alignment attributes determine where the button goes inside the
RelativeLayout, so these attributes control different things.

[newt (away), 2016/01/11 21:08:28]

You're correct. I must have misread the xml when I left that comment.

                 android:layout_toStartOf="@+id/small_icon_footer"
+                android:gravity="start|center_vertical"
+                android:paddingStart="32dp"

[newt (away), 2016/01/08 23:44:16]

Why paddingStart instead of using layout_toEndOf to make this come after the
settings icon?

[Michael van Ouwerkerk, 2016/01/11 16:33:17]

This way, the button has some blank space that overlaps the icon. This makes it
look as if the icon is touchable and part of the button. This is for the same
reason (to apply a color filter) as we discussed for
web_notification_button.xml.

                 android:singleLine="true"
                 android:ellipsize="start"
-                style="@style/WebNotificationOrigin"/>
+                style="@style/WebNotificationSettingsButton"/>
 
             <ViewStub
                 android:id="@id/small_icon_footer"
                 android:inflatedId="@id/small_icon_footer"
                 android:layout="@layout/web_notification_small_icon"
                 android:layout_width="16dp"
                 android:layout_height="16dp"
                 android:layout_marginEnd="8dp"
                 android:layout_alignParentEnd="true"
-                android:layout_alignParentTop="true"/>
+                android:layout_centerVertical="true"/>
 
         </RelativeLayout>
 
     </LinearLayout>
 
 </RelativeLayout>