[WebAPKs] Display same text for menu & engagement banner

chrome/browser/android/shortcut_info_unittest.cc

 // 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.
 
 #include "chrome/browser/android/shortcut_info.h"
 
 #include "base/macros.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/public/common/manifest.h"
@@ skipping 36 matching lines @@
   ASSERT_EQ(manifest_.short_name.string(), info_.name);
 }
 
 TEST_F(ShortcutInfoTest, ShortNameFallsBackToName) {
   manifest_.name = base::NullableString16(base::ASCIIToUTF16("name"), false);
   info_.UpdateFromManifest(manifest_);
 
   ASSERT_EQ(manifest_.name.string(), info_.short_name);
 }
 
-TEST_F(ShortcutInfoTest, UserTitleBecomesShortName) {
-  manifest_.short_name =
-      base::NullableString16(base::ASCIIToUTF16("name"), false);
-  info_.user_title = base::ASCIIToUTF16("title");
+// Test that if a manifest with an empty name and empty short_name is passed,
+// that ShortcutInfo::UpdateFromManifest() does not overwrite the current
+// ShortcutInfo::name and ShortcutInfo::short_name.
+TEST_F(ShortcutInfoTest, IgnoreEmptyNameAndShortName) {
+  base::string16 initial_name(base::ASCIIToUTF16("name"));
+  base::string16 initial_short_name(base::ASCIIToUTF16("short_name"));
+
+  manifest_.display = blink::kWebDisplayModeStandalone;
+  info_.name = initial_name;
+  info_.short_name = initial_short_name;
+  manifest_.name = base::NullableString16(base::string16(), false);
   info_.UpdateFromManifest(manifest_);
 
-  ASSERT_EQ(manifest_.short_name.string(), info_.user_title);
+  ASSERT_EQ(initial_name, info_.name);
+  ASSERT_EQ(initial_short_name, info_.short_name);
 }