[ImportantSites] Implementing dialog level blacklisting and blacklist expiration

chrome/common/pref_names.cc

Index: chrome/common/pref_names.cc
diff --git a/chrome/common/pref_names.cc b/chrome/common/pref_names.cc
index 7b14c9a6ea83e6f322a32cfadd4009a4f8552076..9fbc4cb7e1e9a479fae044bf274ff8b6b9d413e8 100644
--- a/chrome/common/pref_names.cc
+++ b/chrome/common/pref_names.cc
@@ -2388,4 +2388,8 @@ const char kGoogleDSEGeolocationSetting[] = "google_dse_geolocation_setting";
 // value.
 const char kWebShareVisitedTargets[] = "profile.web_share.visited_targets";
 
+// A dictionary which stores when and how many times the user has ignored the
+// important sites dialog.
+const char kImportantSitesDialogHistory[] = "important_sites.dialog_history";

[gab, 2017/02/06 20:34:01]

"important_sites." is not an existing prefs namespace (i.e. no other
important_sites.* prefs), can we find an existing one that makes sense?
Otherwise we create a JSON dict just for it...

[dmurph, 2017/02/06 23:34:22]

It doesn't look like there's anything else - putting it in a global.

+
 }  // namespace prefs