[ImportantSites] Implementing dialog level blacklisting and blacklist expiration

chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java

Index: chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
index def7a49fe00e39400f76608edbfa65d894b4af46..5d11891c62fec324b38b9ab412b0c3fcf065fa7d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/PrefServiceBridge.java
@@ -92,10 +92,13 @@ public final class PrefServiceBridge {
          *                       favicons.
          * @param importantReasons Bitfield of reasons why this domain was selected. Pass this back
          *                         to clearBrowinsgData so we can record metrics.
+         * @param dialogDisabled If the important dialog itself is blacklisted and shouldn't be

[dominickn, 2017/02/03 19:30:53]

You could clarify this comment ("blacklisting" a dialog seems weird). E.g. 

"True if the important dialog has been ignored too many times and should not be
shown"

[dmurph, 2017/02/03 20:03:29]

Done.

[dmurph, 2017/02/03 20:03:29]

Done.

+         *                       shown.
          */
         @CalledByNative("ImportantSitesCallback")
         void onImportantRegisterableDomainsReady(
-                String[] domains, String[] exampleOrigins, int[] importantReasons);
+                String[] domains, String[] exampleOrigins, int[] importantReasons,
+                boolean dialogBlacklisted);

[dominickn, 2017/02/03 19:30:53]

Nit: dialogDisabled, not dialogBlacklisted. The former is less strange.

[dmurph, 2017/02/03 20:03:29]

Done.

     }
 
     /**