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

Unified Diff: composer.postload.js

Issue 29350213: Issue 4364 - Drop support for Chrome 29-40 and remove legacy code (Closed)
Patch Set: Remove legacy check from composer.postload.js Created Aug. 26, 2016, 12:57 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
Index: composer.postload.js
===================================================================
--- a/composer.postload.js
+++ b/composer.postload.js
@@ -485,11 +485,7 @@
ext.onExtensionUnloaded.removeListener(deactivateBlockElement);
}
-// In Chrome 37-40, the document_end content script (this one) runs properly,
-// while the document_start content scripts (that defines ext) might not. Check
-// whether variable ext exists before continuing to avoid
-// "Uncaught ReferenceError: ext is not defined". See https://crbug.com/416907
-if ("ext" in window && document instanceof HTMLDocument)
+if (document instanceof HTMLDocument)
{
// Use a contextmenu handler to save the last element the user right-clicked
// on. To make things easier, we actually save the DOM event. We have to do
@@ -553,7 +549,7 @@
checkCollapse(currentElement.prisoner || currentElement);
// Apply added element hiding filters.
- updateStylesheet();
+ elemhide.apply();
}
deactivateBlockElement();
break;

Powered by Google App Engine
This is Rietveld