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

Unified Diff: lib/windowWrapper.js

Issue 5684127030312960: Issue 1091 - Implement a better templating approach for Element Hiding Helper (Closed)
Patch Set: Switched to a functional approach Created Sept. 11, 2014, 4:50 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 | « lib/main.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/windowWrapper.js
===================================================================
--- a/lib/windowWrapper.js
+++ b/lib/windowWrapper.js
@@ -16,28 +16,26 @@ function getMenuItem()
let stringBundle = Services.strings.createBundle("chrome://elemhidehelper/locale/global.properties?" + Math.random());
let result = [stringBundle.GetStringFromName("selectelement.label"), stringBundle.GetStringFromName("stopselection.label")];
getMenuItem = function() result;
return getMenuItem();
}
exports.WindowWrapper = WindowWrapper;
-function WindowWrapper(wnd, elementMarkerClass)
+function WindowWrapper(wnd)
{
this.window = wnd;
this.popupShowingHandler = this.popupShowingHandler.bind(this);
this.popupHiddenHandler = this.popupHiddenHandler.bind(this);
this.keyPressHandler = this.keyPressHandler.bind(this);
this.toggleSelection = this.toggleSelection.bind(this);
this.hideTooltips = this.hideTooltips.bind(this);
- this.E("ehh-elementmarker").firstElementChild.setAttribute("class", elementMarkerClass);
-
this.init();
}
WindowWrapper.prototype =
{
window: null,
get browser()
{
« no previous file with comments | « lib/main.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld