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

chrome/browser/banners/app_banner_data_fetcher.h

 // Copyright 2015 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_APP_BANNER_DATA_FETCHER_H_
 #define CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/time/time.h"
+#include "chrome/browser/banners/app_banner_debug_log.h"
 #include "chrome/common/web_application_info.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/common/manifest.h"
 #include "third_party/WebKit/public/platform/modules/app_banner/WebAppBannerPromptReply.h"
 
 namespace chrome {
 class BitmapFetcher;
 }  // namespace chrome
@@ skipping 98 matching lines @@
   void OnBannerPromptReply(content::RenderFrameHost* render_frame_host,
                            int request_id,
                            blink::WebAppBannerPromptReply reply,
                            std::string referrer);
 
   // Called when the client has prevented a banner from being shown, and is
   // now requesting that it be shown later.
   void OnRequestShowAppBanner(content::RenderFrameHost* render_frame_host,
                               int request_id);
 
+  // Returns whether the given Manifest meets the requirements to show a web app
+  // banner. If not, sets |code| to the reason why.
+  virtual bool CheckManifest(const content::Manifest& manifest,
+                             OutputDeveloperMessageCode* code);
+
   content::WebContents* GetWebContents();
   virtual std::string GetAppIdentifier();
   const GURL& manifest_url() { return manifest_url_; }
   const content::Manifest& manifest() { return manifest_; }
   void set_app_title(const base::string16& title) { app_title_ = title; }
   int event_request_id() { return event_request_id_; }
 
   // Fetches the icon at the given URL asynchronously, returning |false| if a
   // load could not be started.
   bool FetchAppIcon(content::WebContents* web_contents, const GURL& url);
@@ skipping 29 matching lines @@
                           const base::string16& title,
                           const std::string& referrer) = 0;
 
   // Returns whether the banner should be shown.
   bool CheckIfShouldShowBanner();
 
   // Returns whether the fetcher is active and web contents have not been
   // closed.
   bool CheckFetcherIsStillAlive(content::WebContents* web_contents);
 
-  // Returns whether the given Manifest is following the requirements to show
-  // a web app banner.
-  static bool IsManifestValidForWebApp(const content::Manifest& manifest,
-                                       content::WebContents* web_contents,
-                                       bool is_debug_mode);
-
   const base::WeakPtr<Delegate> weak_delegate_;
   const int ideal_icon_size_in_dp_;
   const int minimum_icon_size_in_dp_;
   base::ObserverList<Observer> observer_list_;
   bool is_active_;
   bool was_canceled_by_page_;
   bool page_requested_prompt_;
   const bool is_debug_mode_;
   ui::PageTransition transition_type_;
   int event_request_id_;
@@ skipping 13 matching lines @@
       content::BrowserThread::UI>;
   friend class base::DeleteHelper<AppBannerDataFetcher>;
   friend class AppBannerDataFetcherUnitTest;
   friend class base::RefCounted<AppBannerDataFetcher>;
   DISALLOW_COPY_AND_ASSIGN(AppBannerDataFetcher);
 };
 
 }  // namespace banners
 
 #endif  // CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_H_