Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5464)

Unified Diff: chrome/installer/util/browser_distribution_unittest.cc

Issue 10446111: Abstract suffixing logic away from GetApplicationName (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase on suffix@r142211 Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/installer/util/browser_distribution.cc ('k') | chrome/installer/util/chrome_frame_distribution.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/util/browser_distribution_unittest.cc
diff --git a/chrome/installer/util/browser_distribution_unittest.cc b/chrome/installer/util/browser_distribution_unittest.cc
index 391320147faf19247bc6a4b96edd138b6fdf66bb..d0e1059d25c7a7973f3380e0c952bb4f26a491b7 100644
--- a/chrome/installer/util/browser_distribution_unittest.cc
+++ b/chrome/installer/util/browser_distribution_unittest.cc
@@ -16,11 +16,11 @@ TEST(BrowserDistributionTest, StringsTest) {
BrowserDistribution::GetSpecificDistribution(
BrowserDistribution::kProductTypes[i]);
ASSERT_TRUE(dist != NULL);
- std::wstring name = dist->GetApplicationName();
+ string16 name = dist->GetBaseAppName();
EXPECT_FALSE(name.empty());
- std::wstring desc = dist->GetAppDescription();
+ string16 desc = dist->GetAppDescription();
EXPECT_FALSE(desc.empty());
- std::wstring alt_name = dist->GetAlternateApplicationName();
+ string16 alt_name = dist->GetAlternateApplicationName();
EXPECT_FALSE(alt_name.empty());
}
}
@@ -28,10 +28,10 @@ TEST(BrowserDistributionTest, StringsTest) {
// The shortcut strings obtained by the shell utility functions should not
// be empty or be the same.
TEST(BrowserDistributionTest, AlternateAndNormalShortcutName) {
- std::wstring normal_name;
- std::wstring alternate_name;
- std::wstring appended_name_one;
- std::wstring appended_name_two;
+ string16 normal_name;
+ string16 alternate_name;
+ string16 appended_name_one;
+ string16 appended_name_two;
BrowserDistribution* dist = BrowserDistribution::GetDistribution();
EXPECT_TRUE(ShellUtil::GetChromeShortcutName(dist, false, L"",
&normal_name));
« no previous file with comments | « chrome/installer/util/browser_distribution.cc ('k') | chrome/installer/util/chrome_frame_distribution.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698