Extensions: Don't pass the selected unpacked extension path from JS to C++

chrome/browser/resources/extensions/extensions.js

Index: chrome/browser/resources/extensions/extensions.js
diff --git a/chrome/browser/resources/extensions/extensions.js b/chrome/browser/resources/extensions/extensions.js
index ac00fc1813f9e8158adb42ee04eb3dd5c748c205..ccea98e52472d85abf5941d0ee836594c8d9aff4 100644
--- a/chrome/browser/resources/extensions/extensions.js
+++ b/chrome/browser/resources/extensions/extensions.js
@@ -67,32 +67,15 @@ cr.define('extensions', function() {
     },
 
     /**
-     * Utility function which asks the C++ to show a platform-specific file
-     * select dialog, and fire |callback| with the |filePath| that resulted.
-     * |selectType| can be either 'file' or 'folder'. |operation| can be 'load',
-     * 'packRoot', or 'pem' which are signals to the C++ to do some
-     * operation-specific configuration.
-     * @private
-     */
-    showFileDialog_: function(selectType, operation, callback) {
-      handleFilePathSelected = function(filePath) {
-        callback(filePath);
-        handleFilePathSelected = function() {};
-      };
-
-      chrome.send('extensionSettingsSelectFilePath', [selectType, operation]);
-    },
-
-    /**
      * Handles the Load Unpacked Extension button.
      * @param {Event} e Change event.
      * @private
      */
     handleLoadUnpackedExtension_: function(e) {
-      this.showFileDialog_('folder', 'pem', function(filePath) {
-        chrome.send('extensionSettingsLoad', [String(filePath)]);
-      });
+      chrome.send('extensionSettingsLoadUnpackedExtension');
 
+      // TODO(jhawkins): Refactor metrics support out of options and use it

[Evan Stade, 2012/03/14 23:13:45]

MetricsHandler

[James Hawkins, 2012/03/15 00:24:13]

I meant pull it out of pref_ui.js so we can add metric="blah" to buttons
declaratively.

+      // in extensions.html.
       chrome.send('coreOptionsUserMetricsAction',
                   ['Options_LoadUnpackedExtension']);
     },