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

Unified Diff: components/variations/variations_http_header_provider_unittest.cc

Issue 421663003: Move variations component code to variations namespace. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fix Created 6 years, 5 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
Index: components/variations/variations_http_header_provider_unittest.cc
===================================================================
--- components/variations/variations_http_header_provider_unittest.cc (revision 285608)
+++ components/variations/variations_http_header_provider_unittest.cc (working copy)
@@ -17,12 +17,6 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "url/gurl.h"
-// TODO(mathp): Remove once everything is under namespace 'variations'.
-using chrome_variations::GOOGLE_WEB_PROPERTIES;
-using chrome_variations::GOOGLE_WEB_PROPERTIES_TRIGGER;
-using chrome_variations::IDCollectionKey;
-using chrome_variations::VariationID;
-
namespace variations {
namespace {
@@ -44,13 +38,14 @@
}
scoped_refptr<base::FieldTrial> CreateTrialAndAssociateId(
- const std::string& trial_name, const std::string& default_group_name,
- IDCollectionKey key, VariationID id) {
+ const std::string& trial_name,
+ const std::string& default_group_name,
+ IDCollectionKey key,
+ VariationID id) {
scoped_refptr<base::FieldTrial> trial(
base::FieldTrialList::CreateFieldTrial(trial_name, default_group_name));
- chrome_variations::AssociateGoogleVariationID(key, trial->trial_name(),
- trial->group_name(), id);
+ AssociateGoogleVariationID(key, trial->trial_name(), trial->group_name(), id);
return trial;
}
@@ -64,7 +59,7 @@
virtual ~VariationsHttpHeaderProviderTest() {}
virtual void TearDown() OVERRIDE {
- chrome_variations::testing::ClearAllVariationIDs();
+ testing::ClearAllVariationIDs();
}
};
@@ -156,7 +151,6 @@
{"http://ssl.gstatic.com", true},
{"http://a.b.gstatic.com", true},
{"http://a.b.ytimg.com", true},
-
};
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) {
« no previous file with comments | « components/variations/variations_http_header_provider.cc ('k') | components/variations/variations_seed_processor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698