summaryrefslogtreecommitdiffstats
path: root/services/sync
diff options
context:
space:
mode:
authorMoonchild <mcwerewolf@gmail.com>2018-05-02 16:23:14 +0200
committerGitHub <noreply@github.com>2018-05-02 16:23:14 +0200
commit04c8f8f8bc2d2dccb6675bd1ed9912f098e76739 (patch)
tree7d78110b55ac5d72eb9f15584ec18afbb5948670 /services/sync
parent832046aeb726dee290d2827a20d571034b6bc64a (diff)
parent3442ae7718679a4da8664bdbb561f746e0357203 (diff)
downloadUXP-04c8f8f8bc2d2dccb6675bd1ed9912f098e76739.tar
UXP-04c8f8f8bc2d2dccb6675bd1ed9912f098e76739.tar.gz
UXP-04c8f8f8bc2d2dccb6675bd1ed9912f098e76739.tar.lz
UXP-04c8f8f8bc2d2dccb6675bd1ed9912f098e76739.tar.xz
UXP-04c8f8f8bc2d2dccb6675bd1ed9912f098e76739.zip
Merge pull request #320 from janekptacijarabaci/instanceof_negations_1
palemoon#1173: Fix unsafe "instanceof" negations
Diffstat (limited to 'services/sync')
-rw-r--r--services/sync/modules/record.js4
-rw-r--r--services/sync/modules/service.js2
-rw-r--r--services/sync/tps/extensions/mozmill/resource/driver/controller.js2
-rw-r--r--services/sync/tps/extensions/mozmill/resource/modules/assertions.js2
4 files changed, 5 insertions, 5 deletions
diff --git a/services/sync/modules/record.js b/services/sync/modules/record.js
index 02f7f281a..f7a69d9ef 100644
--- a/services/sync/modules/record.js
+++ b/services/sync/modules/record.js
@@ -43,7 +43,7 @@ WBORecord.prototype = {
// Get thyself from your URI, then deserialize.
// Set thine 'response' field.
fetch: function fetch(resource) {
- if (!resource instanceof Resource) {
+ if (!(resource instanceof Resource)) {
throw new Error("First argument must be a Resource instance.");
}
@@ -56,7 +56,7 @@ WBORecord.prototype = {
},
upload: function upload(resource) {
- if (!resource instanceof Resource) {
+ if (!(resource instanceof Resource)) {
throw new Error("First argument must be a Resource instance.");
}
diff --git a/services/sync/modules/service.js b/services/sync/modules/service.js
index b0eb0f41d..5c91323b5 100644
--- a/services/sync/modules/service.js
+++ b/services/sync/modules/service.js
@@ -455,7 +455,7 @@ Sync11Service.prototype = {
this.clientsEngine = new ClientEngine(this);
for (let name of engines) {
- if (!name in ENGINE_MODULES) {
+ if (!(name in ENGINE_MODULES)) {
this._log.info("Do not know about engine: " + name);
continue;
}
diff --git a/services/sync/tps/extensions/mozmill/resource/driver/controller.js b/services/sync/tps/extensions/mozmill/resource/driver/controller.js
index a378ce51f..8d66a41ae 100644
--- a/services/sync/tps/extensions/mozmill/resource/driver/controller.js
+++ b/services/sync/tps/extensions/mozmill/resource/driver/controller.js
@@ -978,7 +978,7 @@ function browserAdditions (controller) {
}, "Timeout", timeout, aInterval);
}
catch (ex) {
- if (!ex instanceof errors.TimeoutError) {
+ if (!(ex instanceof errors.TimeoutError)) {
throw ex;
}
timed_out = true;
diff --git a/services/sync/tps/extensions/mozmill/resource/modules/assertions.js b/services/sync/tps/extensions/mozmill/resource/modules/assertions.js
index c9991acf0..c76f95747 100644
--- a/services/sync/tps/extensions/mozmill/resource/modules/assertions.js
+++ b/services/sync/tps/extensions/mozmill/resource/modules/assertions.js
@@ -659,7 +659,7 @@ Expect.prototype.waitFor = function Expect_waitFor(aCallback, aMessage, aTimeout
Assert.prototype.waitFor.apply(this, arguments);
}
catch (ex) {
- if (!ex instanceof errors.AssertionError) {
+ if (!(ex instanceof errors.AssertionError)) {
throw ex;
}
message = ex.message;