Reverse the isSelfHosted test and if blocks in jsapi.cpp and XrayWrapper.cpp

RESOLVED FIXED in mozilla36

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

unspecified
mozilla36
x86
macOS
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

So instead of:

  if (!isSelfHosted() {
    foo;
  } else {
    bar;
  }

do

  if (isSelfHosted() {
    bar;
  } else {
    foo;
  }
Depends on: 1088002
Created attachment 8513726 [details] [diff] [review]
Reverse the isSelfHosted test in XrayWrapper and jsapi.cpp
Attachment #8513726 - Flags: review?(jwalden+bmo)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8513726 - Flags: review?(jwalden+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/2a12da881f92
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.