Sanitize the profile's name when used to create a shortcut.

chrome/browser/profiles/profile_shortcut_manager_unittest_win.cc

 // Copyright (c) 2012 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 <objbase.h>  // For CoInitialize().
 
 #include "base/file_util.h"
 #include "base/location.h"
 #include "base/message_loop.h"
 #include "base/path_service.h"
@@ skipping 218 matching lines @@
 
 TEST_F(ProfileShortcutManagerTest, ShortcutFilename) {
   const string16 kProfileName = L"Harry";
   const string16 expected_name = kProfileName + L" - " +
       distribution_->GetAppShortCutName() + installer::kLnkExt;
   EXPECT_EQ(expected_name,
             profiles::internal::GetShortcutFilenameForProfile(kProfileName,
                                                               distribution_));
 }
 
+TEST_F(ProfileShortcutManagerTest, ShortcutLongFilenameIsTrimmed) {
+  const string16 kLongProfileName = L"Harry Harry Harry Harry Harry Harry Harry"
+      L"Harry Harry Harry Harry Harry Harry Harry Harry Harry Harry Harry"
+      L"Harry Harry Harry Harry Harry Harry Harry Harry Harry Harry Harry";
+  string16 file_name =
+      profiles::internal::GetShortcutFilenameForProfile(
+          kLongProfileName, distribution_);
+  EXPECT_LT(file_name.size(), kLongProfileName.size());
+}
+
+TEST_F(ProfileShortcutManagerTest, ShortcutFilenameStripsReservedCharacters) {
+  const string16 kProfileName = L"<Harry/>";
+  const string16 kSanitizedProfileName = L"Harry";
+  const string16 expected_name = kSanitizedProfileName + L" - " +
+    distribution_->GetAppShortCutName() + installer::kLnkExt;
+  EXPECT_EQ(expected_name,
+    profiles::internal::GetShortcutFilenameForProfile(kProfileName,
+    distribution_));
+}
+
 TEST_F(ProfileShortcutManagerTest, UnbadgedShortcutFilename) {
   EXPECT_EQ(distribution_->GetAppShortCutName() + installer::kLnkExt,
             profiles::internal::GetShortcutFilenameForProfile(string16(),
                                                               distribution_));
 }
 
 TEST_F(ProfileShortcutManagerTest, ShortcutFlags) {
   const string16 kProfileName = L"MyProfileX";
   const FilePath profile_path =
       profile_info_cache_->GetUserDataDir().Append(kProfileName);
@@ skipping 190 matching lines @@
       CreateRegularShortcutWithName(FROM_HERE, L"MyChrome");
 
   // Add another profile and check that one shortcut was renamed and that the
   // other shortcut was updated but kept the same name.
   CreateProfileWithShortcut(FROM_HERE, profile_2_name_, profile_2_path_);
   EXPECT_FALSE(file_util::PathExists(regular_shortcut_path));
   ValidateProfileShortcutAtPath(FROM_HERE, customized_regular_shortcut_path,
                                 profile_1_path_);
   ValidateProfileShortcut(FROM_HERE, profile_1_name_, profile_1_path_);
 }