[Android] Make WebApplicationInfo and Web Manifest mutually exclusive

chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc

Index: chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
index e5f3bc8762e6a643138a6cdda0056b39e99880f3..4f81469e28db610ff5cdbe43ea129c46e68967a7 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
@@ -207,32 +207,17 @@ void AddToHomescreenDataFetcher::OnDidPerformInstallableCheck(
 
   is_installable_check_complete_ = true;
 
-  bool webapk_compatible = false;
-  if (check_webapk_compatibility_) {
-    webapk_compatible = (data.error_code == NO_ERROR_DETECTED &&
-                         AreWebManifestUrlsWebApkCompatible(data.manifest));
-    weak_observer_->OnDidDetermineWebApkCompatibility(webapk_compatible);
-
-    if (webapk_compatible) {
-      // WebAPKs are wholly defined by the Web Manifest. Ignore the <meta> tag
-      // data received in OnDidGetWebApplicationInfo().
-      shortcut_info_ = ShortcutInfo(GURL());
-    }
-  }
-
   if (!data.manifest.IsEmpty()) {
+    shortcut_info_ = ShortcutInfo(GURL());
+
     base::RecordAction(base::UserMetricsAction("webapps.AddShortcut.Manifest"));
     shortcut_info_.UpdateFromManifest(data.manifest);

[dominickn, 2017/06/14 05:16:51]

This isn't quite what I was suggesting on the other CL. What I was suggesting
was adding an if statement right before this line:

if (!data.manifest.name.string().empty() ||
!data.manifest.short_name.string().empty()) {
  shortcut_info_.UpdateFromManifest(data.manifest);
}

The expected behaviour is that a site which specifies a manifest but has no name
or short name shows the <title> in the A2HS dialog. We do this by simply not
updating from the manifest if they haven't bothered to specify a name or
short_name. If they want a site that's standalone they fill in one of those two
fields.

The change you've made means that sites with a manifest with empty name or
short_name end up with an empty title in the A2HS dialog.

     shortcut_info_.manifest_url = data.manifest_url;
+  }
 
-    if (webapk_compatible) {
-      shortcut_info_.UpdateSource(ShortcutInfo::SOURCE_ADD_TO_HOMESCREEN_PWA);
-
-      if (data.badge_icon && !data.badge_icon->drawsNothing()) {
-        shortcut_info_.best_badge_icon_url = data.badge_icon_url;
-        badge_icon_ = *data.badge_icon;
-      }
-    }
+  if (data.badge_icon && !data.badge_icon->drawsNothing()) {
+    shortcut_info_.best_badge_icon_url = data.badge_icon_url;
+    badge_icon_ = *data.badge_icon;
   }
 
   // Save the splash screen URL for the later download.
@@ -244,6 +229,16 @@ void AddToHomescreenDataFetcher::OnDidPerformInstallableCheck(
   shortcut_info_.minimum_splash_image_size_in_px =
       minimum_splash_image_size_in_px_;
 
+  if (check_webapk_compatibility_) {
+    bool webapk_compatible =
+        (data.error_code == NO_ERROR_DETECTED &&
+         AreWebManifestUrlsWebApkCompatible(data.manifest));
+    weak_observer_->OnDidDetermineWebApkCompatibility(webapk_compatible);
+
+    if (webapk_compatible)
+      shortcut_info_.UpdateSource(ShortcutInfo::SOURCE_ADD_TO_HOMESCREEN_PWA);
+  }
+
   weak_observer_->OnUserTitleAvailable(shortcut_info_.user_title);
 
   if (data.primary_icon) {