diff options
author | Nico Weber <thakis@chromium.org> | 2015-07-16 14:25:57 -0700 |
---|---|---|
committer | Nico Weber <thakis@chromium.org> | 2015-07-16 21:26:20 +0000 |
commit | 408c8d5ebeb5afb83cef000c9d16788323cfc7e7 (patch) | |
tree | a5c849157d5ac19fc72f63d2cd2726096141df88 /chrome/installer/setup | |
parent | ac1394f2a6ab863120fc7fa09da4ff8512e83cd7 (diff) | |
download | chromium_src-408c8d5ebeb5afb83cef000c9d16788323cfc7e7.zip chromium_src-408c8d5ebeb5afb83cef000c9d16788323cfc7e7.tar.gz chromium_src-408c8d5ebeb5afb83cef000c9d16788323cfc7e7.tar.bz2 |
Fix Wunused-const-variable warnings after https://codereview.chromium.org/1227843008/
Also move a class only used in official builds under that ifdef, and make a
few test-only consts global.
No intended behavior change. Fixes:
..\..\chrome\installer\setup\setup_util_unittest.cc(531,41) : error: unused
variable 'kVersionString' [-Werror,-Wunused-const-variable]
const wchar_t MigrateMultiToSingleTest::kVersionString[] = L"30.0.1574.0";
^
..\..\chrome\installer\setup\setup_util_unittest.cc(532,41) : error: unused
variable 'kMultiChannel' [-Werror,-Wunused-const-variable]
const wchar_t MigrateMultiToSingleTest::kMultiChannel[] =
^
BUG=153829
TBR=grt
Review URL: https://codereview.chromium.org/1239963005 .
Cr-Commit-Position: refs/heads/master@{#339125}
Diffstat (limited to 'chrome/installer/setup')
-rw-r--r-- | chrome/installer/setup/setup_util_unittest.cc | 19 |
1 files changed, 5 insertions, 14 deletions
diff --git a/chrome/installer/setup/setup_util_unittest.cc b/chrome/installer/setup/setup_util_unittest.cc index 0e01061..1ac23b8 100644 --- a/chrome/installer/setup/setup_util_unittest.cc +++ b/chrome/installer/setup/setup_util_unittest.cc @@ -508,7 +508,12 @@ TEST_F(FindArchiveToPatchTest, DesiredVersionNotFound) { EXPECT_EQ(base::FilePath().value(), patch_source.value()); } +#if defined(GOOGLE_CHROME_BUILD) namespace { +const bool kSystemLevel = false; +const HKEY kRootKey = kSystemLevel ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER; +const wchar_t kVersionString[] = L"30.0.1574.0"; +const wchar_t kMultiChannel[] = L"2.0-dev-multi-chromeframe"; class MigrateMultiToSingleTest : public testing::Test { protected: @@ -516,25 +521,11 @@ class MigrateMultiToSingleTest : public testing::Test { registry_override_manager_.OverrideRegistry(kRootKey); } - static const bool kSystemLevel = false; - static const HKEY kRootKey; - static const wchar_t kVersionString[]; - static const wchar_t kMultiChannel[]; - private: registry_util::RegistryOverrideManager registry_override_manager_; }; - -const bool MigrateMultiToSingleTest::kSystemLevel; -const HKEY MigrateMultiToSingleTest::kRootKey = - kSystemLevel ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER; -const wchar_t MigrateMultiToSingleTest::kVersionString[] = L"30.0.1574.0"; -const wchar_t MigrateMultiToSingleTest::kMultiChannel[] = - L"2.0-dev-multi-chromeframe"; - } // namespace -#if defined(GOOGLE_CHROME_BUILD) // Test migrating Chrome Frame from multi to single. TEST_F(MigrateMultiToSingleTest, ChromeFrame) { installer::ProductState chrome_frame; |