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

Unified Diff: chrome/test/data/webui/settings/search_engines_page_test.js

Issue 1836583003: MD Settings: Use paper-dropdown-menu for search engines. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Change paper-item to div Created 4 years, 9 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/test/data/webui/settings/search_engines_page_test.js
diff --git a/chrome/test/data/webui/settings/search_engines_page_test.js b/chrome/test/data/webui/settings/search_engines_page_test.js
index c1aa03ce7d55b140f9ff7ebe5d3724517bed13b4..a96ecee17301b6f95ca1d76c4f15fbf2e88eebf2 100644
--- a/chrome/test/data/webui/settings/search_engines_page_test.js
+++ b/chrome/test/data/webui/settings/search_engines_page_test.js
@@ -4,91 +4,6 @@
cr.define('settings_search_engines_page', function() {
/**
- * A test version of SearchEnginesBrowserProxy. Provides helper methods
- * for allowing tests to know when a method was called, as well as
- * specifying mock responses.
- *
- * @constructor
- * @implements {settings.SearchEnginesBrowserProxy}
- * @extends {settings.TestBrowserProxy}
- */
- var TestSearchEnginesBrowserProxy = function() {
- settings.TestBrowserProxy.call(this, [
- 'getSearchEnginesList',
- 'removeSearchEngine',
- 'searchEngineEditCancelled',
- 'searchEngineEditCompleted',
- 'searchEngineEditStarted',
- 'setDefaultSearchEngine',
- 'validateSearchEngineInput',
- 'manageExtension',
- 'disableExtension',
- ]);
-
- /** @private {!SearchEnginesInfo} */
- this.searchEnginesInfo_ = {defaults: [], others: [], extensions: []};
- };
-
- TestSearchEnginesBrowserProxy.prototype = {
- __proto__: settings.TestBrowserProxy.prototype,
-
- /** @override */
- setDefaultSearchEngine: function(modelIndex) {
- this.methodCalled('setDefaultSearchEngine', modelIndex);
- },
-
- /** @override */
- removeSearchEngine: function(modelIndex) {
- this.methodCalled('removeSearchEngine', modelIndex);
- },
-
- /** @override */
- searchEngineEditStarted: function(modelIndex) {
- this.methodCalled('searchEngineEditStarted', modelIndex);
- },
-
- /** @override */
- searchEngineEditCancelled: function() {
- this.methodCalled('searchEngineEditCancelled');
- },
-
- /** @override */
- searchEngineEditCompleted: function(searchEngine, keyword, queryUrl) {
- this.methodCalled('searchEngineEditCompleted');
- },
-
- /**
- * Sets the response to be returned by |getSearchEnginesList|.
- * @param {!SearchEnginesInfo}
- */
- setSearchEnginesInfo: function(searchEnginesInfo) {
- this.searchEnginesInfo_ = searchEnginesInfo;
- },
-
- /** @override */
- getSearchEnginesList: function() {
- this.methodCalled('getSearchEnginesList');
- return Promise.resolve(this.searchEnginesInfo_);
- },
-
- /** @override */
- validateSearchEngineInput: function(fieldName, fieldValue) {
- this.methodCalled('validateSearchEngineInput');
- return Promise.resolve(true);
- },
-
- /** @override */
- manageExtension: function(extensionId) {
- this.methodCalled('manageExtension', extensionId);
- },
-
- /** @override */
- disableExtension: function(extensionId) {
- this.methodCalled('disableExtension', extensionId);
- },
- };
-
- /**
* @param {boolean} canBeDefault
* @param {boolean} canBeEdited
* @param {boolean} canBeRemoved
@@ -141,7 +56,7 @@ cr.define('settings_search_engines_page', function() {
var browserProxy = null;
setup(function() {
- browserProxy = new TestSearchEnginesBrowserProxy();
+ browserProxy = new settings_search.TestSearchEnginesBrowserProxy();
settings.SearchEnginesBrowserProxyImpl.instance_ = browserProxy;
PolymerTest.clearBody();
dialog = document.createElement('settings-search-engine-dialog');
@@ -231,7 +146,7 @@ cr.define('settings_search_engines_page', function() {
var browserProxy = null;
setup(function() {
- browserProxy = new TestSearchEnginesBrowserProxy();
+ browserProxy = new settings_search.TestSearchEnginesBrowserProxy();
settings.SearchEnginesBrowserProxyImpl.instance_ = browserProxy;
PolymerTest.clearBody();
entry = document.createElement('settings-search-engine-entry');
@@ -328,7 +243,7 @@ cr.define('settings_search_engines_page', function() {
};
setup(function() {
- browserProxy = new TestSearchEnginesBrowserProxy();
+ browserProxy = new settings_search.TestSearchEnginesBrowserProxy();
browserProxy.setSearchEnginesInfo(searchEnginesInfo);
settings.SearchEnginesBrowserProxyImpl.instance_ = browserProxy;
PolymerTest.clearBody();
@@ -388,7 +303,7 @@ cr.define('settings_search_engines_page', function() {
var browserProxy = null;
setup(function() {
- browserProxy = new TestSearchEnginesBrowserProxy();
+ browserProxy = new settings_search.TestSearchEnginesBrowserProxy();
settings.SearchEnginesBrowserProxyImpl.instance_ = browserProxy;
PolymerTest.clearBody();
entry = document.createElement('settings-omnibox-extension-entry');
@@ -420,11 +335,12 @@ cr.define('settings_search_engines_page', function() {
});
}
-
return {
- registerDialogTests: registerDialogTests,
- registerSearchEngineEntryTests: registerSearchEngineEntryTests,
- registerOmniboxExtensionEntryTests: registerOmniboxExtensionEntryTests,
- registerPageTests: registerPageTests,
+ registerTests: function() {
+ registerDialogTests();
+ registerSearchEngineEntryTests();
+ registerOmniboxExtensionEntryTests();
+ registerPageTests();
+ },
};
});
« no previous file with comments | « chrome/test/data/webui/settings/cr_settings_browsertest.js ('k') | chrome/test/data/webui/settings/search_page_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698