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

Unified Diff: chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc

Issue 1109563003: Implement remaining chrome.searchEnginesPrivate methods. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address last comment sync fix trybot failure Created 5 years, 8 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: chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc
diff --git a/chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc b/chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc
index e03a60b7f73b78415ee4d8040dcf71737f68a1a0..07a959e59c27f46cb7b6a6456daa67b173191c4d 100644
--- a/chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc
+++ b/chrome/browser/extensions/api/search_engines_private/search_engines_private_apitest.cc
@@ -41,15 +41,25 @@ class SearchEnginesPrivateApiTest : public ExtensionApiTest {
} // namespace
-IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest,
- DISABLED_SetSelectedSearchEngine) {
+IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest, SetSelectedSearchEngine) {
EXPECT_TRUE(RunSearchEnginesSubtest("setSelectedSearchEngine")) << message_;
}
-IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest,
- DISABLED_OnDefaultSearchEnginesChanged) {
+IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest, OnSearchEnginesChanged) {
EXPECT_TRUE(RunSearchEnginesSubtest("onDefaultSearchEnginesChanged"))
<< message_;
}
+IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest, AddNewSearchEngine) {
+ EXPECT_TRUE(RunSearchEnginesSubtest("addNewSearchEngine")) << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest, UpdateSearchEngine) {
+ EXPECT_TRUE(RunSearchEnginesSubtest("updateSearchEngine")) << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(SearchEnginesPrivateApiTest, RemoveSearchEngine) {
+ EXPECT_TRUE(RunSearchEnginesSubtest("removeSearchEngine")) << message_;
+}
+
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698