Add a chrome://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
new file mode 100644
index 0000000000000000000000000000000000000000..6fa3f5182dd6a8a5ac83314aef1ce4c9c326ded7
--- /dev/null
+++ b/chrome/browser/android/webapk/webapk_info.h
@@ -0,0 +1,39 @@
+// 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.
+
+#ifndef CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INFO_H_
+#define CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INFO_H_
+
+#include <string>
+
+#include "base/macros.h"
+
+// Structure with information about a WebAPK.

[dominickn, 2017/01/24 23:15:20]

Nit: add a comment like:

"This class is passed around in a std::vector to generate the chrome://webapks
page. To reduce copying overhead, this class is move-only, and move-constructs
its string arguments (which are copied from Java to C++ into a temporary prior
to construction)".

[gonzalon, 2017/01/24 23:37:56]

Done.

+struct WebApkInfo {
+  WebApkInfo(const std::string&& short_name,

[dominickn, 2017/01/24 23:15:20]

Remove the const from the two string arguments. The main reason for taking
rvalue references (&&) as arguments is so they can be moved into this object
(i.e. the original strings are dead after the constructor is run because this
class has stolen their internal buffers). This is nice because the argument
strings are in temporary vectors anyway that disappear after
ShortcutHelper::OnWebApksRetrieved finishes running. Enforcing that the
reference is const is a little beside the point.

[gonzalon, 2017/01/24 23:37:56]

Makes sense, removed!

+             const std::string&& package_name,
+             int shell_apk_version,
+             int version_code);
+  ~WebApkInfo();
+
+  WebApkInfo& operator=(WebApkInfo&& other) = default;
+  WebApkInfo(WebApkInfo&& other) = default;
+
+  // Short name of the WebAPK.
+  std::string short_name;
+
+  // Package name of the WebAPK.
+  std::string package_name;
+
+  // Shell APK version of the WebAPK.
+  int shell_apk_version;
+
+  // Version code of the WebAPK.
+  int version_code;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(WebApkInfo);
+};
+
+#endif  // CHROME_BROWSER_ANDROID_WEBAPK_WEBAPK_INFO_H_