chrome://plugins Mojo-ification part 2/2, populating the page.

chrome/browser/resources/plugins.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.
 
 /**
  * Takes the |pluginsData| input argument which represents data about the
  * currently installed/running plugins and populates the html jstemplate with
- * that data. It expects an object structure like the above.
+ * that data.
  * @param {Object} pluginsData Detailed info about installed plugins. Same
  *     expected format as returnPluginsData().
  */
 function renderTemplate(pluginsData) {
   // This is the javascript code that processes the template:
   var input = new JsEvalContext(pluginsData);
   var output = $('pluginTemplate');
   jstProcess(input, output);
 }
 
 /**
- * Asks the C++ PluginsDOMHandler to get details about the installed plugins and
- * return detailed data about the configuration. The PluginsDOMHandler should
- * reply to returnPluginsData() (below).
+ * @param {boolean} showDetails
  */
-function requestPluginsData() {
-  chrome.send('requestPluginsData');
-  chrome.send('getShowDetails');
-}
+function loadShowDetailsFromPrefs(showDetails) {
+  tmiModeExpanded = showDetails;
+  $('collapse').style.display =
+      showDetails ? 'inline' : 'none';
+  $('expand').style.display =
+      showDetails ? 'none' : 'inline';
 
-function loadShowDetailsFromPrefs(show_details) {
-  tmiModeExpanded = show_details;
-  $('collapse').style.display =
-      show_details ? 'inline' : 'none';
-  $('expand').style.display =
-      show_details ? 'none' : 'inline';
-
-  document.body.className = show_details ? 'show-in-tmi-mode' : 'hide-tmi-mode';
+  document.body.className = showDetails ? 'show-in-tmi-mode' : 'hide-tmi-mode';
 }
 
 /**
  * Called by the web_ui_ to re-populate the page with data representing the
  * current state of installed plugins.
  * @param {Object} pluginsData Detailed info about installed plugins. The
  *     template expects each plugin's format to match the following structure to
  *     correctly populate the page:
  *   {
  *     plugins: [
  *       {
  *         name: 'Group Name',
  *         description: 'description',
  *         version: 'version',
  *         update_url: 'http://update/',
  *         critical: true,
- *         enabled: true,
- *         identifier: 'plugin-name',
+ *         enabled_mode: 'enabledByUser',
+ *         id: 'plugin-name',
+ *         always_allowed: false,
  *         plugin_files: [
  *           {
- *             path: '/blahblah/blahblah/MyCrappyPlugin.plugin',
- *             name: 'MyCrappyPlugin',
- *             version: '1.2.3',
- *             description: 'My crappy plugin',
- *             mimeTypes: [
- *               { description: 'Foo Media',
- *                 fileExtensions: ['foo'],
- *                 mimeType: 'application/x-my-foo' },
- *               { description: 'Bar Stuff',
- *                 fileExtensions: ['bar', 'baz'],
- *                 mimeType: 'application/my-bar' }
+ *             path: '/foo/bar/baz/MyPlugin.plugin',
+ *             name: 'MyPlugin',
+ *             version: '1.2,3'
+ *             description: 'My plugin',
+ *             type: 'BROWSER PLUGIN',
+ *             mime_types: [
+ *               {
+ *                 description: 'Foo Media',
+ *                 file_extensions: [ 'pdf'],

[Dan Beam, 2016/01/26 20:10:43]

[\s -> [

[dpapad, 2016/01/26 22:34:48]

Changed.

+ *                 mime_type: 'application/x-my-foo'
+ *               },
+ *               {
+ *                 description: 'Bar Stuff',
+ *                 file_extensions: [ 'bar', 'baz'],

[Dan Beam, 2016/01/26 20:10:43]

[\s -> [

[dpapad, 2016/01/26 22:34:48]

Changed.

+ *                 mime_type: 'application/my-bar'
+ *               }
  *             ],
- *             enabledMode: 'enabledByUser'
+ *             enabled_mode: 'enabledByUser',
  *           },
  *           {
  *             path: '/tmp/MyFirst.plugin',
  *             name: 'MyFirstPlugin',
  *             version: '3.14r15926',
  *             description: 'My first plugin',
- *             mimeTypes: [
- *               { description: 'New Guy Media',
- *                 fileExtensions: ['mfp'],
- *                 mimeType: 'application/x-my-first' }
+ *             type: 'BROWSER PLUGIN',
+ *             mime_types: [
+ *               {
+ *                 description: 'New Guy Media',
+ *                 file_extensions: [ 'mfp'],

[Dan Beam, 2016/01/26 20:10:43]

[\s -> [

[dpapad, 2016/01/26 22:34:48]

Changed.

+ *                 mime_type: 'application/x-my-first'
+ *               },
  *             ],
- *             enabledMode: 'enabledByPolicy'
+ *             enabled_mode: 'disabledByUser',
  *           },
- *           {
- *             path: '/foobar/baz/YourGreatPlugin.plugin',
- *             name: 'YourGreatPlugin',
- *             version: '4.5',
- *             description: 'Your great plugin',
- *             mimeTypes: [
- *               { description: 'Baz Stuff',
- *                 fileExtensions: ['baz'],
- *                 mimeType: 'application/x-your-baz' }
- *             ],
- *             enabledMode: 'disabledByUser'
- *           },
- *           {
- *             path: '/foobiz/bar/HisGreatPlugin.plugin',
- *             name: 'HisGreatPlugin',
- *             version: '1.2',
- *             description: 'His great plugin',
- *             mimeTypes: [
- *               { description: 'More baz Stuff',
- *                 fileExtensions: ['bor'],
- *                 mimeType: 'application/x-his-bor' }
- *             ],
- *             enabledMode: 'disabledByPolicy'
- *           }
- *         ]
- *       }
- *     ]
+ *         ],
+ *       },
+ *     ],
  *   }
  */
 function returnPluginsData(pluginsData) {
   var bodyContainer = $('body-container');
   var body = document.body;
 
   // Set all page content to be visible so we can measure heights.
   bodyContainer.style.visibility = 'hidden';
   body.className = '';
   var slidables = document.getElementsByClassName('show-in-tmi-mode');
@@ skipping 73 matching lines @@
 }
 
 /**
  * Handles a 'enable' or 'disable' button getting clicked.
  * @param {HTMLElement} node The HTML element for the plugin being changed.
  * @param {boolean} enable Whether to enable or disable the plugin.
  * @param {boolean} isGroup True if we're enabling/disabling a plugin group,
  *     rather than a single plugin.
  */
 function handleEnablePlugin(node, enable, isGroup) {
-  // Tell the C++ PluginsDOMHandler to enable/disable the plugin.
-  chrome.send('enablePlugin', [String(node.path), String(enable),
-              String(isGroup)]);
+  whenBrowserProxyReady.then(function(browserProxy) {
+    isGroup ?
+        browserProxy.setPluginGroupEnabled(node.path, enable) :
+        browserProxy.setPluginEnabled(node.path, enable);
+  });
 }
 
 // Keeps track of whether details have been made visible (expanded) or not.
 var tmiModeExpanded = false;
 
 /*
  * Toggles visibility of details.
  */
 function toggleTmiMode() {
   tmiModeExpanded = !tmiModeExpanded;
 
   $('collapse').style.display =
       tmiModeExpanded ? 'inline' : 'none';
   $('expand').style.display =
       tmiModeExpanded ? 'none' : 'inline';
 
   document.body.className =
       tmiModeExpanded ? 'show-tmi-mode' : 'hide-tmi-mode';
 
-  chrome.send('saveShowDetailsToPrefs', [String(tmiModeExpanded)]);
+  whenBrowserProxyReady.then(function(browserProxy) {
+    browserProxy.saveShowDetailsToPrefs(tmiModeExpanded);
+  });
 }
 
 function handleSetPluginAlwaysAllowed(el) {
-  chrome.send('setPluginAlwaysAllowed', [el.identifier, el.checked]);
+  whenBrowserProxyReady.then(function(browserProxy) {
+    browserProxy.setPluginAlwaysAllowed(el.identifier, el.checked);
+  });
 }
 
 /**
  * @param {Object} plugin An object containing the information about a plugin.
  *     See returnPluginsData() for the format of this object.
  * @return {boolean} Whether the plugin's version should be displayed.
  */
 function shouldDisplayPluginVersion(plugin) {
   return !!plugin.version && plugin.version != '0';
 }
@@ skipping 12 matching lines @@
          plugin.description != 'Version ' + plugin.version &&
          plugin.description != plugin.name + ' ' + plugin.version;
 }
 
 /**
  * @param {Object} plugin An object containing the information about a plugin.
  *     See returnPluginsData() for the format of this object.
  * @return {boolean} Whether the plugin is enabled.
  */
 function isPluginEnabled(plugin) {
-  return plugin.enabledMode == 'enabledByUser' ||
-         plugin.enabledMode == 'enabledByPolicy';
+  return plugin.enabled_mode == 'enabledByUser' ||
+         plugin.enabled_mode == 'enabledByPolicy';
 }
 
 // Unfortunately, we don't have notifications for plugin (list) status changes
 // (yet), so in the meanwhile just update regularly.
