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

Unified Diff: chrome/browser/android/ntp/ntp_snippets_bridge.h

Issue 2061803002: 📰 The Status card reports disabled sync states (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@simplifyBridge
Patch Set: [NTP Snippets] The Status card reports disabled sync states Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/ntp/ntp_snippets_bridge.h
diff --git a/chrome/browser/android/ntp/ntp_snippets_bridge.h b/chrome/browser/android/ntp/ntp_snippets_bridge.h
index b2643b292a218bb90cfc903f5f91ed5681d3b25d..326f54278d4316502a0aff28ec810f7b1b753a89 100644
--- a/chrome/browser/android/ntp/ntp_snippets_bridge.h
+++ b/chrome/browser/android/ntp/ntp_snippets_bridge.h
@@ -46,6 +46,16 @@ class NTPSnippetsBridge : public ntp_snippets::NTPSnippetsServiceObserver {
const base::android::JavaParamRef<jobject>& callback,
const base::android::JavaParamRef<jstring>& jurl);
+ // Returns a reason why the snippet service is disabled, or 0 if it isn't.
+ // Possible return values:
+ // 0 - NONE
+ // 1 - EXPLICITLY_DISABLED,
+ // 2 - HISTORY_SYNC_DISABLED,
+ // 3 - HISTORY_SYNC_STATE_UNKNOWN
Bernhard Bauer 2016/06/14 12:10:40 I think it's fine to just refer to the definition
dgn 2016/06/15 16:46:29 Done.
+ // See NTPSnippetsService::DisabledReason for more info.
+ int GetDisabledReason(JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj);
+
static bool Register(JNIEnv* env);
private:

Powered by Google App Engine
This is Rietveld 408576698