Commit c6ccca4a authored by Devlin Cronin's avatar Devlin Cronin Committed by Commit Bot

[Extensions Cleanup] Move FeatureSwitch unittest

Move feature_switch_unittest.cc to //extensions to neighbor its
source file (feature_switch.[h|cc]). It has no //chrome dependencies.

Bug: None
Change-Id: Ie98bdd50d973e168e5ab593047fd697deec53bf2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2131626Reviewed-by: default avatarIstiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755415}
parent a7fb2233
...@@ -4761,7 +4761,6 @@ test("unit_tests") { ...@@ -4761,7 +4761,6 @@ test("unit_tests") {
"../common/extensions/chrome_manifest_url_handlers_unittest.cc", "../common/extensions/chrome_manifest_url_handlers_unittest.cc",
"../common/extensions/command_unittest.cc", "../common/extensions/command_unittest.cc",
"../common/extensions/extension_unittest.cc", "../common/extensions/extension_unittest.cc",
"../common/extensions/feature_switch_unittest.cc",
"../common/extensions/manifest_handlers/app_display_mode_manifest_unittest.cc", "../common/extensions/manifest_handlers/app_display_mode_manifest_unittest.cc",
"../common/extensions/manifest_handlers/app_theme_color_manifest_unittest.cc", "../common/extensions/manifest_handlers/app_theme_color_manifest_unittest.cc",
"../common/extensions/manifest_handlers/automation_unittest.cc", "../common/extensions/manifest_handlers/automation_unittest.cc",
......
...@@ -421,6 +421,7 @@ if (enable_extensions) { ...@@ -421,6 +421,7 @@ if (enable_extensions) {
"extension_resource_unittest.cc", "extension_resource_unittest.cc",
"extension_set_unittest.cc", "extension_set_unittest.cc",
"extension_unittest.cc", "extension_unittest.cc",
"feature_switch_unittest.cc",
"features/complex_feature_unittest.cc", "features/complex_feature_unittest.cc",
"features/feature_provider_unittest.cc", "features/feature_provider_unittest.cc",
"features/simple_feature_unittest.cc", "features/simple_feature_unittest.cc",
......
...@@ -13,12 +13,13 @@ namespace { ...@@ -13,12 +13,13 @@ namespace {
const char kSwitchName[] = "test-switch"; const char kSwitchName[] = "test-switch";
template<FeatureSwitch::DefaultValue T> template <FeatureSwitch::DefaultValue T>
class FeatureSwitchTest : public testing::Test { class FeatureSwitchTest : public testing::Test {
public: public:
FeatureSwitchTest() FeatureSwitchTest()
: command_line_(base::CommandLine::NO_PROGRAM), : command_line_(base::CommandLine::NO_PROGRAM),
feature_(&command_line_, kSwitchName, T) {} feature_(&command_line_, kSwitchName, T) {}
protected: protected:
base::CommandLine command_line_; base::CommandLine command_line_;
FeatureSwitch feature_; FeatureSwitch feature_;
......
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