Implement important sites dialog for desktop.

chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview 'settings-clear-browsing-data-dialog' allows the user to delete
  * browsing data that has been cached by Chromium.
  */
 Polymer({
   is: 'settings-clear-browsing-data-dialog',
 
   behaviors: [WebUIListenerBehavior],
 
   properties: {
     /**
      * Preferences state.
      */
     prefs: {
       type: Object,
       notify: true,
     },
 
     /**
+     * State of this dialog, either "clearBrowsingData" or "importantSites".
+     * @private
+     */
+    dialogState_: {
+      type: String,
+      value: "clearBrowsingData",
+    },
+
+    /**
      * Results of browsing data counters, keyed by the suffix of
      * the corresponding data type deletion preference, as reported
      * by the C++ side.
      * @private {!Object<string>}
      */
     counters_: {
       type: Object,
       value: {
         // Will be filled as results are reported.
       }
@@ skipping 19 matching lines @@
     clearingInProgress_: {
       type: Boolean,
       value: false,
     },
 
     /** @private */
     showHistoryDeletionDialog_: {
       type: Boolean,
       value: false,
     },
+
+    /** @private {!Array<ImportantSite>} */
+    importantSites_: {
+      type: Array,
+      value: [],
+    },
   },
 
   /** @private {settings.ClearBrowsingDataBrowserProxy} */
   browserProxy_: null,
 
   /** @override */
   ready: function() {
     this.$.clearFrom.menuOptions = this.clearFromOptions_;
     this.addWebUIListener(
         'update-footer',
         this.updateFooter_.bind(this));
     this.addWebUIListener(
         'update-counter-text',
         this.updateCounterText_.bind(this));
   },
 
   /** @override */
   attached: function() {
     this.browserProxy_ =
         settings.ClearBrowsingDataBrowserProxyImpl.getInstance();
     this.browserProxy_.initialize().then(function() {
       this.$.dialog.showModal();
+      this.browserProxy_.fetchImportantSites().then(
+          this.receiveImportantSites.bind(this));
     }.bind(this));
   },
 
   /**
+   * @param {!ImportantSitesResponse} result
+   * @private
+   */
+  receiveImportantSites: function(result) {
+    if (result.dialogDisabled) {
+      this.set('importantSites_', []);
+    } else {
+      this.set('importantSites_', result.importantSites);
+    }
+  },
+
+  /**
    * Updates the footer to show only those sentences that are relevant to this
    * user.
    * @param {boolean} syncing Whether the user is syncing data.
    * @param {boolean} otherFormsOfBrowsingHistory Whether the user has other
    *     forms of browsing history in their account.
    * @private
    */
   updateFooter_: function(syncing, otherFormsOfBrowsingHistory) {
     this.$.googleFooter.hidden = !otherFormsOfBrowsingHistory;
     this.$.syncedDataSentence.hidden = !syncing;
     this.$.dialog.classList.add('fully-rendered');
   },
 
   /**
    * Updates the text of a browsing data counter corresponding to the given
    * preference.
    * @param {string} prefName Browsing data type deletion preference.
    * @param {string} text The text with which to update the counter
    * @private
    */
   updateCounterText_: function(prefName, text) {
     // Data type deletion preferences are named "browser.clear_data.<datatype>".
     // Strip the common prefix, i.e. use only "<datatype>".
     var matches = prefName.match(/^browser\.clear_data\.(\w+)$/);
     this.set('counters_.' + assert(matches[1]), text);
   },
 
   /**
+   * @param {string} currentDialogState The current dialog state.
+   * @return {boolean} Whether this is the clear browsing data dialog.
+   * @private
+   */
+  isClearBrowsingDataDialog_: function(currentDialogState) {
+    return currentDialogState == "clearBrowsingData";
+  },
+
+  /**
+   * @param {string} currentDialogState The current dialog state.
+   * @return {boolean} Whether this is the important sites dialog.
+   * @private
+   */
+  isImportantSitesDialog_: function(currentDialogState) {
+    return currentDialogState == "importantSites";
+  },
+
+  /** @private */
+  shouldShowImportantSites_: function() {
+    if (!this.$.cookiesCheckbox.checked && !this.$.cacheCheckbox.checked) {
+      return false;
+    }
+    return this.isClearBrowsingDataDialog_(this.dialogState_) &&

[dmurph, 2017/03/06 21:16:48]

Please also reference
https://cs.chromium.org/chromium/src/chrome/browser/engagement/important_site...

And check (maybe add test) to make sure that if you ignore the dialog (don't
uncheck things) 3 times
(https://cs.chromium.org/chromium/src/chrome/browser/engagement/important_site...,
which you can also add a method to set a testing value if you want), then it'll
not show the dialog. You can optimize this as well to not bother fetching the
important sites, but that's not necessarily needed.

[dullweber, 2017/03/07 12:24:34]

I'm already passing the result of this method to javascript and I handled it in
receiveImportantSites() but I will store and access it explicitly to make it
more clear

+        this.importantSites_.length;
+    // TODO(dullweber): Log Histogram?
+  },
+
+  /**
    * Handles the tap on the Clear Data button.
    * @private
    */
   onClearBrowsingDataTap_: function() {
+    if (this.shouldShowImportantSites_()) {
+      this.dialogState_ = "importantSites";
+      return
+    }
+
     this.clearingInProgress_ = true;
 
-    this.browserProxy_.clearBrowsingData().then(
+    this.browserProxy_.clearBrowsingData(this.importantSites_).then(
       /**
        * @param {boolean} shouldShowNotice Whether we should show the notice
        *     about other forms of browsing history before closing the dialog.
        */
       function(shouldShowNotice) {
         this.clearingInProgress_ = false;
         this.showHistoryDeletionDialog_ = shouldShowNotice;
 
         if (!shouldShowNotice)
           this.$.dialog.close();
       }.bind(this));
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.dialog.cancel();
   },
 
   /**
    * Handles the closing of the notice about other forms of browsing history.
    * @private
    */
   onHistoryDeletionDialogClose_: function() {
     this.showHistoryDeletionDialog_ = false;
     this.$.dialog.close();
   }
 });