Promote Apps Developer Tools in the chrome:extensions page

chrome/browser/resources/extensions/extensions.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.
 
 <include src="../uber/uber_utils.js"></include>
 <include src="extension_commands_overlay.js"></include>
 <include src="extension_focus_manager.js"></include>
 <include src="extension_list.js"></include>
 <include src="pack_extension_overlay.js"></include>
 <include src="extension_error_overlay.js"></include>
@@ skipping 66 matching lines @@
   /**
    * ExtensionSettings class
    * @class
    */
   function ExtensionSettings() {}
 
   cr.addSingletonGetter(ExtensionSettings);
 
   ExtensionSettings.prototype = {
     __proto__: HTMLDivElement.prototype,
 

[Dan Beam, 2014/04/18 23:08:33]

nit:

/**
 * Whether the App Developer Tools promo has been dismissed on this page.
 * @type {boolean}
 * @private
 */
promoDismissed_: false,

[Devlin, 2014/04/18 23:41:02]

Done.

     /**
      * Perform initial setup.
      */
     initialize: function() {
       uber.onContentFrameLoaded();
       cr.ui.FocusOutlineManager.forDocument(document);
       measureCheckboxStrings();
 
       // Set the title.
       var title = loadTimeData.getString('extensionSettings');
       uber.invokeMethodOnParent('setTitle', {title: title});
 
       // This will request the data to show on the page and will get a response
       // back in returnExtensionsData.
       chrome.send('extensionSettingsRequestExtensionsData');
 
       $('toggle-dev-on').addEventListener('change',
           this.handleToggleDevMode_.bind(this));
       $('dev-controls').addEventListener('webkitTransitionEnd',
           this.handleDevControlsTransitionEnd_.bind(this));
 
       // Set up the three dev mode buttons (load unpacked, pack and update).
       $('load-unpacked').addEventListener('click',
           this.handleLoadUnpackedExtension_.bind(this));
       $('pack-extension').addEventListener('click',
           this.handlePackExtension_.bind(this));
       $('update-extensions-now').addEventListener('click',
           this.handleUpdateExtensionNow_.bind(this));
 
+      // Set up the close dialog for the apps developer tools promo.
+      this.promoDismissed_ = false;

[Dan Beam, 2014/04/18 23:08:33]

nit: move to prototype (see above)

[Devlin, 2014/04/18 23:41:02]

Done.

+      $('apps-developer-tools-promo').querySelector('.close-button').
+          addEventListener('click', function(e) {
+        this.promoDismissed_ = true;
+        $('extension-settings').classList.remove('adt-promo');
+      }.bind(this));
+
       if (!loadTimeData.getBoolean('offStoreInstallEnabled')) {
         this.dragWrapper_ = new cr.ui.DragWrapper(document.documentElement,
                                                   dragWrapperHandler);
       }
 
       extensions.PackExtensionOverlay.getInstance().initializePage();
 
       // Hook up the configure commands link to the overlay.
       var link = document.querySelector('.extension-commands-config');
       link.addEventListener('click',
@@ skipping 163 matching lines @@
 
     if (extensionsData.developerMode) {
       pageDiv.classList.add('dev-mode');
       $('toggle-dev-on').checked = true;
       $('dev-controls').hidden = false;
     } else {
       pageDiv.classList.remove('dev-mode');
       $('toggle-dev-on').checked = false;
     }
 
+    var showPromo = extensionsData.promoteAppsDevTools && !this.promoDismissed_;
+    pageDiv.classList.toggle('adt-promo', showPromo);
+
     $('load-unpacked').disabled = extensionsData.loadUnpackedDisabled;
 
     ExtensionsList.prototype.data_ = extensionsData;
     var extensionList = $('extension-settings-list');
     ExtensionsList.decorate(extensionList);
   }
 
   // Indicate that warning |message| has occured for pack of |crx_path| and
   // |pem_path| files.  Ask if user wants override the warning.  Send
   // |overrideFlags| to repeated 'pack' call to accomplish the override.
@@ skipping 89 matching lines @@
 
   // Export
   return {
     ExtensionSettings: ExtensionSettings
   };
 });
 
 window.addEventListener('load', function(e) {
   extensions.ExtensionSettings.getInstance().initialize();
 });