webapps: initial addition of splash screen icon downloading

chrome/browser/android/shortcut_helper.h

Index: chrome/browser/android/shortcut_helper.h
diff --git a/chrome/browser/android/shortcut_helper.h b/chrome/browser/android/shortcut_helper.h
index 93e1c9d589a86307befc9bbc041867eb15ffadae..6c61479e696d83e390200f0f7f0d10e27256787b 100644
--- a/chrome/browser/android/shortcut_helper.h
+++ b/chrome/browser/android/shortcut_helper.h
@@ -11,8 +11,7 @@
 #include "third_party/skia/include/core/SkBitmap.h"
 
 // ShortcutHelper is the C++ counterpart of org.chromium.chrome.browser's
-// ShortcutHelper in Java. The object is owned by the Java object. It is created
-// from there via a JNI (Initialize) call and MUST BE DESTROYED via Destroy().
+// ShortcutHelper in Java.
 class ShortcutHelper {

[mlamouri (slow - plz ping), 2015/08/24 16:43:54]

Maybe make it final?

[Lalit Maganti, 2015/08/25 11:38:44]

Done.

  public:
   // Registers JNI hooks.
@@ -21,9 +20,16 @@ class ShortcutHelper {
   // Adds a shortcut to the launcher using a SkBitmap.
   // Must be called on the IO thread.
   static void AddShortcutInBackgroundWithSkBitmap(const ShortcutInfo& info,
+                                                  const std::string& id,

[mlamouri (slow - plz ping), 2015/08/24 16:43:54]

nit: id -> webapp_id

[Lalit Maganti, 2015/08/25 11:38:44]

Done.

                                                   const SkBitmap& icon_bitmap);
+
+  // Add splashscreen icon to the data of the webapp.
+  static void AddSplashscreenIconToWebappData(const std::string& id,

[mlamouri (slow - plz ping), 2015/08/24 16:43:54]

ditto

[Lalit Maganti, 2015/08/25 11:38:44]

Done.

+                                              const SkBitmap& splash_icon);
+
  private:
-  ShortcutHelper();
+  ShortcutHelper() = delete;
+  ~ShortcutHelper() = delete;
 
   DISALLOW_COPY_AND_ASSIGN(ShortcutHelper);
 };