[Extensions] Make chrome://extensions use developerPrivate for inspect

chrome/browser/resources/extensions/extension_list.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="extension_error.js">
 
 /**
  * The type of the extension data object. The definition is based on
  * chrome/browser/ui/webui/extensions/extension_basic_info.cc
  * and
@@ skipping 713 matching lines @@
           var displayName = view.generatedBackgroundPage ?
               loadTimeData.getString('backgroundPage') : view.path;
           var label = displayName +
               (view.incognito ?
                   ' ' + loadTimeData.getString('viewIncognito') : '') +
               (view.renderProcessId == -1 ?
                   ' ' + loadTimeData.getString('viewInactive') : '');
           link.textContent = label;
           link.clickHandler = function(e) {
             // TODO(estade): remove conversion to string?
-            chrome.send('extensionSettingsInspect', [
-              String(extension.id),
-              String(view.renderProcessId),
-              String(view.renderViewId),
-              view.incognito
-            ]);
+            chrome.developerPrivate.inspect(

[not at google - send to devlin, 2015/03/09 18:26:38]

I very much prefer a style like

chrome.developerProvider.inspect({
  extension_id: ...
  ...
});

similar to functions, and how I usually see objects declared. Though, how
clang-format insists on NOT doing unfortunately.

[Devlin, 2015/03/09 19:13:57]

I tend to prefer it, too, but not sure how common it is.  Changed for now; dbeam
can decide  if it should change back.

+              {extension_id: String(extension.id),
+               render_process_id: String(view.renderProcessId),
+               render_view_id: String(view.renderViewId),
+               incognito: view.incognito});
           };
           link.addEventListener('click', link.clickHandler);
 
           if (i < extension.views.length - 1) {
             link = link.cloneNode(true);
             item.appendChild(link);
           }
         });
 
         var allLinks = item.querySelectorAll('a');
@@ skipping 157 matching lines @@
       // TODO(dbeam): why do we need to focus <extensionoptions> before and
       // after its showing animation? Makes very little sense to me.
       overlay.setInitialFocus();
     },
   };
 
   return {
     ExtensionList: ExtensionList
   };
 });