Fetch snippets from ChromeReader and show them on the NTP

chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
index 22b29fe59eb486db769473727de4b2bda4e29b2e..46c4ccdb168ed76f59216e84895b3e09a5205359 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
@@ -89,6 +89,7 @@ import org.chromium.chrome.browser.metrics.StartupMetrics;
 import org.chromium.chrome.browser.metrics.UmaSessionStats;
 import org.chromium.chrome.browser.nfc.BeamController;
 import org.chromium.chrome.browser.nfc.BeamProvider;
+import org.chromium.chrome.browser.ntp.snippets.SnippetsController;
 import org.chromium.chrome.browser.offlinepages.OfflinePageUtils;
 import org.chromium.chrome.browser.omaha.UpdateMenuItemHelper;
 import org.chromium.chrome.browser.pageinfo.WebsiteSettingsPopup;
@@ -680,6 +681,9 @@ public abstract class ChromeActivity extends AsyncInitializationActivity
             mToolbarManager.onDeferredStartup(getOnCreateTimestampMs(), simpleName);
         }
         recordKeyboardLocaleUma();
+
+        // Initialize snippets
+        SnippetsController.get(this);

[Bernhard Bauer, 2016/02/08 18:19:26]

If we have an explicit call here anyway (as opposed to SnippetsController just
getting created the first time someone needs it), could we go from lazy creation
to an explicit create() method?

Also, could you make sure we don't start fetching stuff here by default?

[May, 2016/02/09 17:38:52]

So this was a temporary solution so we fetch new snippets on clank start while
we are implementing the piece that will fetch snippets on a schedule instead. In
the real implementation, we wouldn't be directly fetching snippets at this
point. I could remove it for now, and leave the SnippetsController the way I was
originally intending it to be.

[Bernhard Bauer, 2016/02/10 10:48:44]

OK... I just want to make sure we no-op out if the snippets experiment is not
enabled. Can we do that maybe in the SnippetsFetcher?

[May, 2016/02/10 18:16:46]

I wrapped this around a command line flag for now.

     }
 
     @Override