Make Sber1/2 pref metrics into Nullable Booleans so we can track how often these prefs are unset.

components/safe_browsing_db/safe_browsing_prefs.h

Index: components/safe_browsing_db/safe_browsing_prefs.h
diff --git a/components/safe_browsing_db/safe_browsing_prefs.h b/components/safe_browsing_db/safe_browsing_prefs.h
index da25461697ea0949598bce3a39b12f1fa675afdf..361f5ef0b545cd11b0fb573c89c35053123f2596 100644
--- a/components/safe_browsing_db/safe_browsing_prefs.h
+++ b/components/safe_browsing_db/safe_browsing_prefs.h
@@ -23,6 +23,14 @@ extern const char kSafeBrowsingScoutReportingEnabled[];
 // affects which of SafeBrowsingExtendedReporting or SafeBrowsingScoutReporting
 // is used.
 extern const char kSafeBrowsingScoutGroupSelected[];
+
+// Boolean indicating whether the user has ever seen a security interstitial
+// containing the legacy Extended Reporting opt-in.
+extern const char kSafeBrowsingSawInterstitialSber1[];

[Mike Lerman, 2017/03/08 20:08:36]

Can you make these variable names for descriptive? Sber1 vs Sber2 doesn't tell
me anything.

[lpz, 2017/03/09 17:58:01]

Done.

+
+// Boolean indicating whether the user has ever seen a security interstitial
+// containing the new Scout opt-in.
+extern const char kSafeBrowsingSawInterstitialSber2[];
 }
 
 namespace safe_browsing {