Bug 2033191 made sure we only call ownerGlobal on nodes.
As per the discussion in dev-platform, the front-end ~always wants
.documentGlobal. This removes ownerGlobal from the bindings. Following
patch will rename internal GetOwnerGlobal in gecko to GetRelevantGlobal
as per the HTML spec terminology.
Differential Revision: https://phabricator.services.mozilla.com/D297183
Bug 2033191 made sure we only call ownerGlobal on nodes.
As per the discussion in dev-platform, the front-end ~always wants
.documentGlobal. This removes ownerGlobal from the bindings. Following
patch will rename internal GetOwnerGlobal in gecko to GetRelevantGlobal
as per the HTML spec terminology.
Differential Revision: https://phabricator.services.mozilla.com/D297183
Bug 2033191 made sure we only call ownerGlobal on nodes.
As per the discussion in dev-platform, the front-end ~always wants
.documentGlobal. This removes ownerGlobal from the bindings. Following
patch will rename internal GetOwnerGlobal in gecko to GetRelevantGlobal
as per the HTML spec terminology.
Differential Revision: https://phabricator.services.mozilla.com/D297183
Bug 2033191 made sure we only call ownerGlobal on nodes.
As per the discussion in dev-platform, the front-end ~always wants
.documentGlobal. This removes ownerGlobal from the bindings. Following
patch will rename internal GetOwnerGlobal in gecko to GetRelevantGlobal
as per the HTML spec terminology.
Differential Revision: https://phabricator.services.mozilla.com/D297183
Pass the exception's actual stack via the harness's stack field instead of
synthesizing one rooted in simpletestOnerror or assertNoUncaughtRejections.
For uncaught rejections, also pass the rejection date as the assertion's
timestamp rather than baking it into the failure message, so otherwise
identical failures fingerprint to the same message. The rejection failure
message is simplified to "uncaught rejection: <message>".
Plumbing changes:
- Assert.report / Assert.AssertionError accept a string stack and an
optional time.
- StructuredLog.testStatus accepts an optional time that overrides the
default Date.now().
- The mochitest reporter and browser-harness.xhtml forward the new time
field through testResult and addResult.
Differential Revision: https://phabricator.services.mozilla.com/D296797
Pass the exception's actual stack via the harness's stack field instead of
synthesizing one rooted in simpletestOnerror or assertNoUncaughtRejections.
For uncaught rejections, also pass the rejection date as the assertion's
timestamp rather than baking it into the failure message, so otherwise
identical failures fingerprint to the same message. The rejection failure
message is simplified to "uncaught rejection: <message>".
Plumbing changes:
- Assert.report / Assert.AssertionError accept a string stack and an
optional time.
- StructuredLog.testStatus accepts an optional time that overrides the
default Date.now().
- The mochitest reporter and browser-harness.xhtml forward the new time
field through testResult and addResult.
Differential Revision: https://phabricator.services.mozilla.com/D296797
This logic was only tracking privileged parent process javascript,
whereas JS_CODE_COVERAGE_OUTPUT_DIR is the typical equivalent and works
on all processes.
Differential Revision: https://phabricator.services.mozilla.com/D285084
This reverts commit 2708c7e775.
Revert "Bug 2009377 - make browser-chrome tests log testEnd with a TIMEOUT status when they shutdown the browser due to a timeout, r=ahal."
This reverts commit 56681d556f.
Revert "Bug 2009377 - make browser-chrome tests give a PASS/FAIL status to testEnd rather than a constant OK, r=ahal,jwatt."
This reverts commit 8b9335a98e.
This reverts commit 5739a42054.
Revert "Bug 2009377 - make browser-chrome tests log testEnd with a TIMEOUT status when they shutdown the browser due to a timeout, r=ahal."
This reverts commit 7e143ba52b.
Revert "Bug 2009377 - make browser-chrome tests give a PASS/FAIL status to testEnd rather than a constant OK, r=ahal,jwatt."
This reverts commit f404f8ebb2.
This reverts commit d96e9bd92c.
Revert "Bug 1993543 - Change XPCOMUtils.defineLazyServiceGetter(s) calls to pass an nsIID rather than a string. r=firefox-desktop-core-reviewers ,mtigley,joschmidt,perftest-reviewers,home-newtab-reviewers,nbarrett,Gijs,sparky"
This reverts commit 024c584172.
This reverts commit ca0a5d1707.
Revert "Bug 1983850 - Fix newtab trainhop compatibility issue with ChromeUtils.now usage in TelemetryFeed and mochitest. r=home-newtab-reviewers,thecount,florian"
This reverts commit 085062c936.
Defining `skipIf` like `get: MochiaImpl.skipIf` installs the function as a
getter, not a method, which causes `MochiaImpl.skipIf()` to be invoked with
zero arguments when `it.skipIf` is accessed. `it.skipIf("desc", cond, test)`
then becomes a call to the result of that zero-argument invocation, instead
of invoking the actual `skipIf` function with arguments
Depends on D259870
Differential Revision: https://phabricator.services.mozilla.com/D259871
Alexandre Lissy