From 7d25709a7dc5920f7bceda81947d288125fac42c Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Fri, 24 Jul 2015 14:42:46 +0300
Subject: [PATCH 1/7] Don't wrongly style tabs with identical openerTabId

---
 background.js | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/background.js b/background.js
index 889c715f..161b4221 100644
--- a/background.js
+++ b/background.js
@@ -1,3 +1,11 @@
+var frameIdMessageable;
+try {
+	chrome.tabs.sendMessage(0, {}, {frameId: 0}, function() {
+		var clearError = chrome.runtime.lastError;
+		frameIdMessageable = true;
+	});
+} catch(e) {}
+
 // This happens right away, sometimes so fast that the content script isn't even ready. That's
 // why the content script also asks for this stuff.
 chrome.webNavigation.onCommitted.addListener(webNavigationListener.bind(this, "styleApply"));
@@ -7,15 +15,18 @@ function webNavigationListener(method, data) {
 	// Until Chrome 41, we can't target a frame with a message
 	// (https://developer.chrome.com/extensions/tabs#method-sendMessage)
 	// so a style affecting a page with an iframe will affect the main page as well.
-	// Skip doing this for frames for now, which can result in flicker.
-	if (data.frameId != 0) {
+	// Skip doing this for frames in pre-41 to prevent page flicker.
+	if (data.frameId != 0 && !frameIdMessageable) {
 		return;
 	}
 	getStyles({matchUrl: data.url, enabled: true, asHash: true}, function(styleHash) {
 		if (method) {
-			chrome.tabs.sendMessage(data.tabId, {method: method, styles: styleHash});
+			chrome.tabs.sendMessage(data.tabId, {method: method, styles: styleHash},
+				frameIdMessageable ? {frameId: data.frameId} : undefined);
+		}
+		if (data.frameId == 0) {
+			updateIcon({id: data.tabId, url: data.url}, styleHash);
 		}
-		updateIcon({id: data.tabId, url: data.url}, styleHash);
 	});
 }
 

From a3401b057209dc9f02687592c27b0a5458558b3e Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Fri, 24 Jul 2015 14:36:43 +0300
Subject: [PATCH 2/7] Simplify getActiveTabRealURL: main frame is always 0

---
 messaging.js | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/messaging.js b/messaging.js
index cfe9fbd8..ea30055b 100644
--- a/messaging.js
+++ b/messaging.js
@@ -76,14 +76,9 @@ function getActiveTabRealURL(callback) {
 function getTabRealURL(tab, callback) {
 	if (tab.url != "chrome://newtab/") {
 		callback(tab.url);
-		return;
-	}
-	chrome.webNavigation.getAllFrames({tabId: tab.id}, function(frames) {
-		frames.some(function(frame) {
-			if (frame.parentFrameId == -1) { // parentless frame is the main frame
-				callback(frame.url);
-				return true;
-			}
+	} else {
+		chrome.webNavigation.getFrame({tabId: tab.id, frameId: 0, processId: -1}, function(frame) {
+			frame && callback(frame.url);
 		});
-	});
+	}
 }

From ae8683873a59d172dc96502f20b1955e6f7deff5 Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Sun, 26 Jul 2015 05:24:18 +0300
Subject: [PATCH 3/7] fixes for Chrome 31

---
 background.js   | 3 ++-
 edit.js         | 3 +++
 localization.js | 9 ++++++---
 storage.js      | 2 +-
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/background.js b/background.js
index 161b4221..655c52da 100644
--- a/background.js
+++ b/background.js
@@ -76,7 +76,7 @@ chrome.commands.onCommand.addListener(function(command) {
 
 // contextMenus API is present in ancient Chrome but it throws an exception
 // upon encountering the unsupported parameter value "browser_action", so we have to catch it.
-
+try {
 chrome.contextMenus.create({
 	id: "show-badge", title: chrome.i18n.getMessage("menuShowBadge"),
 	type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("show-badge")
@@ -85,6 +85,7 @@ chrome.contextMenus.create({
 	id: "disableAll", title: chrome.i18n.getMessage("disableAllStyles"),
 	type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("disableAll")
 }, function() { var clearError = chrome.runtime.lastError; });
+} catch(e) {}
 chrome.contextMenus.onClicked.addListener(function(info, tab) {
 	if (info.menuItemId == "disableAll") {
 		disableAllStylesToggle(info.checked);
diff --git a/edit.js b/edit.js
index 8e4b176e..d3c76588 100644
--- a/edit.js
+++ b/edit.js
@@ -58,6 +58,9 @@ var jumpToLineTemplate = t('editGotoLine') + ': <input class="CodeMirror-jump-fi
 	NodeList.prototype[method]= Array.prototype[method];
 });
 
+// Chrome pre-34
+Element.prototype.matches = Element.prototype.matches || Element.prototype.webkitMatchesSelector;
+
 // reroute handling to nearest editor when keypress resolves to one of these commands
 var commandsToReroute = {
 	save: true, jumpToLine: true, nextEditor: true, prevEditor: true,
diff --git a/localization.js b/localization.js
index 18fd9059..e758fdc2 100644
--- a/localization.js
+++ b/localization.js
@@ -35,13 +35,14 @@ function tNodeList(nodes) {
 		if (node.nodeType != 1) { // not an ELEMENT_NODE
 			continue;
 		}
-		for (var a = 0; a < node.attributes.length; a++) {
-			var name = node.attributes[a].nodeName;
+		for (var a = node.attributes.length - 1; a >= 0; a--) {
+			var attr = node.attributes[a];
+			var name = attr.nodeName;
 			if (name.indexOf("i18n-") != 0) {
 				continue;
 			}
 			name = name.substr(5); // "i18n-".length
-			var value = t(node.attributes[a].nodeValue);
+			var value = t(attr.nodeValue);
 			switch (name) {
 				case "text":
 					node.insertBefore(document.createTextNode(value), node.firstChild);
@@ -52,6 +53,7 @@ function tNodeList(nodes) {
 				default:
 					node.setAttribute(name, value);
 			}
+			node.removeAttributeNode(attr);
 		}
 	}
 }
@@ -69,5 +71,6 @@ function tDocLoader() {
 	observer.observe(document, {subtree: true, childList: true});
 	document.addEventListener("DOMContentLoaded", function() {
 		observer.disconnect();
+		tNodeList(document.querySelectorAll("*"));
 	});
 }
diff --git a/storage.js b/storage.js
index 44376022..62332718 100644
--- a/storage.js
+++ b/storage.js
@@ -271,7 +271,7 @@ function shallowCopy(obj) {
 }
 
 function equal(a, b) {
-	if (!a || !b || typeof a != "object") {
+	if (!a || !b || typeof a != "object" || typeof b != "object") {
 		return a === b;
 	}
 	if (Object.keys(a).length != Object.keys(b).length) {

From bbddba95328fc8caf60c4b459bf8b7a21f5685e0 Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Sat, 1 Aug 2015 15:06:47 +0300
Subject: [PATCH 4/7] Isolate try-catch in background script

---
 background.js | 41 ++++++++++++++++++++++++-----------------
 1 file changed, 24 insertions(+), 17 deletions(-)

diff --git a/background.js b/background.js
index 655c52da..f20248bf 100644
--- a/background.js
+++ b/background.js
@@ -1,10 +1,10 @@
 var frameIdMessageable;
-try {
+runTryCatch(function() {
 	chrome.tabs.sendMessage(0, {}, {frameId: 0}, function() {
 		var clearError = chrome.runtime.lastError;
 		frameIdMessageable = true;
 	});
-} catch(e) {}
+});
 
 // This happens right away, sometimes so fast that the content script isn't even ready. That's
 // why the content script also asks for this stuff.
@@ -76,16 +76,17 @@ chrome.commands.onCommand.addListener(function(command) {
 
 // contextMenus API is present in ancient Chrome but it throws an exception
 // upon encountering the unsupported parameter value "browser_action", so we have to catch it.
-try {
-chrome.contextMenus.create({
-	id: "show-badge", title: chrome.i18n.getMessage("menuShowBadge"),
-	type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("show-badge")
-}, function() { var clearError = chrome.runtime.lastError; });
-chrome.contextMenus.create({
-	id: "disableAll", title: chrome.i18n.getMessage("disableAllStyles"),
-	type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("disableAll")
-}, function() { var clearError = chrome.runtime.lastError; });
-} catch(e) {}
+runTryCatch(function() {
+	chrome.contextMenus.create({
+		id: "show-badge", title: chrome.i18n.getMessage("menuShowBadge"),
+		type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("show-badge")
+	}, function() { var clearError = chrome.runtime.lastError });
+	chrome.contextMenus.create({
+		id: "disableAll", title: chrome.i18n.getMessage("disableAllStyles"),
+		type: "checkbox", contexts: ["browser_action"], checked: prefs.getPref("disableAll")
+	}, function() { var clearError = chrome.runtime.lastError });
+});
+
 chrome.contextMenus.onClicked.addListener(function(info, tab) {
 	if (info.menuItemId == "disableAll") {
 		disableAllStylesToggle(info.checked);
@@ -251,13 +252,12 @@ function sectionAppliesToUrl(section, url) {
 		if (regexp[regexp.length - 1] != "$") {
 			regexp += "$";
 		}
-		try {
-			var re = new RegExp(regexp);
-		} catch (ex) {
+		var re = runTryCatch(function() { return new RegExp(regexp) });
+		if (re) {
+			return (re).test(url);
+		} else {
 			console.log(section.id + "'s regexp '" + regexp + "' is not valid");
-			return false;
 		}
-		return (re).test(url);
 	})) {
 		//console.log(section.id + " applies to " + url + " due to regexp rules");
 		return true;
@@ -403,6 +403,13 @@ function openURL(options) {
 	});
 }
 
+// js engine can't optimize the entire function if it contains try-catch
+// so we should keep it isolated from normal code in a minimal wrapper
+function runTryCatch(func) {
+	try { return func() }
+	catch(e) {}
+}
+
 var codeMirrorThemes;
 getCodeMirrorThemes(function(themes) {
 	 codeMirrorThemes = themes;

From 6d5637e69abd094f21159a1a239da0a9a62ea4a2 Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Wed, 29 Jul 2015 20:45:54 +0300
Subject: [PATCH 5/7] Editor: faster page load

---
 edit.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/edit.js b/edit.js
index d3c76588..553d3b29 100644
--- a/edit.js
+++ b/edit.js
@@ -952,8 +952,8 @@ function initWithStyle(style) {
 	});
 	var queue = style.sections.length ? style.sections : [{code: ""}];
 	var queueStart = new Date().getTime();
-	// after 200ms the sections will be added asynchronously
-	while (new Date().getTime() - queueStart <= 200 && queue.length) {
+	// after 100ms the sections will be added asynchronously
+	while (new Date().getTime() - queueStart <= 100 && queue.length) {
 		maximizeCodeHeight(addSection(null, queue.shift()), !queue.length);
 	}
 	if (queue.length) {

From e1b8b48980f05e517bc31da86c9e429e67486472 Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Sat, 1 Aug 2015 18:16:40 +0300
Subject: [PATCH 6/7] Editor: only attach hotkey rerouter when leaving codebox

---
 edit.js | 55 +++++++++++++++++++++++++++++++------------------------
 1 file changed, 31 insertions(+), 24 deletions(-)

diff --git a/edit.js b/edit.js
index 553d3b29..707c4a14 100644
--- a/edit.js
+++ b/edit.js
@@ -62,9 +62,30 @@ var jumpToLineTemplate = t('editGotoLine') + ': <input class="CodeMirror-jump-fi
 Element.prototype.matches = Element.prototype.matches || Element.prototype.webkitMatchesSelector;
 
 // reroute handling to nearest editor when keypress resolves to one of these commands
-var commandsToReroute = {
-	save: true, jumpToLine: true, nextEditor: true, prevEditor: true,
-	find: true, findNext: true, findPrev: true, replace: true, replaceAll: true
+var hotkeyRerouter = {
+	commands: {
+		save: true, jumpToLine: true, nextEditor: true, prevEditor: true,
+		find: true, findNext: true, findPrev: true, replace: true, replaceAll: true
+	},
+	setState: function(enable) {
+		setTimeout(function() {
+			document[(enable ? "add" : "remove") + "EventListener"]("keydown", hotkeyRerouter.eventHandler);
+		}, 0);
+	},
+	eventHandler: function(event) {
+		var keyName = CodeMirror.keyName(event);
+		if ("handled" == CodeMirror.lookupKey(keyName, CodeMirror.getOption("keyMap"), handleCommand)
+		 || "handled" == CodeMirror.lookupKey(keyName, CodeMirror.defaults.extraKeys, handleCommand)) {
+			event.preventDefault();
+			event.stopPropagation();
+		}
+		function handleCommand(command) {
+			if (hotkeyRerouter.commands[command] === true) {
+				CodeMirror.commands[command](getEditorInSight(event.target));
+				return true;
+			}
+		}
+	}
 };
 
 function onChange(event) {
@@ -264,6 +285,8 @@ function initCodeMirror() {
 		document.getElementById("options").addEventListener("change", acmeEventListener, false);
 		loadPrefs(controlPrefs);
 	});
+
+	hotkeyRerouter.setState(true);
 }
 initCodeMirror();
 
@@ -316,7 +339,11 @@ function setupCodeMirror(textarea, index) {
 	var cm = CodeMirror.fromTextArea(textarea);
 
 	cm.on("change", indicateCodeChange);
-	cm.on("blur", function(cm) { editors.lastActive = cm });
+	cm.on("blur", function(cm) {
+		editors.lastActive = cm;
+		hotkeyRerouter.setState(true);
+	});
+	cm.on("focus", hotkeyRerouter.setState.bind(null, false));
 
 	var resizeGrip = cm.display.wrapper.appendChild(document.createElement("div"));
 	resizeGrip.className = "resize-grip";
@@ -374,26 +401,6 @@ function getCodeMirrorForSection(section) {
 	return null;
 }
 
-// prevent the browser from seeing hotkeys that should be handled by nearest editor
-document.addEventListener("keydown", function(event) {
-	if (event.target.localName == "textarea") {
-		return; // let CodeMirror handle it
-	}
-	var keyName = CodeMirror.keyName(event);
-	if ("handled" == CodeMirror.lookupKey(keyName, CodeMirror.getOption("keyMap"), handleCommand)
-	 || "handled" == CodeMirror.lookupKey(keyName, CodeMirror.defaults.extraKeys, handleCommand)) {
-		event.preventDefault();
-		event.stopPropagation();
-	}
-
-	function handleCommand(command) {
-		if (commandsToReroute[command] === true) {
-			CodeMirror.commands[command](getEditorInSight(event.target));
-			return true;
-		}
-	}
-});
-
 // remind Chrome to repaint a previously invisible editor box by toggling any element's transform
 // this bug is present in some versions of Chrome (v37-40 or something)
 document.addEventListener("scroll", function(event) {

From 1ad92001c00e3988dbe6adca39d8977ee908ae6e Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Sun, 2 Aug 2015 18:11:15 +0300
Subject: [PATCH 7/7] Editor: less obtrusive CSSLint reporting in header, wait
 ~5s

* Keep the old timeout after page init ~500ms
* Immediately show the report in onbeforeunload and save
---
 edit.js | 83 ++++++++++++++++++++++++++++++++++++++++-----------------
 1 file changed, 58 insertions(+), 25 deletions(-)

diff --git a/edit.js b/edit.js
index 707c4a14..d1cf8b61 100644
--- a/edit.js
+++ b/edit.js
@@ -461,7 +461,11 @@ window.onbeforeunload = function() {
 		});
 	}
 	document.activeElement.blur();
-	return !isCleanGlobal() ? t('styleChangesNotSaved') : null;
+	if (isCleanGlobal()) {
+		return;
+	}
+	updateLintReport(null, 0);
+	return confirm(t('styleChangesNotSaved'));
 }
 
 function addAppliesTo(list, name, value) {
@@ -763,24 +767,47 @@ function getEditorInSight(nearbyElement) {
 	}
 }
 
-function updateLintReport(cm) {
+function updateLintReport(cm, delay) {
+	if (delay == 0) {
+		// immediately show pending csslint messages in onbeforeunload and save
+		update.call(cm);
+		return;
+	}
+	if (delay > 0) {
+		// give csslint some time to find the issues, e.g. 500 (1/10 of our default 5s)
+		// by settings its internal delay to 1ms and restoring it back later
+		var lintOpt = editors[0].state.lint.options;
+		setTimeout((function(opt, delay) {
+			opt.delay = delay;
+			update(this);
+		}).bind(cm, lintOpt, lintOpt.delay), delay);
+		lintOpt.delay = 1;
+		return;
+	}
 	var state = cm.state.lint;
 	clearTimeout(state.reportTimeout);
-	state.reportTimeout = setTimeout(update.bind(cm), (state.options.delay || 500) + 500);
+	state.reportTimeout = setTimeout(update.bind(cm), (state.options.delay || 500) + 4500);
 	function update() { // this == cm
-		var html = this.state.lint.marked.length == 0 ? "" : "<tbody>" +
-			this.state.lint.marked.map(function(mark) {
-				var info = mark.__annotation;
-				return "<tr class='" + info.severity + "'>" +
-					"<td role='severity' class='CodeMirror-lint-marker-" + info.severity + "'>" +
-						info.severity + "</td>" +
-					"<td role='line'>" + (info.from.line+1) + "</td>" +
-					"<td role='sep'>:</td>" +
-					"<td role='col'>" + (info.from.ch+1) + "</td>" +
-					"<td role='message'>" + info.message.replace(/ at line \d.+$/, "") + "</td></tr>";
-			}).join("") + "</tbody>";
-		if (this.state.lint.html != html) {
-			this.state.lint.html = html;
+		var scope = this ? [this] : editors;
+		var changed = false;
+		scope.forEach(function(cm) {
+			var html = cm.state.lint.marked.length == 0 ? "" : "<tbody>" +
+				cm.state.lint.marked.map(function(mark) {
+					var info = mark.__annotation;
+					return "<tr class='" + info.severity + "'>" +
+						"<td role='severity' class='CodeMirror-lint-marker-" + info.severity + "'>" +
+							info.severity + "</td>" +
+						"<td role='line'>" + (info.from.line+1) + "</td>" +
+						"<td role='sep'>:</td>" +
+						"<td role='col'>" + (info.from.ch+1) + "</td>" +
+						"<td role='message'>" + info.message.replace(/ at line \d.+$/, "") + "</td></tr>";
+				}).join("") + "</tbody>";
+			if (cm.state.lint.html != html) {
+				cm.state.lint.html = html;
+				changed = true;
+			}
+		});
+		if (changed) {
 			renderLintReport(true);
 		}
 	}
@@ -961,17 +988,21 @@ function initWithStyle(style) {
 	var queueStart = new Date().getTime();
 	// after 100ms the sections will be added asynchronously
 	while (new Date().getTime() - queueStart <= 100 && queue.length) {
-		maximizeCodeHeight(addSection(null, queue.shift()), !queue.length);
-	}
-	if (queue.length) {
-		setTimeout(function processQueue() {
-			maximizeCodeHeight(addSection(null, queue.shift()), !queue.length);
-			if (queue.length) {
-				setTimeout(processQueue, 0);
-			}
-		}, 0);
+		add();
 	}
+	(function processQueue() {
+		if (queue.length) {
+			add();
+			setTimeout(processQueue, 0);
+		}
+	})();
 	initHooks();
+
+	function add() {
+		var sectionDiv = addSection(null, queue.shift());
+		maximizeCodeHeight(sectionDiv, !queue.length);
+		updateLintReport(getCodeMirrorForSection(sectionDiv), 500);
+	}
 }
 
 function initHooks() {
@@ -1081,6 +1112,8 @@ function validate() {
 }
 
 function save() {
+	updateLintReport(null, 0);
+
 	// save the contents of the CodeMirror editors back into the textareas
 	for (var i=0; i < editors.length; i++) {
 		editors[i].save();