Adds more metadata to the about:webapks page

chrome/browser/android/webapk/webapk_info.h

Index: chrome/browser/android/webapk/webapk_info.h
diff --git a/chrome/browser/android/webapk/webapk_info.h b/chrome/browser/android/webapk/webapk_info.h
index 746c5f517f29d43679cc5c8c80e05e74a7bafae8..0cf6b5e79b02865253808a67191158b11a26abb1 100644
--- a/chrome/browser/android/webapk/webapk_info.h
+++ b/chrome/browser/android/webapk/webapk_info.h
@@ -8,6 +8,8 @@
 #include <string>
 
 #include "base/macros.h"
+#include "content/public/common/manifest.h"

[pkotwicz, 2017/02/27 20:50:58]

Why this include?

Do you need to include third_party/WebKit/public/platform/WebDisplayMode.h

+#include "third_party/WebKit/public/platform/modules/screen_orientation/WebScreenOrientationLockType.h"
 
 // Structure with information about a WebAPK.
 //
@@ -16,16 +18,28 @@
 // move-constructs its string arguments (which are copied from Java to C++ into
 // a temporary prior to construction).
 struct WebApkInfo {
-  WebApkInfo(std::string short_name,
+  WebApkInfo(std::string name,
+             std::string short_name,
              std::string package_name,
              int shell_apk_version,
-             int version_code);
+             int version_code,
+             std::string uri,
+             std::string scope,
+             std::string manifest_url,
+             std::string manifest_start_url,
+             blink::WebDisplayMode display,
+             blink::WebScreenOrientationLockType orientation,
+             int64_t theme_color,
+             int64_t background_color);
   ~WebApkInfo();
 
   WebApkInfo& operator=(WebApkInfo&& other) = default;
   WebApkInfo(WebApkInfo&& other) = default;
 
   // Short name of the WebAPK.
+  std::string name;
+
+  // Short name of the WebAPK.
   std::string short_name;
 
   // Package name of the WebAPK.
@@ -37,6 +51,15 @@ struct WebApkInfo {
   // Version code of the WebAPK.
   int version_code;
 

[pkotwicz, 2017/02/27 20:50:58]

Can you please add comments for each member variable?

+  std::string uri;
+  std::string scope;
+  std::string manifest_url;
+  std::string manifest_start_url;
+  blink::WebDisplayMode display;
+  blink::WebScreenOrientationLockType orientation;
+  int64_t theme_color;
+  int64_t background_color;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(WebApkInfo);
 };