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

Unified Diff: chrome/test/data/webui/cr_elements/cr_toolbar_search_field_tests.js

Issue 2449663002: MD Settings: Implement search URLs. (Closed)
Patch Set: Fix test. Created 4 years, 1 month 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/cr_elements/cr_toolbar_search_field_tests.js
diff --git a/chrome/test/data/webui/cr_elements/cr_toolbar_search_field_tests.js b/chrome/test/data/webui/cr_elements/cr_toolbar_search_field_tests.js
index 48107ee2a51565afec2e32c5afdcafdfae1d31c2..61265aeb8b0eff368b4f7b6f6128c36a3c1bc009 100644
--- a/chrome/test/data/webui/cr_elements/cr_toolbar_search_field_tests.js
+++ b/chrome/test/data/webui/cr_elements/cr_toolbar_search_field_tests.js
@@ -110,6 +110,14 @@ cr.define('cr_toolbar_search_field', function() {
assertEquals(['foo', '', 'bar', 'baz'].join(), searches.join());
});
+ test('does not notify on setValue with noEvent=true', function() {
+ MockInteractions.tap(field);
+ field.setValue('foo', true);
+ field.setValue('bar');
+ field.setValue('baz', true);
+ assertEquals(['bar'].join(), searches.join());
+ });
+
// Tests that calling setValue() from within a 'search-changed' callback
// does not result in an infinite loop.
test('no infinite loop', function() {

Powered by Google App Engine
This is Rietveld 408576698