Skip to content

Commit 4f9eb7f

Browse files
committed
Use strict equal everywhere
s/ == / === /g s/ != / !== /g
1 parent 9e1c31d commit 4f9eb7f

5 files changed

Lines changed: 41 additions & 41 deletions

File tree

lib/CSSImportRule.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -50,31 +50,31 @@ CSSOM.CSSImportRule.prototype.__defineSetter__("cssText", function(cssText) {
5050
case '\r':
5151
case '\n':
5252
case '\f':
53-
if (state == 'after-import') {
53+
if (state === 'after-import') {
5454
state = 'url';
5555
} else {
5656
buffer += character;
5757
}
5858
break;
5959

6060
case '@':
61-
if (!state && cssText.indexOf('@import', i) == i) {
61+
if (!state && cssText.indexOf('@import', i) === i) {
6262
state = 'after-import';
6363
i += 'import'.length;
6464
buffer = '';
6565
}
6666
break;
6767

6868
case 'u':
69-
if (state == 'url' && cssText.indexOf('url(', i) == i) {
69+
if (state === 'url' && cssText.indexOf('url(', i) === i) {
7070
index = cssText.indexOf(')', i + 1);
71-
if (index == -1) {
71+
if (index === -1) {
7272
throw i + ': ")" not found';
7373
}
7474
i += 'url('.length;
7575
var url = cssText.slice(i, index);
7676
if (url[0] === url[url.length - 1]) {
77-
if (url[0] == '"' || url[0] == "'") {
77+
if (url[0] === '"' || url[0] === "'") {
7878
url = url.slice(1, -1);
7979
}
8080
}
@@ -85,7 +85,7 @@ CSSOM.CSSImportRule.prototype.__defineSetter__("cssText", function(cssText) {
8585
break;
8686

8787
case '"':
88-
if (state == 'url') {
88+
if (state === 'url') {
8989
index = cssText.indexOf('"', i + 1);
9090
if (!index) {
9191
throw i + ": '\"' not found";
@@ -97,7 +97,7 @@ CSSOM.CSSImportRule.prototype.__defineSetter__("cssText", function(cssText) {
9797
break;
9898

9999
case "'":
100-
if (state == 'url') {
100+
if (state === 'url') {
101101
index = cssText.indexOf("'", i + 1);
102102
if (!index) {
103103
throw i + ': "\'" not found';
@@ -109,15 +109,15 @@ CSSOM.CSSImportRule.prototype.__defineSetter__("cssText", function(cssText) {
109109
break;
110110

111111
case ';':
112-
if (state == 'media') {
112+
if (state === 'media') {
113113
if (buffer) {
114114
this.media.mediaText = buffer.trim();
115115
}
116116
}
117117
break;
118118

119119
default:
120-
if (state == 'media') {
120+
if (state === 'media') {
121121
buffer += character;
122122
}
123123
break;

lib/CSSStyleRule.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -107,10 +107,10 @@ CSSOM.CSSStyleRule.parse = function(ruleText) {
107107

108108
// Comment
109109
case "/":
110-
if (ruleText.charAt(i + 1) == "*") {
110+
if (ruleText.charAt(i + 1) === "*") {
111111
i += 2;
112112
index = ruleText.indexOf("*/", i);
113-
if (index == -1) {
113+
if (index === -1) {
114114
throw new SyntaxError("Missing */");
115115
} else {
116116
i = index + 1;
@@ -121,15 +121,15 @@ CSSOM.CSSStyleRule.parse = function(ruleText) {
121121
break;
122122

123123
case "{":
124-
if (state == "selector") {
124+
if (state === "selector") {
125125
styleRule.selectorText = buffer.trim();
126126
buffer = "";
127127
state = "name";
128128
}
129129
break;
130130

131131
case ":":
132-
if (state == "name") {
132+
if (state === "name") {
133133
name = buffer.trim();
134134
buffer = "";
135135
state = "value";
@@ -139,7 +139,7 @@ CSSOM.CSSStyleRule.parse = function(ruleText) {
139139
break;
140140

141141
case "!":
142-
if (state == "value" && ruleText.indexOf("!important", i) === i) {
142+
if (state === "value" && ruleText.indexOf("!important", i) === i) {
143143
priority = "important";
144144
i += "important".length;
145145
} else {
@@ -148,7 +148,7 @@ CSSOM.CSSStyleRule.parse = function(ruleText) {
148148
break;
149149

150150
case ";":
151-
if (state == "value") {
151+
if (state === "value") {
152152
styleRule.style.setProperty(name, buffer.trim(), priority);
153153
priority = "";
154154
buffer = "";
@@ -159,11 +159,11 @@ CSSOM.CSSStyleRule.parse = function(ruleText) {
159159
break;
160160

161161
case "}":
162-
if (state == "value") {
162+
if (state === "value") {
163163
styleRule.style.setProperty(name, buffer.trim(), priority);
164164
priority = "";
165165
buffer = "";
166-
} else if (state == "name") {
166+
} else if (state === "name") {
167167
break;
168168
} else {
169169
buffer += character;

lib/MediaList.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ CSSOM.MediaList.prototype = {
3737
* @param {string} medium
3838
*/
3939
appendMedium: function(medium) {
40-
if (Array.prototype.indexOf.call(this, medium) == -1) {
40+
if (Array.prototype.indexOf.call(this, medium) === -1) {
4141
this[this.length] = medium;
4242
this.length++;
4343
}
@@ -48,11 +48,11 @@ CSSOM.MediaList.prototype = {
4848
*/
4949
deleteMedium: function(medium) {
5050
var index = Array.prototype.indexOf.call(this, medium);
51-
if (index != -1) {
51+
if (index !== -1) {
5252
Array.prototype.splice.call(this, index, 1);
5353
}
5454
}
55-
55+
5656
};
5757

5858

lib/parse.js

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -105,69 +105,69 @@ CSSOM.parse = function parse(token, options) {
105105

106106
// Comment
107107
case "/":
108-
if (token.charAt(i + 1) == "*") {
108+
if (token.charAt(i + 1) === "*") {
109109
i += 2;
110110
index = token.indexOf("*/", i);
111-
if (index == -1) {
111+
if (index === -1) {
112112
throw new SyntaxError("Missing */");
113113
} else {
114114
i = index + 1;
115115
}
116116
} else {
117117
buffer += character;
118118
}
119-
if (state == "importRule-begin") {
119+
if (state === "importRule-begin") {
120120
buffer += " ";
121121
state = "importRule";
122122
}
123123
break;
124124

125125
// At-rule
126126
case "@":
127-
if (token.indexOf("@media", i) == i) {
127+
if (token.indexOf("@media", i) === i) {
128128
state = "atBlock";
129129
mediaRule = new CSSOM.CSSMediaRule;
130130
mediaRule.__starts = i;
131131
i += "media".length;
132132
buffer = "";
133133
break;
134-
} else if (token.indexOf("@import", i) == i) {
134+
} else if (token.indexOf("@import", i) === i) {
135135
state = "importRule-begin";
136136
i += "import".length;
137137
buffer += "@import";
138138
break;
139-
} else if (token.indexOf("@-webkit-keyframes", i) == i) {
139+
} else if (token.indexOf("@-webkit-keyframes", i) === i) {
140140
state = "keyframesRule-begin";
141141
keyframesRule = new CSSOM.CSSKeyframesRule;
142142
keyframesRule.__starts = i;
143143
i += "-webkit-keyframes".length;
144144
buffer = "";
145145
break;
146-
} else if (state == "selector") {
146+
} else if (state === "selector") {
147147
state = "atRule";
148148
}
149149
buffer += character;
150150
break;
151151

152152
case "{":
153-
if (state == "selector" || state == "atRule") {
153+
if (state === "selector" || state === "atRule") {
154154
styleRule.selectorText = buffer.trim();
155155
styleRule.style.__starts = i;
156156
buffer = "";
157157
state = "before-name";
158-
} else if (state == "atBlock") {
158+
} else if (state === "atBlock") {
159159
mediaRule.media.mediaText = buffer.trim();
160160
mediaRule.parentRule = currentScope;
161161
currentScope = mediaRule;
162162
buffer = "";
163163
state = "before-selector";
164-
} else if (state == "keyframesRule-begin") {
164+
} else if (state === "keyframesRule-begin") {
165165
keyframesRule.name = buffer.trim();
166166
keyframesRule.parentRule = currentScope;
167167
currentScope = keyframesRule;
168168
buffer = "";
169169
state = "keyframeRule-begin";
170-
} else if (state == "keyframeRule-begin") {
170+
} else if (state === "keyframeRule-begin") {
171171
styleRule = new CSSOM.CSSKeyframeRule;
172172
styleRule.keyText = buffer.trim();
173173
styleRule.__starts = i;
@@ -177,7 +177,7 @@ CSSOM.parse = function parse(token, options) {
177177
break;
178178

179179
case ":":
180-
if (state == "name") {
180+
if (state === "name") {
181181
name = buffer.trim();
182182
buffer = "";
183183
state = "before-value";
@@ -187,9 +187,9 @@ CSSOM.parse = function parse(token, options) {
187187
break;
188188

189189
case '(':
190-
if (state == 'value') {
190+
if (state === 'value') {
191191
index = token.indexOf(')', i + 1);
192-
if (index == -1) {
192+
if (index === -1) {
193193
throw i + ': unclosed "("';
194194
}
195195
buffer += token.slice(i, index + 1);
@@ -200,7 +200,7 @@ CSSOM.parse = function parse(token, options) {
200200
break;
201201

202202
case "!":
203-
if (state == "value" && token.indexOf("!important", i) === i) {
203+
if (state === "value" && token.indexOf("!important", i) === i) {
204204
priority = "important";
205205
i += "important".length;
206206
} else {
@@ -245,7 +245,7 @@ CSSOM.parse = function parse(token, options) {
245245
styleRule.parentRule = currentScope;
246246
currentScope.cssRules.push(styleRule);
247247
buffer = "";
248-
if (currentScope.constructor == CSSOM.CSSKeyframesRule) {
248+
if (currentScope.constructor === CSSOM.CSSKeyframesRule) {
249249
state = "keyframeRule-begin";
250250
} else {
251251
state = "before-selector";

test/helper.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ function cloneOwnProperties(object, stack) {
2222
stack.push(object);
2323
var result = {};
2424
for (var key in object) {
25-
if (key.charAt(0) == "_" || !object.hasOwnProperty(key)) {
25+
if (key.charAt(0) === "_" || !object.hasOwnProperty(key)) {
2626
continue;
2727
}
28-
if (typeof object[key] == "object") {
28+
if (typeof object[key] === "object") {
2929
var stackIndex = stack.indexOf(object[key]);
3030
if (stackIndex > -1) {
3131
result[key] = buildPath(stack.length - stackIndex);
@@ -46,7 +46,7 @@ function cloneOwnProperties(object, stack) {
4646
* @return {string}
4747
*/
4848
function buildPath(level) {
49-
if (level == 0) {
49+
if (level === 0) {
5050
return '.';
5151
} else {
5252
var result = '..';
@@ -79,14 +79,14 @@ function subsetOfOwnProperties(base, another) {
7979
return false;
8080
}
8181

82-
if (typeof base != "object" || typeof another != "object") {
82+
if (typeof base !== "object" || typeof another !== "object") {
8383
return another;
8484
}
8585

8686
var diff = {};
8787
var isDiff = false;
8888
for (var key in another) {
89-
if (key.charAt(0) == "_" || !another.hasOwnProperty(key)) {
89+
if (key.charAt(0) === "_" || !another.hasOwnProperty(key)) {
9090
continue;
9191
}
9292
if (key in base) {

0 commit comments

Comments
 (0)