Fetch snippets from ChromeReader and show them on the NTP

chrome/browser/android/ntp_snippets_bridge.cc

Index: chrome/browser/android/ntp_snippets_bridge.cc
diff --git a/chrome/browser/android/ntp_snippets_bridge.cc b/chrome/browser/android/ntp_snippets_bridge.cc
new file mode 100644
index 0000000000000000000000000000000000000000..58da875cfeefddce6e637eb54e4201ce7cc5eeb6
--- /dev/null
+++ b/chrome/browser/android/ntp_snippets_bridge.cc
@@ -0,0 +1,86 @@
+// Copyright 2016 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.
+
+#include "chrome/browser/android/ntp_snippets_bridge.h"
+
+#include <jni.h>
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_array.h"
+#include "base/android/scoped_java_ref.h"
+#include "chrome/browser/ntp_snippets/ntp_snippets_service_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_android.h"
+#include "components/ntp_snippets/ntp_snippet.h"
+#include "components/ntp_snippets/ntp_snippets_service.h"
+#include "jni/SnippetsBridge_jni.h"
+
+using base::android::JavaParamRef;
+using base::android::ToJavaArrayOfStrings;
+using ntp_snippets::NTPSnippetsService;
+using ntp_snippets::NTPSnippetsServiceObserver;
+
+static jlong Init(JNIEnv* env,
+                  const JavaParamRef<jobject>& obj,
+                  const JavaParamRef<jobject>& j_profile,
+                  const JavaParamRef<jobject>& j_observer) {
+  NTPSnippetsBridge* snippets_bridge =
+      new NTPSnippetsBridge(env, j_profile, j_observer);
+  return reinterpret_cast<intptr_t>(snippets_bridge);
+}
+
+NTPSnippetsBridge::NTPSnippetsBridge(JNIEnv* env,
+                                     jobject j_profile,
+                                     jobject j_observer)
+    : snippet_service_observer_(this) {
+  observer_.Reset(env, j_observer);
+
+  Profile* profile = ProfileAndroid::FromProfileAndroid(j_profile);
+  if (profile) {

[Bernhard Bauer, 2016/02/10 18:36:20]

Actually, could we remove this? I would rather crash if this is null than
silently swallowing it.

[May, 2016/02/11 15:25:24]

Done.

+    ntp_snippets_service_ = NTPSnippetsServiceFactory::GetForProfile(profile);
+    snippet_service_observer_.Add(ntp_snippets_service_);
+    if (ntp_snippets_service_->is_loaded())
+      NTPSnippetsServiceLoaded(ntp_snippets_service_);
+  }
+}
+
+NTPSnippetsBridge::~NTPSnippetsBridge() {}
+
+void NTPSnippetsBridge::Destroy(JNIEnv* env, const JavaParamRef<jobject>& obj) {
+  delete this;
+}
+
+void NTPSnippetsBridge::NTPSnippetsServiceLoaded(NTPSnippetsService* service) {
+  if (observer_.is_null())
+    return;
+
+  std::vector<std::string> titles;
+  std::vector<std::string> urls;
+  std::vector<std::string> thumbnail_urls;
+  std::vector<std::string> snippets;
+  for (const ntp_snippets::NTPSnippet& snippet : *service) {
+    titles.push_back(snippet.title());
+    urls.push_back(snippet.url().spec());
+    thumbnail_urls.push_back(snippet.salient_image_url().spec());
+    snippets.push_back(snippet.snippet());
+  }
+
+  JNIEnv* env = base::android::AttachCurrentThread();
+  Java_SnippetsObserver_onSnippetsAvailable(
+      env, observer_.obj(), ToJavaArrayOfStrings(env, titles).obj(),
+      ToJavaArrayOfStrings(env, urls).obj(),
+      ToJavaArrayOfStrings(env, thumbnail_urls).obj(),
+      ToJavaArrayOfStrings(env, snippets).obj());
+}
+
+void NTPSnippetsBridge::NTPSnippetsServiceShutdown(
+    NTPSnippetsService* service) {
+  observer_.Reset();
+  snippet_service_observer_.Remove(ntp_snippets_service_);
+}
+
+// static
+bool NTPSnippetsBridge::Register(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}