-setInterval(requestPluginsData, 30000);
-
-// Get data and have it displayed upon loading.
-document.addEventListener('DOMContentLoaded', requestPluginsData);
+setInterval(function() {
+  whenBrowserProxyReady.then(function(browserProxy) {
+    return browserProxy.getPluginsData();
+  }).then(returnPluginsData);
+}, 30000);
 
 // Add handlers to static HTML elements.
 $('collapse').onclick = toggleTmiMode;
 $('expand').onclick = toggleTmiMode;
 $('details-link').onclick = toggleTmiMode;
+
+
+/**
+ * A promise to use for getting a reference to the proxy object allowing
+ * communication with the browser side.
+ * @type {!Promise}
+ */
+var whenBrowserProxyReady = getBrowserProxy_();

[Dan Beam, 2016/01/26 20:10:43]

creating a method just to call it once here adds no benefit

[dpapad, 2016/01/26 22:34:48]

Changed.

+
+// NOTE: Need to keep a reference to the stub here such that it is not garbage
+// collected, which causes the pipe to close and future calls from C++ to JS to
+// get dropped.
+var pluginsPageStub = null;
+
+/**
+ * Retrieves the proxy object to use for communicating with the browser side. It
+ * also performs necessary setup for the C++ side to make calls to JS at will.
+ * @return {!Promise}
+ */
+function getBrowserProxy_() {

[Dan Beam, 2016/01/26 20:10:43]

only @private methods or members should have a trailing _

[dpapad, 2016/01/26 22:34:48]

Removed this function.

+  return new Promise(function(resolve, reject) {
+    define([
+      'mojo/public/js/bindings',
+      'mojo/public/js/core',
+      'mojo/public/js/connection',
+      'chrome/browser/ui/webui/plugins/plugins.mojom',
+      'content/public/renderer/service_provider',
+    ], function(bindings, core, connection, pluginsMojom, serviceProvider) {
+      var browserProxy = connection.bindHandleToProxy(
+          serviceProvider.connectToService(
+              pluginsMojom.PluginsHandlerMojo.name),
+          pluginsMojom.PluginsHandlerMojo);
+
+      // Connect pipe handle to JS code.
+      var pipe = core.createMessagePipe();
+      pluginsPageStub = connection.bindHandleToStub(
+          pipe.handle0, pluginsMojom.PluginsPageMojo);
+
+      /** @constructor */
+      function PluginsPageImpl() {}
+      PluginsPageImpl.prototype = {
+        __proto__: pluginsMojom.PluginsPageMojo.stubClass.prototype,
+        onPluginsUpdated: function(plugins) {
+          returnPluginsData({plugins: plugins});
+        },
+      };
+      bindings.StubBindings(pluginsPageStub).delegate = new PluginsPageImpl();
+
+      // Send pipe handle to C++.
+      browserProxy.setClientPage(pipe.handle1);
+
+      resolve(browserProxy);
+    });
+  });
+}
+
+/**
+ * @return {!Promise} A promise firing when DOMContentLoaded event is received.
+ */
+function whenDomContentLoaded() {

[Dan Beam, 2016/01/26 20:10:43]

this doesn't need to be a function

[dpapad, 2016/01/26 22:34:48]

Changed.

+  return new Promise(function(resolve, reject) {
+    document.addEventListener('DOMContentLoaded', resolve);
+  });
+}
+
+Promise.all([
+  whenDomContentLoaded(),
+  whenBrowserProxyReady
+]).then(function(results) {
+  var browserProxy = results[1];
+  browserProxy.getPluginsData().then(returnPluginsData);
+  browserProxy.getShowDetails().then(function(response) {
+    loadShowDetailsFromPrefs(response.show_details);
+  });
+});