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

Unified Diff: components/supervised_user_error_page/supervised_user_error_page.h

Issue 2695113002: [Webview, Child Accounts] Automatically sign in to Chrome if needed. (Closed)
Patch Set: Created 3 years, 10 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: components/supervised_user_error_page/supervised_user_error_page.h
diff --git a/components/supervised_user_error_page/supervised_user_error_page.h b/components/supervised_user_error_page/supervised_user_error_page.h
index 01ca3e219511c1469e649b0a4549c05266d411dd..f8ee24917bb369b6d52558e7e49f89095d12f74a 100644
--- a/components/supervised_user_error_page/supervised_user_error_page.h
+++ b/components/supervised_user_error_page/supervised_user_error_page.h
@@ -9,12 +9,16 @@
namespace supervised_user_error_page {
+// Note: values need to be explicit, since NOT_SIGNED_IN is also
+// detected in Java (SupervisedUserContentProvider.java) and the value
+// needs to match.
Bernhard Bauer 2017/02/15 15:09:38 We can actually create a Java enum (well, @IntDef)
aberent 2017/02/15 18:34:46 Done.
enum FilteringBehaviorReason {
- DEFAULT,
- ASYNC_CHECKER,
- BLACKLIST,
- MANUAL,
- WHITELIST
+ DEFAULT = 0,
+ ASYNC_CHECKER = 1,
+ BLACKLIST = 2,
+ MANUAL = 3,
+ WHITELIST = 4,
+ NOT_SIGNED_IN = 5,
};
int GetBlockMessageID(

Powered by Google App Engine
This is Rietveld 408576698