Settings: Remove unnecessary wrapper in default browser tests.

Bug: 1026426
Change-Id: Ib0f10d3267899069c53794e4ae42db23b4f8b98d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2037026
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738476}

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

@@ -2,160 +2,158 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-cr.define('settings_default_browser', function() {
+/**
+ * A test version of DefaultBrowserBrowserProxy. Provides helper methods
+ * for allowing tests to know when a method was called, as well as
+ * specifying mock responses.
+ *
+ * @implements {settings.DefaultBrowserBrowserProxy}
+ */
+class TestDefaultBrowserBrowserProxy extends TestBrowserProxy {
+  constructor() {
+    super([
+      'requestDefaultBrowserState',
+      'setAsDefaultBrowser',
+    ]);
+
+    /** @private {!DefaultBrowserInfo} */
+    this.defaultBrowserInfo_ = {
+      canBeDefault: true,
+      isDefault: false,
+      isDisabledByPolicy: false,
+      isUnknownError: false
+    };
+  }
+
+  /** @override */
+  requestDefaultBrowserState() {
+    this.methodCalled('requestDefaultBrowserState');
+    return Promise.resolve(this.defaultBrowserInfo_);
+  }
+
+  /** @override */
+  setAsDefaultBrowser() {
+    this.methodCalled('setAsDefaultBrowser');
+  }
+
   /**
-   * A test version of DefaultBrowserBrowserProxy. Provides helper methods
-   * for allowing tests to know when a method was called, as well as
-   * specifying mock responses.
-   *
-   * @implements {settings.DefaultBrowserBrowserProxy}
+   * Sets the response to be returned by |requestDefaultBrowserState|.
+   * @param {!DefaultBrowserInfo} info Fake info for testing.
    */
-  class TestDefaultBrowserBrowserProxy extends TestBrowserProxy {
-    constructor() {
-      super([
-        'requestDefaultBrowserState',
-        'setAsDefaultBrowser',
-      ]);
-
-      /** @private {!DefaultBrowserInfo} */
-      this.defaultBrowserInfo_ = {
-        canBeDefault: true,
-        isDefault: false,
-        isDisabledByPolicy: false,
-        isUnknownError: false
-      };
-    }
-
-    /** @override */
-    requestDefaultBrowserState() {
-      this.methodCalled('requestDefaultBrowserState');
-      return Promise.resolve(this.defaultBrowserInfo_);
-    }
-
-    /** @override */
-    setAsDefaultBrowser() {
-      this.methodCalled('setAsDefaultBrowser');
-    }
-
-    /**
-     * Sets the response to be returned by |requestDefaultBrowserState|.
-     * @param {!DefaultBrowserInfo} info Fake info for testing.
-     */
-    setDefaultBrowserInfo(info) {
-      this.defaultBrowserInfo_ = info;
-    }
+  setDefaultBrowserInfo(info) {
+    this.defaultBrowserInfo_ = info;
+  }
+}
+
+suite('DefaultBrowserPageTest', function() {
+  let page = null;
+
+  /** @type {?settings.TestDefaultBrowserBrowserProxy} */
+  let browserProxy = null;
+
+  setup(function() {
+    browserProxy = new TestDefaultBrowserBrowserProxy();
+    settings.DefaultBrowserBrowserProxyImpl.instance_ = browserProxy;
+    return initPage();
+  });
+
+  teardown(function() {
+    page.remove();
+    page = null;
+  });
+
+  /** @return {!Promise} */
+  function initPage() {
+    browserProxy.reset();
+    PolymerTest.clearBody();
+    page = document.createElement('settings-default-browser-page');
+    document.body.appendChild(page);
+    return browserProxy.whenCalled('requestDefaultBrowserState');
   }
 
-  suite('DefaultBrowserPageTest', function() {
-    let page = null;
+  test('default-browser-test-can-be-default', function() {
+    browserProxy.setDefaultBrowserInfo({
+      canBeDefault: true,
+      isDefault: false,
+      isDisabledByPolicy: false,
+      isUnknownError: false
+    });
 
-    /** @type {?settings.TestDefaultBrowserBrowserProxy} */
-    let browserProxy = null;
+    return initPage().then(function() {
+      Polymer.dom.flush();
+      assertTrue(!!page.$$('#canBeDefaultBrowser'));
+      assertTrue(!page.$$('#isDefault'));
+      assertTrue(!page.$$('#isSecondaryInstall'));
+      assertTrue(!page.$$('#isUnknownError'));
+    });
+  });
 
-    setup(function() {
-      browserProxy = new TestDefaultBrowserBrowserProxy();
-      settings.DefaultBrowserBrowserProxyImpl.instance_ = browserProxy;
-      return initPage();
+  test('default-browser-test-is-default', function() {
+    assertTrue(!!page);
+    browserProxy.setDefaultBrowserInfo({
+      canBeDefault: true,
+      isDefault: true,
+      isDisabledByPolicy: false,
+      isUnknownError: false
     });
 
-    teardown(function() {
-      page.remove();
-      page = null;
+    return initPage().then(function() {
+      Polymer.dom.flush();
+      assertFalse(!!page.$$('#canBeDefaultBrowser'));
+      assertFalse(page.$$('#isDefault').hidden);
+      assertTrue(page.$$('#isSecondaryInstall').hidden);
+      assertTrue(page.$$('#isUnknownError').hidden);
     });
+  });
 
-    /** @return {!Promise} */
-    function initPage() {
-      browserProxy.reset();
-      PolymerTest.clearBody();
-      page = document.createElement('settings-default-browser-page');
-      document.body.appendChild(page);
-      return browserProxy.whenCalled('requestDefaultBrowserState');
-    }
-
-    test('default-browser-test-can-be-default', function() {
-      browserProxy.setDefaultBrowserInfo({
-        canBeDefault: true,
-        isDefault: false,
-        isDisabledByPolicy: false,
-        isUnknownError: false
-      });
-
-      return initPage().then(function() {
-        Polymer.dom.flush();
-        assertTrue(!!page.$$('#canBeDefaultBrowser'));
-        assertTrue(!page.$$('#isDefault'));
-        assertTrue(!page.$$('#isSecondaryInstall'));
-        assertTrue(!page.$$('#isUnknownError'));
-      });
+  test('default-browser-test-is-secondary-install', function() {
+    browserProxy.setDefaultBrowserInfo({
+      canBeDefault: false,
+      isDefault: false,
+      isDisabledByPolicy: false,
+      isUnknownError: false
     });
 
-    test('default-browser-test-is-default', function() {
-      assertTrue(!!page);
-      browserProxy.setDefaultBrowserInfo({
-        canBeDefault: true,
-        isDefault: true,
-        isDisabledByPolicy: false,
-        isUnknownError: false
-      });
-
-      return initPage().then(function() {
-        Polymer.dom.flush();
-        assertFalse(!!page.$$('#canBeDefaultBrowser'));
-        assertFalse(page.$$('#isDefault').hidden);
-        assertTrue(page.$$('#isSecondaryInstall').hidden);
-        assertTrue(page.$$('#isUnknownError').hidden);
-      });
+    return initPage().then(function() {
+      Polymer.dom.flush();
+      assertFalse(!!page.$$('#canBeDefaultBrowser'));
+      assertTrue(page.$$('#isDefault').hidden);
+      assertFalse(page.$$('#isSecondaryInstall').hidden);
+      assertTrue(page.$$('#isUnknownError').hidden);
     });
+  });
 
-    test('default-browser-test-is-secondary-install', function() {
-      browserProxy.setDefaultBrowserInfo({
-        canBeDefault: false,
-        isDefault: false,
-        isDisabledByPolicy: false,
-        isUnknownError: false
-      });
-
-      return initPage().then(function() {
-        Polymer.dom.flush();
-        assertFalse(!!page.$$('#canBeDefaultBrowser'));
-        assertTrue(page.$$('#isDefault').hidden);
-        assertFalse(page.$$('#isSecondaryInstall').hidden);
-        assertTrue(page.$$('#isUnknownError').hidden);
-      });
+  test('default-browser-test-is-disabled-by-policy', function() {
+    browserProxy.setDefaultBrowserInfo({
+      canBeDefault: true,
+      isDefault: false,
+      isDisabledByPolicy: true,
+      isUnknownError: false
     });
 
-    test('default-browser-test-is-disabled-by-policy', function() {
-      browserProxy.setDefaultBrowserInfo({
-        canBeDefault: true,
-        isDefault: false,
-        isDisabledByPolicy: true,
-        isUnknownError: false
-      });
-
-      return initPage().then(function() {
-        Polymer.dom.flush();
-        assertFalse(!!page.$$('#canBeDefaultBrowser'));
-        assertTrue(page.$$('#isDefault').hidden);
-        assertTrue(page.$$('#isSecondaryInstall').hidden);
-        assertFalse(page.$$('#isUnknownError').hidden);
-      });
+    return initPage().then(function() {
+      Polymer.dom.flush();
+      assertFalse(!!page.$$('#canBeDefaultBrowser'));
+      assertTrue(page.$$('#isDefault').hidden);
+      assertTrue(page.$$('#isSecondaryInstall').hidden);
+      assertFalse(page.$$('#isUnknownError').hidden);
+    });
+  });
+
+  test('default-browser-test-is-unknown-error', function() {
+    browserProxy.setDefaultBrowserInfo({
+      canBeDefault: true,
+      isDefault: false,
+      isDisabledByPolicy: false,
+      isUnknownError: true
     });
 
-    test('default-browser-test-is-unknown-error', function() {
-      browserProxy.setDefaultBrowserInfo({
-        canBeDefault: true,
-        isDefault: false,
-        isDisabledByPolicy: false,
-        isUnknownError: true
-      });
-
-      return initPage().then(function() {
-        Polymer.dom.flush();
-        assertFalse(!!page.$$('#canBeDefaultBrowser'));
-        assertTrue(page.$$('#isDefault').hidden);
-        assertTrue(page.$$('#isSecondaryInstall').hidden);
-        assertFalse(page.$$('#isUnknownError').hidden);
-      });
+    return initPage().then(function() {
+      Polymer.dom.flush();
+      assertFalse(!!page.$$('#canBeDefaultBrowser'));
+      assertTrue(page.$$('#isDefault').hidden);
+      assertTrue(page.$$('#isSecondaryInstall').hidden);
+      assertFalse(page.$$('#isUnknownError').hidden);
     });
   });
 });