Correctly record WebAPK install source for Launch.HomescreenSource

chrome/browser/android/shortcut_info.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_ANDROID_SHORTCUT_INFO_H_
 #define CHROME_BROWSER_ANDROID_SHORTCUT_INFO_H_
 
 #include <stdint.h>
 
 #include <vector>
@@ skipping 31 matching lines @@
 
     // Used for bookmark-type shortcuts that launch the tabbed browser.
     SOURCE_ADD_TO_HOMESCREEN_SHORTCUT = 8,
 
     // Used for WebAPKs launched via an external intent.
     SOURCE_EXTERNAL_INTENT = 9,
 
     // Used for WebAPK PWAs added via the banner.
     SOURCE_APP_BANNER_WEBAPK = 10,
 
-    SOURCE_COUNT = 11
+    // Used for WebAPK PWAs whose install source info are lost.

[pkotwicz, 2017/05/05 21:05:43]

Nit: "are lost" -> "was lost" (because "source information" is singular)

[Xi Han, 2017/05/08 20:55:28]

Thanks!

+    SOURCE_WEBAPK_UNKNOWN = 11,
+
+    SOURCE_COUNT = 12
   };
 
   explicit ShortcutInfo(const GURL& shortcut_url);
   ShortcutInfo(const ShortcutInfo& other);
   ~ShortcutInfo();
 
   // Updates the info based on the given |manifest|.
   void UpdateFromManifest(const content::Manifest& manifest);
 
   // Updates the source of the shortcut.
@@ skipping 12 matching lines @@
   int64_t background_color;
   int ideal_splash_image_size_in_px;
   int minimum_splash_image_size_in_px;
   GURL splash_image_url;
   GURL best_primary_icon_url;
   GURL best_badge_icon_url;
   std::vector<std::string> icon_urls;
 };
 
 #endif  // CHROME_BROWSER_ANDROID_SHORTCUT_INFO_H_