[MD settings] clear incognito only checkbox

chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.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.
 
 /**
  * Only used for tests.
  * @typedef {{
  *   auto_downloads: !Array<!RawSiteException>},
  *   background_sync: !Array<!RawSiteException>},
  *   camera: !Array<!RawSiteException>},
@@ skipping 59 matching lines @@
  * A test version of SiteSettingsPrefsBrowserProxy. Provides helper methods
  * for allowing tests to know when a method was called, as well as
  * specifying mock responses.
  *
  * @constructor
  * @implements {settings.SiteSettingsPrefsBrowserProxy}
  * @extends {settings.TestBrowserProxy}
  */
 var TestSiteSettingsPrefsBrowserProxy = function() {
   settings.TestBrowserProxy.call(this, [
-    'fetchUsbDevices',
-    'fetchZoomLevels',
-    'getCookieDetails',
-    'getDefaultValueForContentType',
-    'getExceptionList',
-    'isPatternValid',
-    'observeProtocolHandlers',
-    'observeProtocolHandlersEnabledState',
-    'reloadCookies',
-    'removeCookie',
-    'removeProtocolHandler',
-    'removeUsbDevice',
-    'removeZoomLevel',
-    'resetCategoryPermissionForOrigin',
-    'setCategoryPermissionForOrigin',
-    'setDefaultValueForContentType',
-    'setProtocolDefault'

[dpapad, 2017/05/02 00:36:35]

Nit (optional): Can you try adding |updateIncognitoStatus| at the end of this
list, but include a trailing comma, and re-run clang-format? I suspect that this
will cause clang-format to keep the 1-per-line formatting (which I think is more
readable).

[dschuyler, 2017/05/02 01:05:22]

Yep! that worked thanks.

Done.

+    'fetchUsbDevices', 'fetchZoomLevels', 'getCookieDetails',
+    'getDefaultValueForContentType', 'getExceptionList', 'isPatternValid',
+    'observeProtocolHandlers', 'observeProtocolHandlersEnabledState',
+    'reloadCookies', 'removeCookie', 'removeProtocolHandler', 'removeUsbDevice',
+    'removeZoomLevel', 'resetCategoryPermissionForOrigin',
+    'setCategoryPermissionForOrigin', 'setDefaultValueForContentType',
+    'setProtocolDefault', 'updateIncognitoStatus'
   ]);
 
   /** @private {boolean} */
   this.hasIncognito_ = false;
 
   /** @private {!SiteSettingsPref} */
   this.prefs_ = prefsEmpty;
 
   /** @private {!Array<ZoomLevelEntry>} */
   this.zoomList_ = [];
@@ skipping 257 matching lines @@
   },
 
   /** @override */
   setProtocolDefault: function() {
     this.methodCalled('setProtocolDefault', arguments);
   },
 
   /** @override */
   removeProtocolHandler: function() {
     this.methodCalled('removeProtocolHandler', arguments);
+  },
+
+  /** @override */
+  updateIncognitoStatus: function() {
+    this.methodCalled('updateIncognitoStatus', arguments);
   }
 };