Notification custom layouts: spacing of the timestamp.

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 |
@@ skipping 39 matching lines @@
                 android:layout_height="wrap_content"
                 android:layout_alignParentStart="true"
                 android:layout_toStartOf="@id/time"
                 android:layout_alignParentTop="true"
                 android:layout_marginTop="4dp"
                 android:singleLine="true"
                 android:ellipsize="end"/>
 
             <TextView
                 android:id="@+id/time"
-                style="@style/WebNotificationTime"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
+                android:layout_alignBaseline="@id/title"
                 android:layout_alignParentEnd="true"
                 android:layout_alignParentTop="true"
-                android:singleLine="true"/>
+                android:layout_height="wrap_content"

[newt (away), 2015/10/29 20:38:19]

Since we're talking about it, the attribute order that we use more or less
throughout Chrome is:

  id
  layout_width
  layout_height
  other layout_* attributes (alphabetized, or just "logically" arranged)
  other attributes (usually alphabetized)

Rationale:
width and height come first after id, since they're mandatory unlike other
attributes. layout_* attributes are grouped together because they are
instructions for the parent ViewGroup rather than for the view itself.

[Michael van Ouwerkerk, 2015/10/30 09:58:39]

Thanks! That seems reasonable, I'll converge on this pattern as I make changes
then. Done for the parts touched by this CL.

+                android:layout_width="wrap_content"
+                android:layout_marginEnd="8dp"
+                android:layout_marginStart="8dp"
+                android:layout_marginTop="8dp"
+                android:singleLine="true"
+                style="@style/WebNotificationTime"/>
 
         </RelativeLayout>
 
         <TextView
             android:id="@+id/body"
             style="@style/WebNotificationBodyBig"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginEnd="6dp"
             android:layout_marginBottom="8dp"
@@ skipping 30 matching lines @@
             style="@style/WebNotificationOrigin"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_marginBottom="10dp"
             android:singleLine="true"
             android:ellipsize="end"/>
 
     </LinearLayout>
 
 </RelativeLayout>