Mojo-ifying chrome://version.

components/version_ui/resources/about_version.js

 // Copyright (c) 2011 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.
 
-/**
- * Callback from the backend with the list of variations to display.
- * This call will build the variations section of the version page, or hide that
- * section if there are none to display.
- * @param {!Array<string>} variationsList The list of variations.
- */
-function returnVariationInfo(variationsList) {
-  $('variations-section').hidden = !variationsList.length;
-  $('variations-list').appendChild(
-      parseHtmlSubset(variationsList.join('<br>'), ['BR']));
-}
+define([
+  'mojo/public/js/connection',
+  'chrome/browser/ui/webui/version.mojom',

[Evan Stade, 2015/12/04 22:30:20]

why is this not version.mojom.js (with the same change in version_ui.cc)?

mojo/public/js/constants.cc has a similar naming convention, but since these are
all js files it seems clearer to retain the .js suffix, especially as there's a
version.mojom file which this is NOT loading.

Perhaps Scott can comment if this is a general mojo pattern.

[dpapad, 2015/12/04 23:28:14]

I was just following the pattern, see the other two places were Mojo is used in
WebUI
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
and
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/....

I agree that it would be clearer to retain the '.js' suffix and I am fine
changing this if we don't mind about breaking the (emerging) pattern (I would
prefer not to edit site_engagement and omnibox in this CL to keep it smaller).

[Evan Stade, 2015/12/04 23:32:26]

now seems like a great time to set the right precedent

[sky, 2015/12/04 23:58:39]

That's the id of the module, which need not match the file name. It was decided
to match the file name the mojom comes from and not the on disk file. I didn't
come up with the naming. I think aa or abarth likely came up with the naming. I
don't feel strongly that it should be .mojom vs .mojom.js. I suggest moving
naming discussion to a separate thread else you'll derail this patch.

[Evan Stade, 2015/12/05 00:07:25]

OK, I guess there's more precedent for this than I initially thought: 

https://code.google.com/p/chromium/codesearch#chromium/src/out/Debug/gen/comp...

I have no problem going with the flow here.

+  'content/public/renderer/service_provider',
+], function(connection, versionMojom, serviceProvider) {
+  // TODO(dpapad): Need to wait for DOMContentLoaded too?
+  var uiHandler = connection.bindHandleToProxy(
+      serviceProvider.connectToService(
+          versionMojom.VersionHandlerMojo.name),
+      versionMojom.VersionHandlerMojo);
 
-/**
- * Callback from the backend with the executable and profile paths to display.
- * @param {string} execPath The executable path to display.
- * @param {string} profilePath The profile path to display.
- */
-function returnFilePaths(execPath, profilePath) {
-  $('executable_path').textContent = execPath;
-  $('profile_path').textContent = profilePath;
-}
-
-/**
- * Callback from the backend with the Flash version to display.
- * @param {string} flashVersion The Flash version to display.
- */
-function returnFlashVersion(flashVersion) {
-  $('flash_version').textContent = flashVersion;
-}
-
-/**
- * Callback from the backend with the OS version to display.
- * @param {string} osVersion The OS version to display.
- */
-function returnOsVersion(osVersion) {
-  $('os_version').textContent = osVersion;
-}
-
-/* All the work we do onload. */
-function onLoadWork() {
-  chrome.send('requestVersionInfo');
-}
-
-document.addEventListener('DOMContentLoaded', onLoadWork);
+  uiHandler.getFilePaths().then(function(response) {
+    $('executable_path').textContent = response.execPath;
+    $('profile_path').textContent = response.profilePath;
+  });
+  uiHandler.getFlashVersion().then(function(response) {
+    $('flash_version').textContent = response.flashVersion;
+  });
+  uiHandler.getVariations().then(function(response) {
+    $('variations-section').hidden = response.variations.length == 0;
+    $('variations-list').appendChild(
+        parseHtmlSubset(response.variations.join('<br>'), ['BR']));
+  });
+  if (cr.isChromeOS) {
+    uiHandler.getOsVersion().then(function(response) {
+      $('os_version').textContent = response.osVersion;
+    });
+  }
+});