Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Side by Side Diff: chrome/browser/android/ntp/ntp_snippets_bridge.cc

Issue 1980033002: [NTP Snippets] Clear the snippets when the user signs out (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix double we Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/ntp/ntp_snippets_bridge.h" 5 #include "chrome/browser/android/ntp/ntp_snippets_bridge.h"
6 6
7 #include <jni.h> 7 #include <jni.h>
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_array.h" 10 #include "base/android/jni_array.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 ToJavaArrayOfStrings(env, snippets).obj(), 161 ToJavaArrayOfStrings(env, snippets).obj(),
162 ToJavaLongArray(env, timestamps).obj(), 162 ToJavaLongArray(env, timestamps).obj(),
163 ToJavaArrayOfStrings(env, publishers).obj()); 163 ToJavaArrayOfStrings(env, publishers).obj());
164 } 164 }
165 165
166 void NTPSnippetsBridge::NTPSnippetsServiceShutdown() { 166 void NTPSnippetsBridge::NTPSnippetsServiceShutdown() {
167 observer_.Reset(); 167 observer_.Reset();
168 snippet_service_observer_.Remove(ntp_snippets_service_); 168 snippet_service_observer_.Remove(ntp_snippets_service_);
169 } 169 }
170 170
171 void NTPSnippetsBridge::NTPSnippetsServiceCleared() {
172 // The user signed out or disabled sync. Since snippets rely on those, we
173 // clear them to be consistent with the initially signed out state.
174 JNIEnv* env = base::android::AttachCurrentThread();
175 Java_SnippetsBridge_onSnippetsCleared(env, observer_.obj());
176 }
177
171 void NTPSnippetsBridge::OnImageFetched(ScopedJavaGlobalRef<jobject> callback, 178 void NTPSnippetsBridge::OnImageFetched(ScopedJavaGlobalRef<jobject> callback,
172 const std::string& snippet_id, 179 const std::string& snippet_id,
173 const SkBitmap* bitmap) { 180 const SkBitmap* bitmap) {
174 JNIEnv* env = AttachCurrentThread(); 181 JNIEnv* env = AttachCurrentThread();
175 182
176 ScopedJavaLocalRef<jobject> j_bitmap; 183 ScopedJavaLocalRef<jobject> j_bitmap;
177 if (bitmap && !bitmap->isNull()) 184 if (bitmap && !bitmap->isNull())
178 j_bitmap = gfx::ConvertToJavaBitmap(bitmap); 185 j_bitmap = gfx::ConvertToJavaBitmap(bitmap);
179 186
180 Java_FetchSnippetImageCallback_onSnippetImageAvailable(env, callback.obj(), 187 Java_FetchSnippetImageCallback_onSnippetImageAvailable(env, callback.obj(),
181 j_bitmap.obj()); 188 j_bitmap.obj());
182 } 189 }
183 190
184 // static 191 // static
185 bool NTPSnippetsBridge::Register(JNIEnv* env) { 192 bool NTPSnippetsBridge::Register(JNIEnv* env) {
186 return RegisterNativesImpl(env); 193 return RegisterNativesImpl(env);
187 } 194 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698