Bug 1425244 - Enable ESLint rule space-unary-ops for accessible/tests/mochitest/. r?surkov
MozReview-Commit-ID: BPufvA8kUkv
--- a/accessible/tests/mochitest/.eslintrc.js
+++ b/accessible/tests/mochitest/.eslintrc.js
@@ -10,11 +10,10 @@ module.exports = {
// XXX These are rules that are enabled in the recommended configuration, but
// disabled here due to failures when initially implemented. They should be
// removed (and hence enabled) at some stage.
"no-nested-ternary": "off",
"no-redeclare": "off",
"no-shadow": "off",
"no-undef": "off",
- "space-unary-ops": "off",
}
};
--- a/accessible/tests/mochitest/common.js
+++ b/accessible/tests/mochitest/common.js
@@ -452,17 +452,17 @@ function testAccessibleTree(aAccOrElmOrI
case "tagName":
is(accTree[prop], acc.DOMNode.tagName, msg);
break;
case "textAttrs": {
var prevOffset = -1;
for (var offset in accTree[prop]) {
- if (prevOffset != - 1) {
+ if (prevOffset != -1) {
var attrs = accTree[prop][prevOffset];
testTextAttrs(acc, prevOffset, attrs, { }, prevOffset, +offset, true);
}
prevOffset = +offset;
}
if (prevOffset != -1) {
var charCount = getAccessible(acc, [nsIAccessibleText]).characterCount;
--- a/accessible/tests/mochitest/events.js
+++ b/accessible/tests/mochitest/events.js
@@ -2114,17 +2114,17 @@ var gLogger =
Services.console.logStringMessage("events: " + aMsg);
},
/**
* Return true if logging feature is enabled.
*/
hasFeature: function logger_hasFeature(aFeature) {
var startIdx = gA11yEventDumpFeature.indexOf(aFeature);
- if (startIdx == - 1)
+ if (startIdx == -1)
return false;
var endIdx = startIdx + aFeature.length;
return endIdx == gA11yEventDumpFeature.length ||
gA11yEventDumpFeature[endIdx] == ";";
}
};
--- a/accessible/tests/mochitest/role.js
+++ b/accessible/tests/mochitest/role.js
@@ -162,17 +162,17 @@ function getRole(aAccOrElmOrID) {
return role;
}
/**
* Analogy of SimpleTest.is function used to check the role.
*/
function isRole(aIdentifier, aRole, aMsg) {
var role = getRole(aIdentifier);
- if (role == - 1)
+ if (role == -1)
return;
if (role == aRole) {
ok(true, aMsg);
return;
}
var got = roleToString(role);
--- a/accessible/tests/mochitest/table.js
+++ b/accessible/tests/mochitest/table.js
@@ -232,17 +232,17 @@ function testTableIndexes(aIdentifier, a
try {
cellAcc = null;
cellAcc = tableAcc.getCellAt(rowIdx, colIdx);
} catch (e) { }
ok(idx != -1 && cellAcc || idx == -1 && !cellAcc,
id + ": Can't get cell accessible at row = " + rowIdx + ", column = " + colIdx);
- if (idx != - 1) {
+ if (idx != -1) {
// getRowIndexAt
var origRowIdx = rowIdx;
while (origRowIdx > 0 &&
aIdxes[rowIdx][colIdx] == aIdxes[origRowIdx - 1][colIdx])
origRowIdx--;
try {