Upstream: Add additional checks before creating a WebAPK after clicking "Add to Homescreen"

chrome/browser/banners/webapp_manifest_validator.h

Index: chrome/browser/banners/webapp_manifest_validator.h
diff --git a/chrome/browser/banners/webapp_manifest_validator.h b/chrome/browser/banners/webapp_manifest_validator.h
new file mode 100644
index 0000000000000000000000000000000000000000..6d11f5f0f20de7841b0bb28f9aa2d680f4319e5e
--- /dev/null
+++ b/chrome/browser/banners/webapp_manifest_validator.h
@@ -0,0 +1,31 @@
+// Copyright 2016 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_BANNERS_WEBAPP_MANIFEST_VALIDATOR_H_
+#define CHROME_BROWSER_BANNERS_WEBAPP_MANIFEST_VALIDATOR_H_
+
+#include "chrome/browser/banners/app_banner_debug_log.h"
+
+namespace content {
+struct Manifest;
+}
+
+namespace banners {
+namespace webapp_manifest_validator {
+
+// Returns whether a page with |manifest| can be launched as a fullscreen web
+// app. If |manifest| prevents the page from launching as a fullscreen web app,
+// sets |code| to the reason why.
+bool IsWebappCapable(const content::Manifest& manifest,
+                     OutputDeveloperMessageCode* code);
+
+// Returns true if a page with |manifest| will make a good fullscreen web app.
+// If not, sets |code| as to why.

[pkotwicz, 2016/06/09 01:39:38]

I am not sold on this function name. Do you have any suggestions?

+bool IsManifestGoodForWebapp(const content::Manifest& manifest,
+                             OutputDeveloperMessageCode* code);
+
+}  //  namespace webapp_manifest_validator
+}  //  namespace banners
+
+#endif  // CHROME_BROWSER_BANNERS_WEBAPP_MANIFEST_VALIDATOR_H_