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

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

Issue 2837703002: Change SettingsLanguagesPageBrowserTest to CrSettingsLanguagesPageTest (Closed)
Patch Set: rebase Created 3 years, 7 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/test_util.js
diff --git a/chrome/test/data/webui/settings/test_util.js b/chrome/test/data/webui/settings/test_util.js
index ea50c397dd552c3c9dd866be3b3db3d89b985477..d6cf3f7b5ca9500fcd759e263b082a7c71446447 100644
--- a/chrome/test/data/webui/settings/test_util.js
+++ b/chrome/test/data/webui/settings/test_util.js
@@ -41,8 +41,29 @@ cr.define('test_util', function() {
});
}
+ /**
+ * Data-binds two Polymer properties using the property-changed events and
+ * set/notifyPath API. Useful for testing components which would normally be
+ * used together.
+ * @param {!HTMLElement} el1
+ * @param {!HTMLElement} el2
+ * @param {string} property
+ */
+ function fakeDataBind(el1, el2, property) {
+ var forwardChange = function(el, event) {
+ if (event.detail.hasOwnProperty('path'))
+ el.notifyPath(event.detail.path, event.detail.value);
+ else
+ el.set(property, event.detail.value);
+ };
+ // Add the listeners symmetrically. Polymer will prevent recursion.
+ el1.addEventListener(property + '-changed', forwardChange.bind(null, el2));
+ el2.addEventListener(property + '-changed', forwardChange.bind(null, el1));
+ }
+
return {
eventToPromise: eventToPromise,
+ fakeDataBind: fakeDataBind,
whenAttributeIs: whenAttributeIs,
};
});
« no previous file with comments | « chrome/test/data/webui/settings/test_languages_browser_proxy.js ('k') | ui/webui/resources/cr_elements/shared_vars_css.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698