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 96 matching lines @@
           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;
+      $('apps-developer-tools-promo').querySelector('.close-button')

[Dan Beam, 2014/04/18 21:27:28]

nit: . at end

[Devlin, 2014/04/18 22:19:55]

Whoops, sorry - git cl format is making bad habits for me ;)

+          .addEventListener('click', function(e) {
+        this.promoDismissed_ = true;
+        $('apps-developer-tools-promo').hidden = 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;
     }
 
+    if (extensionsData.promoteAppsDevTools && !this.promoDismissed_) {
+      $('apps-developer-tools-promo').hidden = false;
+      pageDiv.classList.add('adt-promo');

[Dan Beam, 2014/04/18 21:27:28]

why are we hiding the promo at all?  can't we just leave it unhidden but
obscured from view until .adt-promo is added/removed?

right now it seems like this would immediately hide the promo and transition its
parent's height after that... might look a bit jarring.

[Devlin, 2014/04/18 22:19:55]

Oh, I guess we can.  Nice.

+    } else {
+      $('apps-developer-tools-promo').hidden = true;
+      pageDiv.classList.remove('adt-promo');
+    }
+
     $('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();
 });