Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: html/static/js/IESettings.js

Issue 6353703170736128: Issue 1112 - Properly use polyfills in firstrun and settings pages (Closed)
Patch Set: Created July 23, 2014, 12:33 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | html/static/js/firstRun.js » ('j') | html/static/js/settings.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: html/static/js/IESettings.js
===================================================================
--- a/html/static/js/IESettings.js
+++ b/html/static/js/IESettings.js
@@ -53,7 +53,7 @@
if (aaCheckbox)
{
aaCheckbox.checked = UserSettings().IsAcceptableAdsEnabled();
- addListener(aaCheckbox, "change", acceptableAdsChange, false);
+ aaCheckbox.addEventListener("change", acceptableAdsChange, false);
}
var optionsLanguage = document.getElementById("language");
@@ -68,7 +68,7 @@
optionsLanguage.add(el, 0);
}
- addListener(optionsLanguage, "change", function ()
+ optionsLanguage.addEventListener("change", function ()
{
UserSettings().SetLanguage(optionsLanguage[optionsLanguage.selectedIndex].value);
}, false);
@@ -102,7 +102,7 @@
}
var addDomain = document.getElementById("addDomain");
- addListener(addDomain, "click", function()
+ addDomain.addEventListener("click", function()
{
var newDomain = document.getElementById("newDomain");
@@ -139,7 +139,7 @@
}, false);
var removeDomains = document.getElementById("removeDomains");
- addListener(removeDomains, "click", function()
+ removeDomains.addEventListener("click", function()
{
var removeOptions = [];
var domains = optionsDomain.options;
« no previous file with comments | « no previous file | html/static/js/firstRun.js » ('j') | html/static/js/settings.js » ('J')

Powered by Google App Engine
This is Rietveld