Commit 7ebc2c47 authored by wfh's avatar wfh Committed by Commit bot

Make 64-bit Chrome stable report as CHANNEL_STABLE

BUG=407204
TEST=installer_util_unittests
TEST=install x64 stable.  Check chrome://chrome shows 'm' and not 'unknown-m'.

Review URL: https://codereview.chromium.org/467303004

Cr-Commit-Position: refs/heads/master@{#291791}
parent e9c71127
...@@ -27,7 +27,8 @@ const wchar_t kSfxMultiFail[] = L"-multifail"; ...@@ -27,7 +27,8 @@ const wchar_t kSfxMultiFail[] = L"-multifail";
const wchar_t* const kChannels[] = { const wchar_t* const kChannels[] = {
installer::kChromeChannelBeta, installer::kChromeChannelBeta,
installer::kChromeChannelDev installer::kChromeChannelDev,
installer::kChromeChannelStableExplicit
}; };
const wchar_t* const kModifiers[] = { const wchar_t* const kModifiers[] = {
...@@ -168,7 +169,11 @@ bool ChannelInfo::GetChannelName(std::wstring* channel_name) const { ...@@ -168,7 +169,11 @@ bool ChannelInfo::GetChannelName(std::wstring* channel_name) const {
*const* end = &kChannels[arraysize(kChannels)]; scan != end; *const* end = &kChannels[arraysize(kChannels)]; scan != end;
++scan) { ++scan) {
if (value_.find(*scan) != std::wstring::npos) { if (value_.find(*scan) != std::wstring::npos) {
channel_name->assign(*scan); // Report channels with "stable" in them as stable (empty string).
if (*scan == installer::kChromeChannelStableExplicit)
channel_name->erase();
else
channel_name->assign(*scan);
return true; return true;
} }
} }
......
...@@ -50,6 +50,16 @@ TEST(ChannelInfoTest, Channels) { ...@@ -50,6 +50,16 @@ TEST(ChannelInfoTest, Channels) {
EXPECT_TRUE(ci.GetChannelName(&channel)); EXPECT_TRUE(ci.GetChannelName(&channel));
EXPECT_EQ(kChannelDev, channel); EXPECT_EQ(kChannelDev, channel);
ci.set_value(L"x64-dev");
EXPECT_TRUE(ci.GetChannelName(&channel));
EXPECT_EQ(kChannelDev, channel);
ci.set_value(L"x64-beta");
EXPECT_TRUE(ci.GetChannelName(&channel));
EXPECT_EQ(kChannelBeta, channel);
ci.set_value(L"x64-stable");
EXPECT_TRUE(ci.GetChannelName(&channel));
EXPECT_EQ(kChannelStable, channel);
ci.set_value(L"fuzzy"); ci.set_value(L"fuzzy");
EXPECT_FALSE(ci.GetChannelName(&channel)); EXPECT_FALSE(ci.GetChannelName(&channel));
} }
......
...@@ -263,6 +263,7 @@ const wchar_t kChromeChannelCanary[] = L"canary"; ...@@ -263,6 +263,7 @@ const wchar_t kChromeChannelCanary[] = L"canary";
const wchar_t kChromeChannelDev[] = L"dev"; const wchar_t kChromeChannelDev[] = L"dev";
const wchar_t kChromeChannelBeta[] = L"beta"; const wchar_t kChromeChannelBeta[] = L"beta";
const wchar_t kChromeChannelStable[] = L""; const wchar_t kChromeChannelStable[] = L"";
const wchar_t kChromeChannelStableExplicit[] = L"stable";
const size_t kMaxAppModelIdLength = 64U; const size_t kMaxAppModelIdLength = 64U;
......
...@@ -256,6 +256,7 @@ extern const wchar_t kChromeChannelCanary[]; ...@@ -256,6 +256,7 @@ extern const wchar_t kChromeChannelCanary[];
extern const wchar_t kChromeChannelDev[]; extern const wchar_t kChromeChannelDev[];
extern const wchar_t kChromeChannelBeta[]; extern const wchar_t kChromeChannelBeta[];
extern const wchar_t kChromeChannelStable[]; extern const wchar_t kChromeChannelStable[];
extern const wchar_t kChromeChannelStableExplicit[];
extern const size_t kMaxAppModelIdLength; extern const size_t kMaxAppModelIdLength;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment