[Extensions UI] Use developerPrivate API for profile configuration

chrome/browser/ui/webui/extensions/extension_settings_browsertest.js

 // Copyright (c) 2012 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.
 
 // TODO(dbeam): test for loading upacked extensions?
 
 GEN('#include "chrome/browser/ui/webui/extensions/' +
     'extension_settings_browsertest.h"');
 
 // chrome/test/data/extensions/good.crx's extension ID. good.crx is loaded by
@@ skipping 101 matching lines @@
   /** @protected */
   verifyUninstallWorks: function() {
     var next = this.nextStep.bind(this);
     chrome.test.runWithUserGesture(function() {
       chrome.management.uninstall(GOOD_CRX_ID, function() {
         assertEquals(null, $(GOOD_CRX_ID));
         next();
       });
     });
   },
+
+  /** @protected */
+  verifyDeveloperModeWorks: function() {
+    var extensionSettings = getRequiredElement('extension-settings');
+    assertFalse(extensionSettings.classList.contains('dev-mode'));
+    getRequiredElement('toggle-dev-on').click();

[Dan Beam, 2015/04/10 02:48:06]

there's really no downside to using getRequiredElement() here, but this is akin
to assert($(id)).click() where $(id).click() would also fail if $(id) returns
null (so it's just doing unnecessary work).  typically we don't assert() things
that will just be used right after anyways...

[Devlin, 2015/04/10 16:30:38]

Fair point.  C++ habit carryover where a failed ASSERT is a lot nicer than a
segfault. :)

Fixed.

+    assertTrue(extensionSettings.classList.contains('dev-mode'));
+    chrome.developerPrivate.getProfileConfiguration(function(profileInfo) {
+      assertTrue(profileInfo.inDeveloperMode);
+      this.nextStep();
+    }.bind(this));
+  },
 };
 
 TEST_F('BasicExtensionSettingsWebUITest', 'testDisable', function() {
   this.steps = [this.waitForPageLoad,
                 this.verifyDisabledWorks,
                 testDone];
   this.nextStep();
 });
 
 TEST_F('BasicExtensionSettingsWebUITest', 'testEnable', function() {
   this.steps = [this.waitForPageLoad,
                 this.verifyDisabledWorks,
                 this.verifyEnabledWorks,
                 testDone];
   this.nextStep();
 });
 
 TEST_F('BasicExtensionSettingsWebUITest', 'testUninstall', function() {
   this.steps = [this.waitForPageLoad,
                 this.verifyUninstallWorks,
                 testDone];
   this.nextStep();
 });
 
+TEST_F('BasicExtensionSettingsWebUITest', 'testDeveloperMode', function() {
+  var next = this.nextStep.bind(this);
+  var checkDevModeIsOff = function() {
+    chrome.developerPrivate.getProfileConfiguration(function(profileInfo) {
+      assertFalse(profileInfo.inDeveloperMode);
+      next();
+    });
+  };
+  this.steps = [checkDevModeIsOff,
+                this.waitForPageLoad,
+                this.verifyDeveloperModeWorks,
+                testDone];
+  this.nextStep();
+});
+
 function AsyncExtensionSettingsWebUITest() {}
 
 AsyncExtensionSettingsWebUITest.prototype = {
   __proto__: ExtensionSettingsWebUITest.prototype,
 
   /** @override */
   isAsync: true,
 
   /** @override */
   testGenPreamble: function() {
@@ skipping 157 matching lines @@
 
 ExtensionOptionsDialogWebUITest.prototype = {
   __proto__: InstalledExtensionSettingsWebUITest.prototype,
 
   /** @override */
   browsePreload: ExtensionSettingsWebUITest.prototype.browsePreload +
       '?options=' + GOOD_CRX_ID,
 };
 
 TEST_F('ExtensionOptionsDialogWebUITest', 'testAccessibility', runAudit);