Add progressbar to animated doodle

chrome/android/java/res/layout/new_tab_page.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. -->
 
 <org.chromium.chrome.browser.ntp.NewTabPageView
     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="match_parent"
@@ skipping 25 matching lines @@
                  layout_weight="0". The NewTabPageLayout will redistribute the extra space between
                  them. -->
             <View
                 android:id="@+id/ntp_top_spacer"
                 android:layout_width="0dp"
                 android:layout_height="0dp"
                 android:layout_weight="1"
                 android:visibility="invisible" />
 
             <!-- Search provider logo -->
-            <org.chromium.chrome.browser.ntp.LogoView
-                android:id="@+id/search_provider_logo"
+            <FrameLayout
                 android:layout_width="wrap_content"
-                android:layout_height="@dimen/ntp_logo_height"
-                android:layout_marginStart="16dp"
-                android:layout_marginEnd="16dp"
-                android:layout_marginTop="26dp"
-                android:layout_marginBottom="30dp"
-                android:src="@drawable/google_logo" />
+                android:layout_height="wrap_content" >
+
+                <org.chromium.chrome.browser.ntp.LogoView

[newt (away), 2015/10/13 05:24:59]

You could just make the LogoView extend FrameLayout, and avoid this extra
nesting

[Ian Wen, 2015/10/26 22:45:37]

Done.

+                    android:id="@+id/search_provider_logo"
+                    android:layout_width="wrap_content"
+                    android:layout_height="@dimen/ntp_logo_height"
+                    android:layout_marginBottom="30dp"
+                    android:layout_marginEnd="16dp"
+                    android:layout_marginStart="16dp"
+                    android:layout_marginTop="26dp"
+                    android:src="@drawable/google_logo" />
+
+                <org.chromium.chrome.browser.widget.LoadingView

[newt (away), 2015/10/13 05:24:59]

don't you need to add a ProgressBar inside the LoadingView?

[Ian Wen, 2015/10/26 22:45:37]

Done.

+                    android:id="@+id/loading_view"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_gravity="center" />
+
+            </FrameLayout>
 
             <!-- Search box -->
             <LinearLayout
                 android:id="@+id/search_box"
                 android:layout_width="match_parent"
                 android:layout_height="48dp"
                 android:layout_marginStart="12dp"
                 android:layout_marginEnd="12dp"
                 android:layout_marginBottom="8dp"
                 android:gravity="center_vertical"
@@ skipping 131 matching lines @@
                 android:layout_height="match_parent"
                 android:layout_gravity="center_horizontal"
                 android:drawablePadding="8dp"
                 android:gravity="center_vertical"
                 android:text="@string/recent_tabs"
                 android:textSize="12sp"
                 android:textColor="#5a5a5a" />
         </FrameLayout>
     </org.chromium.chrome.browser.ntp.NewTabPageToolbar>
 </org.chromium.chrome.browser.ntp.NewTabPageView>