summaryrefslogtreecommitdiffstats
path: root/chrome/installer/setup
diff options
context:
space:
mode:
authorNico Weber <thakis@chromium.org>2015-07-16 14:25:57 -0700
committerNico Weber <thakis@chromium.org>2015-07-16 21:26:20 +0000
commit408c8d5ebeb5afb83cef000c9d16788323cfc7e7 (patch)
treea5c849157d5ac19fc72f63d2cd2726096141df88 /chrome/installer/setup
parentac1394f2a6ab863120fc7fa09da4ff8512e83cd7 (diff)
downloadchromium_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.cc19
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;