Adds more metadata to the about:webapks page

chrome/browser/resources/webapks/about_webapks.js

 // Copyright 2017 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.
 
 /**
  * @typedef {{
+ *   name: string,
  *   shortName: string,
  *   packageName: string,
  *   shellApkVersion: number,
  *   versionCode: number
+ *   uri: string,
+ *   scope: string,
+ *   manifestUrl: string,
+ *   manifestStartUrl: string,
+ *   displayMode: string,
+ *   orientation: string,
+ *   themeColor: string,
+ *   backgroundColor: string,
  * }}
  */
 var WebApkInfo;
 
 /**
  * Creates and returns an element (with |text| as content) assigning it the
  * |className| class.
  *
  * @param {string} text Text to be shown in the span.
  * @param {string} className Class to be assigned to the new element.
@@ skipping 11 matching lines @@
  * one will be appended at the end of the other.
  *
  * @param {!Array<WebApkInfo>} webApkList List of objects with information about
  * WebAPKs installed.
  */
 function returnWebApksInfo(webApkList) {
   for (let webApkInfo of webApkList) {
     addWebApk(webApkInfo);
   }
 }
 

[Dan Beam, 2017/02/24 18:36:16]

can you add @param documentation here?

[gonzalon, 2017/02/24 22:30:52]

Done.

+function addWebApkField(webApkList, label, value) {
+  webApkList.appendChild(document.createElement('br'));

[Dan Beam, 2017/02/24 18:36:16]

instead of <br><span>, why not just <div>?

[gonzalon, 2017/02/24 22:30:52]

Done.

+  webApkList.appendChild(createSpanWithTextAndClass(
+      label, 'app-property-label'));
+  webApkList.appendChild(document.createTextNode(value));
+}
+
 /**
  * Adds a new entry to the page with the information of a WebAPK.
  *
  * @param {WebApkInfo} webApkInfo Information about an installed WebAPK.
  */
 function addWebApk(webApkInfo) {
   var webApkList = $('webapk-list');
 
   webApkList.appendChild(
-      createSpanWithTextAndClass(webApkInfo.shortName, 'app-name'));
+      createSpanWithTextAndClass(webApkInfo.name, 'app-name'));
 
   webApkList.appendChild(
-      createSpanWithTextAndClass('Package name: ', 'app-property-label'));
-  webApkList.appendChild(document.createTextNode(webApkInfo.packageName));
+      createSpanWithTextAndClass('Short name: ', 'app-property-label'));
+  webApkList.appendChild(document.createTextNode(webApkInfo.shortName));
 
-  webApkList.appendChild(document.createElement('br'));
-  webApkList.appendChild(createSpanWithTextAndClass(
-      'Shell APK version: ', 'app-property-label'));
-  webApkList.appendChild(document.createTextNode(webApkInfo.shellApkVersion));
-
-  webApkList.appendChild(document.createElement('br'));
-  webApkList.appendChild(
-      createSpanWithTextAndClass('Version code: ', 'app-property-label'));
-  webApkList.appendChild(document.createTextNode(webApkInfo.versionCode));
+  addWebApkField(webApkList, 'Package name: ', webApkInfo.packageName);
+  addWebApkField(webApkList, 'Shell APK version: ', webApkInfo.shellApkVersion);
+  addWebApkField(webApkList, 'Version code: ', webApkInfo.versionCode);
+  addWebApkField(webApkList, 'URI: ', webApkInfo.uri);
+  addWebApkField(webApkList, 'Scope: ', webApkInfo.scope);
+  addWebApkField(webApkList, 'Manifest URL: ', webApkInfo.manifestUrl);
+  addWebApkField(webApkList, 'Manifest Start URL: ', webApkInfo.manifestStartUrl);

[Dan Beam, 2017/02/24 18:36:16]

80 col wrap

[gonzalon, 2017/02/24 22:30:52]

Done.

+  addWebApkField(webApkList, 'Display Mode: ', webApkInfo.displayMode);
+  addWebApkField(webApkList, 'Orientation: ', webApkInfo.orientation);
+  addWebApkField(webApkList, 'Theme color: ', webApkInfo.themeColor);
+  addWebApkField(webApkList, 'Background color: ', webApkInfo.backgroundColor);
 }
 
 document.addEventListener('DOMContentLoaded', function() {
   chrome.send('requestWebApksInfo');
 });