diff options
| author | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-23 23:37:00 +0000 |
|---|---|---|
| committer | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-23 23:37:00 +0000 |
| commit | 140905c70becf18808f2ad9d55d6efc00cb96461 (patch) | |
| tree | c8a71d1149012d01bfd91c59740704d164d82b81 /components | |
| parent | 8f3cf5f5413e223a40fddc111759536df1400893 (diff) | |
| download | chromium_src-140905c70becf18808f2ad9d55d6efc00cb96461.zip chromium_src-140905c70becf18808f2ad9d55d6efc00cb96461.tar.gz chromium_src-140905c70becf18808f2ad9d55d6efc00cb96461.tar.bz2 | |
Update all users of base::Version to explicitly specify the namespace, and clean up the header.
BUG=none
TEST=no change
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/100543005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242414 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'components')
| -rw-r--r-- | components/variations/processed_study.cc | 4 | ||||
| -rw-r--r-- | components/variations/variations_seed_processor_unittest.cc | 20 |
2 files changed, 10 insertions, 14 deletions
diff --git a/components/variations/processed_study.cc b/components/variations/processed_study.cc index 9d0e4f1..a34f894 100644 --- a/components/variations/processed_study.cc +++ b/components/variations/processed_study.cc @@ -23,13 +23,13 @@ bool ValidateStudyAndComputeTotalProbability( return false; } if (study.filter().has_min_version() && - !Version::IsValidWildcardString(study.filter().min_version())) { + !base::Version::IsValidWildcardString(study.filter().min_version())) { DVLOG(1) << study.name() << " has invalid min version: " << study.filter().min_version(); return false; } if (study.filter().has_max_version() && - !Version::IsValidWildcardString(study.filter().max_version())) { + !base::Version::IsValidWildcardString(study.filter().max_version())) { DVLOG(1) << study.name() << " has invalid max version: " << study.filter().max_version(); return false; diff --git a/components/variations/variations_seed_processor_unittest.cc b/components/variations/variations_seed_processor_unittest.cc index cbf0864..e03fcd9 100644 --- a/components/variations/variations_seed_processor_unittest.cc +++ b/components/variations/variations_seed_processor_unittest.cc @@ -399,9 +399,8 @@ TEST_F(VariationsSeedProcessorTest, CheckStudyVersion) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(min_test_cases); ++i) { filter.set_min_version(min_test_cases[i].min_version); - const bool result = - seed_processor.CheckStudyVersion(filter, - Version(min_test_cases[i].version)); + const bool result = seed_processor.CheckStudyVersion( + filter, base::Version(min_test_cases[i].version)); EXPECT_EQ(min_test_cases[i].expected_result, result) << "Min. version case " << i << " failed!"; } @@ -409,9 +408,8 @@ TEST_F(VariationsSeedProcessorTest, CheckStudyVersion) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(max_test_cases); ++i) { filter.set_max_version(max_test_cases[i].max_version); - const bool result = - seed_processor.CheckStudyVersion(filter, - Version(max_test_cases[i].version)); + const bool result = seed_processor.CheckStudyVersion( + filter, base::Version(max_test_cases[i].version)); EXPECT_EQ(max_test_cases[i].expected_result, result) << "Max version case " << i << " failed!"; } @@ -423,16 +421,14 @@ TEST_F(VariationsSeedProcessorTest, CheckStudyVersion) { filter.set_max_version(max_test_cases[j].max_version); if (!min_test_cases[i].expected_result) { - const bool result = - seed_processor.CheckStudyVersion( - filter, Version(min_test_cases[i].version)); + const bool result = seed_processor.CheckStudyVersion( + filter, base::Version(min_test_cases[i].version)); EXPECT_FALSE(result) << "Case " << i << "," << j << " failed!"; } if (!max_test_cases[j].expected_result) { - const bool result = - seed_processor.CheckStudyVersion( - filter, Version(max_test_cases[j].version)); + const bool result = seed_processor.CheckStudyVersion( + filter, base::Version(max_test_cases[j].version)); EXPECT_FALSE(result) << "Case " << i << "," << j << " failed!"; } } |