MD Settings: hide browsing history-related rows for supervised users

chrome/test/data/webui/settings/privacy_page_test.js

 // Copyright 2016 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.
 
 cr.define('settings_privacy_page', function() {
   /**
    * @constructor
    * @extends {TestBrowserProxy}
    * @implements {settings.ClearBrowsingDataBrowserProxy}
    */
   function TestClearBrowsingDataBrowserProxy() {
     settings.TestBrowserProxy.call(this, [
       'initialize',
       'clearBrowsingData',
+      'shouldShowBrowsingHistoryRows',
     ]);
 
     /**
      * The promise to return from |clearBrowsingData|.
      * Allows testing code to test what happens after the call is made, and
      * before the browser responds.
      * @private {?Promise}
      */
     this.clearBrowsingDataPromise_ = null;
+
+    /** @private */
+    this.shouldShowBrowsingHistoryRows_ = true;
   }
 
   TestClearBrowsingDataBrowserProxy.prototype = {
     __proto__: settings.TestBrowserProxy.prototype,
 
     /** @param {!Promise} promise */
     setClearBrowsingDataPromise: function(promise) {
       this.clearBrowsingDataPromise_ = promise;
     },
 
+    /** @param {boolean} shouldShow */
+    setShouldShowBrowsingHistoryRows: function(shouldShow) {
+      this.shouldShowBrowsingHistoryRows_ = shouldShow;
+    },
+
     /** @override */
     clearBrowsingData: function() {
       this.methodCalled('clearBrowsingData');
       cr.webUIListenerCallback('browsing-data-removing', true);
       return this.clearBrowsingDataPromise_ !== null ?
           this.clearBrowsingDataPromise_ : Promise.resolve();
     },
 
     /** @override */
     initialize: function() {
       this.methodCalled('initialize');
       return Promise.resolve(false);
     },
+
+    /** @override */
+    shouldShowBrowsingHistoryRows: function() {
+      return this.shouldShowBrowsingHistoryRows_;
+    },
   };
 
   function registerNativeCertificateManagerTests() {
     suite('NativeCertificateManager', function() {
       /** @type {settings.TestPrivacyPageBrowserProxy} */
       var testBrowserProxy;
 
       /** @type {SettingsPrivacyPageElement} */
       var page;
 
@@ skipping 165 matching lines @@
         });
         var checkbox = element.$$('settings-checkbox');
         assertEquals('browser.clear_data.browsing_history', checkbox.pref.key);
 
         // Simulate a browsing data counter result for history. This checkbox's
         // sublabel should be updated.
         cr.webUIListenerCallback(
             'update-counter-text', checkbox.pref.key, 'result');
         assertEquals('result', checkbox.subLabel);
       });
+
+      test('history rows are hidden for supervised users', function() {
+        assertTrue(!!element.$$('#browsingCheckbox'));
+        assertTrue(!!element.$$('#downloadCheckbox'));
+
+        element.remove();
+        testBrowserProxy.reset();
+        testBrowserProxy.setShouldShowBrowsingHistoryRows(false);
+
+        element = document.createElement('settings-clear-browsing-data-dialog');

[dpapad, 2017/05/04 22:43:02]

If we are going to recreate the element anyway, just for this test() case, how
about the following instead?

1) Revert changes to CBD proxy.
2) In clear_browsing_data_dialog.js
   this.showBrowsingHistoryRows_ = loadTimeData.getBoolean('isSupervised');
3) Use loadTimeData.overrideValues() here, similar to what we do in other
occanions, see
https://cs.chromium.org/search/?q=overrideValues+file:%5Esrc/chrome/test/data...

This would be require less code (no proxy changes), and more aligned with how we
inject values to loadTimeData in other tests.

[Dan Beam, 2017/05/04 23:21:28]

Done. (thanks for reminding me about overrideValues! makes everything much
simpler!)

+        document.body.appendChild(element);
+        Polymer.dom.flush();
+
+        return testBrowserProxy.whenCalled('initialize').then(function() {
+          assertFalse(!!element.$$('#browsingCheckbox'));
+          assertFalse(!!element.$$('#downloadCheckbox'));
+        });
+      });
     });
   }
 
   function registerSafeBrowsingExtendedReportingTests() {
     suite('SafeBrowsingExtendedReporting', function() {
       /** @type {settings.TestPrivacyPageBrowserProxy} */
       var testBrowserProxy;
 
       /** @type {SettingsPrivacyPageElement} */
       var page;
@@ skipping 35 matching lines @@
     registerTests: function() {
       if (cr.isMac || cr.isWin)
         registerNativeCertificateManagerTests();
 
       registerClearBrowsingDataTests();
       registerPrivacyPageTests();
       registerSafeBrowsingExtendedReportingTests();
     },
   };
 });