Adds functionality to pack an extension / app from the app.

chrome/browser/resources/apps_debugger/js/items.js

Index: chrome/browser/resources/apps_debugger/js/items.js
diff --git a/chrome/browser/resources/apps_debugger/js/items.js b/chrome/browser/resources/apps_debugger/js/items.js
index 28565609590af3e6b4d2e2d6dc93971bc757c2af..4a63134cf28f04c4ab597b27e5e4ca72e65cc1d7 100644
--- a/chrome/browser/resources/apps_debugger/js/items.js
+++ b/chrome/browser/resources/apps_debugger/js/items.js
@@ -117,14 +117,18 @@ AppsDevTool.prototype = {
     // Set up the three buttons (load unpacked, pack and update).
     $('load-unpacked').addEventListener('click',
         this.handleLoadUnpackedItem_.bind(this));
+    $('pack-item').addEventListener('click',
+        this.handlePackItem_.bind(this));
     $('update-items-now').addEventListener('click',
         this.handleUpdateItemNow_.bind(this));
 
+    var packItemOverlay = appsDevtool.PackItemOverlay.getInstance();
+      packItemOverlay.initializePage();

[Dan Beam, 2013/02/11 23:44:24]

nit: indent off

[Gaurav, 2013/02/12 19:33:17]

Done.

+

[Dan Beam, 2013/02/11 23:44:24]

nit: remove \n

[Gaurav, 2013/02/12 19:33:17]

Done.

   },
 
-  /**
-   * Handles the Load Unpacked Extension button.
-   * @param {!Event} e Change event.
+  /** Handles the Load Unpacked Extension button.
+   * @param {Event} e Change event.
    * @private
    */
   handleLoadUnpackedItem_: function(e) {
@@ -133,6 +137,14 @@ AppsDevTool.prototype = {
     });
   },
 
+  /** Handles the Pack Extension button.
+   * @param {Event} e Change event.
+   * @private
+   */
+  handlePackItem_: function(e) {
+    AppsDevTool.showOverlay($('packItemOverlay'));
+  },
+
   /**
    * Handles the Update Extension Now Button.
    * @param {Event} e Change event.
@@ -148,7 +160,7 @@ AppsDevTool.prototype = {
  * @return {Element} The overlay element.
  */
 AppsDevTool.getCurrentOverlay = function() {
-  return this.ownerDocument.querySelector('#overlay .page.showing');
+  return document.querySelector('#overlay .page.showing');

[Dan Beam, 2013/02/11 23:44:24]

why did you change this?

[Gaurav, 2013/02/12 19:33:17]

this.ownerDocument returns null.

On 2013/02/11 23:44:24, Dan Beam wrote:

[Dan Beam, 2013/02/14 21:10:40]

ah, 'cuz it's static, OK

 };
 
 /**