webapps: initial addition of splash screen icon downloading

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 "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 103 matching lines @@
                            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);
 
   // Called when it is determined that the webapp has fulfilled the initial
   // criteria of having a manifest and a service worker.
-  void OnHasServiceWorker(content::WebContents* web_contents);
+  virtual void OnHasServiceWorker(content::WebContents* web_contents);

[Lalit Maganti, 2015/08/26 13:11:56]

Reverted this change as it is not necessary and also made this method private.

 
   content::WebContents* GetWebContents();
   virtual std::string GetAppIdentifier();
   const content::Manifest& web_app_data() { return web_app_data_; }
   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);
 
   // Records that a banner was shown. The |event_name| corresponds to the RAPPOR
   // metric being recorded.
   void RecordDidShowBanner(const std::string& event_name);
 
+  // Returns whether the fetcher is active and web contents have not been
+  // closed.
+  bool CheckFetcherIsStillAlive(content::WebContents* web_contents);

[Lalit Maganti, 2015/08/26 13:11:56]

Reverted as no longer necessary.

+
  private:
   // Callbacks for data retrieval.
   void OnDidGetManifest(const content::Manifest& manifest);
   void OnDidCheckHasServiceWorker(bool has_service_worker);
   void OnAppIconFetched(const SkBitmap& bitmap);
 
   // Returns whether the given web app has already been installed.
   // Implemented on desktop platforms only.
   virtual bool IsWebAppInstalled(content::BrowserContext* browser_context,
                                  const GURL& start_url);
 
   // Record that the banner could be shown at this point, if the triggering
   // heuristic allowed.
   void RecordCouldShowBanner();
 
   // Creates a banner for the app using the given |icon|.
   virtual void ShowBanner(const SkBitmap* icon,
                           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);
 
   const int ideal_icon_size_;
   const base::WeakPtr<Delegate> weak_delegate_;
   base::ObserverList<Observer> observer_list_;
   bool is_active_;
   bool was_canceled_by_page_;
   bool page_requested_prompt_;
   ui::PageTransition transition_type_;
   int event_request_id_;
   scoped_ptr<SkBitmap> app_icon_;
   std::string referrer_;
 
   GURL validated_url_;
   base::string16 app_title_;
   content::Manifest web_app_data_;
 
   friend class AppBannerDataFetcherUnitTest;
   friend class base::RefCounted<AppBannerDataFetcher>;
   DISALLOW_COPY_AND_ASSIGN(AppBannerDataFetcher);
 };
 
 }  // namespace banners
 
 #endif  // CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_H_