Added a UI for the Interests Prototype.

chrome/android/java/src/org/chromium/chrome/browser/ntp/InterestsPage.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/InterestsPage.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/InterestsPage.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/InterestsPage.java
new file mode 100644
index 0000000000000000000000000000000000000000..edea4d841e66c2f9d5b371e93bc91f0852dbed53
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/InterestsPage.java
@@ -0,0 +1,126 @@
+// 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.
+
+package org.chromium.chrome.browser.ntp;
+
+import android.app.Activity;
+import android.content.Context;
+import android.view.LayoutInflater;
+import android.view.View;
+
+import org.chromium.base.ApiCompatibilityUtils;
+import org.chromium.chrome.R;
+import org.chromium.chrome.browser.NativePage;
+import org.chromium.chrome.browser.UrlConstants;
+import org.chromium.chrome.browser.ntp.InterestsService.GetInterestsCallback;
+import org.chromium.chrome.browser.ntp.InterestsService.Interest;
+import org.chromium.chrome.browser.profiles.Profile;
+import org.chromium.chrome.browser.tab.Tab;
+import org.chromium.ui.widget.Toast;
+
+import java.util.Arrays;
+import java.util.List;
+
+/**
+ * List the interests of the user. When an interest is clicked the user is redirected to a search

[newt (away), 2015/12/12 00:30:31]

How about: "Displays a list of topics that we think the user is interested in."

[PEConn, 2015/12/14 17:05:15]

Done.

+ * for that subject.
+ */
+public class InterestsPage implements NativePage {

[newt (away), 2015/12/12 00:30:31]

With the current setup, this doesn't need to be a NativePage since we never
actually navigate a tab to chrome-native://interests. Likewise, you don't need
to add UrlConstants.INTERESTS_URL.

On the other hand, bookmarks and recent tabs are both dialogs on phones, but are
loaded inside a tab on tablets. That's why they implement NativePage. We should
consider doing the same for the interests page. (That could certainly be a
separate CL.)

[PEConn, 2015/12/14 17:05:15]

Acknowledged.

+
+    private static final String TAG = "interests_page";
+
+    private final InterestsView mPageView;
+    private final String mTitle;
+    private final int mBackgroundColor;
+    private final int mThemeColor;
+
+    /**
+     * Creates a InterestsPage to be shown in document mode.

[newt (away), 2015/12/12 00:30:31]

only in document mode?? If so, why?

[PEConn, 2015/12/14 17:05:15]

Done.

+     *
+     * @param context The view context for showing the page.
+     * @param tab The tab from which interests page is loaded.
+     * @param profile The profile from which to load interests.
+     * @param activity The activity from which the interests page is loaded.

[newt (away), 2015/12/12 00:30:31]

You don't need the "activity" param. The context is sufficient for getting
resources.

[PEConn, 2015/12/14 17:05:15]

Done.

+     */
+    public InterestsPage(final Context context, Tab tab, Profile profile, Activity activity) {
+        mTitle = context.getResources().getString(R.string.ntp_interests);
+        mBackgroundColor = ApiCompatibilityUtils.getColor(activity.getResources(), R.color.ntp_bg);
+        mThemeColor = ApiCompatibilityUtils.getColor(
+                activity.getResources(), R.color.default_primary_color);
+
+        LayoutInflater inflater = LayoutInflater.from(context);
+
+        mPageView = (InterestsView) inflater.inflate(R.layout.interests_page, null);

[newt (away), 2015/12/12 00:30:31]

You can use the View.inflate() convenience method:

    mPageView = (InterestsView) View.inflate(context, R.layout.interests_page,
null);

[PEConn, 2015/12/14 17:05:15]

Done.

+
+        new InterestsService(profile).getInterests(
+                new GetInterestsCallback() {
+                    @Override
+                    public void onInterestsAvailableCallback(Interest[] interests) {

[newt (away), 2015/12/12 00:30:31]

style nit: remove "Callback" from this method name

[PEConn, 2015/12/14 17:05:15]

Done.

+                        if (interests == null) {
+                            Toast toast = Toast.makeText(context,
+                                    R.string.ntp_no_interests_toast, Toast.LENGTH_SHORT);
+                            toast.show();
+                            return;
+                        }
+                        List<Interest> interestList = Arrays.asList(interests);
+
+                        mPageView.setInterests(interestList);
+                    }
+                });
+    }
+
+    public void setListener(InterestsClickListener listener) {
+        mPageView.setListener(listener);
+    }
+
+    /**
+     * Interface to be notified when the user clicks on a interest.
+     **/
+    public interface InterestsClickListener {
+        /**
+         * Called when a interest is selected.

[newt (away), 2015/12/12 00:30:31]

s/a interest/an interest/  (and elsewhere too)

[PEConn, 2015/12/14 17:05:15]

Done.

+         *
+         * @param name The name of the selected interest.
+         */
+        void onInterestClicked(String name);
+    }
+
+    @Override
+    public String getTitle() {
+        return mTitle;
+    }
+
+    @Override
+    public int getBackgroundColor() {
+        return mBackgroundColor;
+    }
+
+    @Override
+    public int getThemeColor() {
+        return mThemeColor;
+    }
+    @Override
+    public View getView() {
+        return mPageView;
+    }
+
+
+    @Override
+    public void destroy() {
+    }
+
+    @Override
+    public String getHost() {
+        return "interests";

[newt (away), 2015/12/12 00:30:31]

If you're going to use a NativePage, this constant should be moved into
UrlConstants.

[PEConn, 2015/12/14 17:05:15]

Done.

+    }
+
+    @Override
+    public String getUrl() {
+        return UrlConstants.INTERESTS_URL;
+    }
+
+    @Override
+    public void updateForUrl(String url) {
+    }
+}