Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit c67b172

Browse files
committed
chore($sniffer): Remove remnants of separate documentMode handling
Since msie is now set to document.documentMode, it's not necessary to keep the documentMode in a separate property.
1 parent 74a214c commit c67b172

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/ng/sce.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -731,7 +731,7 @@ function $SceProvider() {
731731
$parse, $sniffer, $sceDelegate) {
732732
// Prereq: Ensure that we're not running in IE8 quirks mode. In that mode, IE allows
733733
// the "expression(javascript expression)" syntax which is insecure.
734-
if (enabled && $sniffer.msie && $sniffer.msieDocumentMode < 8) {
734+
if (enabled && $sniffer.msie < 8) {
735735
throw $sceMinErr('iequirks',
736736
'Strict Contextual Escaping does not support Internet Explorer version < 9 in quirks ' +
737737
'mode. You can fix this by adding the text <!doctype html> to the top of your HTML ' +

src/ng/sniffer.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ function $SnifferProvider() {
2222
int((/android (\d+)/.exec(lowercase(($window.navigator || {}).userAgent)) || [])[1]),
2323
boxee = /Boxee/i.test(($window.navigator || {}).userAgent),
2424
document = $document[0] || {},
25-
documentMode = document.documentMode,
2625
vendorPrefix,
2726
vendorRegex = /^(Moz|webkit|O|ms)(?=[A-Z])/,
2827
bodyStyle = document.body && document.body.style,
@@ -86,8 +85,7 @@ function $SnifferProvider() {
8685
transitions : transitions,
8786
animations : animations,
8887
android: android,
89-
msie : msie,
90-
msieDocumentMode: documentMode
88+
msie : msie
9189
};
9290
}];
9391
}

0 commit comments

Comments
 (0)