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

Unified Diff: html/static/js/settings.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 | « html/static/js/firstRun.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: html/static/js/settings.js
===================================================================
--- a/html/static/js/settings.js
+++ b/html/static/js/settings.js
@@ -1,23 +1,11 @@
"use strict";
-function addListener(element, eventType, callback, propagate)
-{
- if (element.addEventListener)
- {
- element.addEventListener(eventType, callback, propagate);
- }
- else
- {
- element.attachEvent("on" + eventType, callback);
- }
-}
-
(function()
{
function init()
{
var manageButton = document.getElementById("manageExceptions");
- addListener(manageButton, "click", toggleManage, false);
+ manageButton.addEventListener("click", toggleManage);
Felix Dahlke 2014/07/23 12:45:17 Might keep the false too.
}
function toggleManage(ev)
« no previous file with comments | « html/static/js/firstRun.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld