Fix small logic error in AppBannerManagerAndroid::GetName()

This ternery expression got flipped in:
https://chromium-review.googlesource.com/c/chromium/src/+/3060401/23/components/webapps/browser/android/app_banner_manager_android.cc#468
As a result sites that specify a short_name in their mainfest have
empty string app name.

This CL restores the logic back to what it was.

Bug: 1238735
Change-Id: I7c5aa4950ca99a8fe9cc9ac8b615702eb66dd46a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3089985
Reviewed-by: Glenn Hartmann <[email protected]>
Commit-Queue: Alan Cutter <[email protected]>
Cr-Commit-Position: refs/heads/master@{#911599}

Author: alancutter

components/webapps/browser/android/app_banner_manager_android.cc

@@ -467,7 +467,7 @@ std::u16string AppBannerManagerAndroid::GetAppName() const {
     // Prefer the short name if it's available. It's guaranteed that at least
     // one of these is non-empty.
     std::u16string short_name = manifest().short_name.value_or(u"");
-    return short_name.empty() ? manifest().name.value_or(u"") : u"";
+    return short_name.empty() ? manifest().name.value_or(u"") : short_name;
   }
 
   return native_app_title_;