Closed Bug 1626404 Opened 4 years ago Closed 4 years ago

[Fission] Google Docs' "Code Blocks" Add-on fails with "NetworkError: Connection failure due to HTTP 0"

Tracking

()

Status:

VERIFIED FIXED

Milestone:

mozilla77

Project Flags:

Fission Milestone

Tracking Flags:

Tracking

Status

firefox77

---

verified

People

(Reporter: annyG, Assigned: kmag)

References

Details

Attachments

(10 files)

Screen Shot 2020-03-31 at 2.00.55 PM.png 4 years ago Anny Gakhokidze [:annyG] 447.50 KB, image/png		Details
a GIF describing what the failed scenario looks like with Fission enabled 4 years ago Anny Gakhokidze [:annyG] 2.15 MB, image/gif		Details
a GIF showing what the success scenario looks like WITHOUT Fission enabled 4 years ago Anny Gakhokidze [:annyG] 2.13 MB, image/gif		Details
Bug 1626404: Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Part 2b - Serialize source BC with LoadState. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1626404: Follow-up: Fix missing source BC in some window.open calls. 4 years ago Kris Maglione [:kmag] 47 bytes, text/x-phabricator-request		Details \| Review

Anny Gakhokidze [:annyG]

Reporter

Description

•

4 years ago

Attached image Screen Shot 2020-03-31 at 2.00.55 PM.png — Details

Steps to Reproduce:

Open a new google doc
Write some text and click on Add-Ons -> Code Blocks -> Start
Select the text and click Format

Expected behaviour: text gets formatted
Actual behaviour: a network error "NetworkError: Connection failure due to HTTP 0" is displayed to the user and the page is redirected to a different URL (and the page looks blank). Since the redirection happens too fast I attached (and transcribed) a snapshot of what the error looks like.

Anny Gakhokidze [:annyG]

Reporter

Comment 1

•

4 years ago

Attached image a GIF describing what the failed scenario looks like with Fission enabled — Details

Anny Gakhokidze [:annyG]

Reporter

Comment 2

•

4 years ago

Attached image a GIF showing what the success scenario looks like WITHOUT Fission enabled — Details

I have attached two GIFs showing behaviour with/without Fission

Anny Gakhokidze [:annyG]

Reporter

Updated

•

4 years ago

Blocks: fission-dogfooding

Anny Gakhokidze [:annyG]

Reporter

Updated

•

4 years ago

Summary: [Fission] Unable to use Code-Blocks extension in Google Doc → [Fission] Unable to use Google Doc's Code-Blocks extension

Chris Peterson [:cpeterson]

Comment 3

•

4 years ago

Tentatively moving to the Networking component because the network error ("NetworkError: Connection failure due to HTTP 0") is suspicious.

Anny clarified on Matrix that "Code Blocks" is a Google Docs Add-on, not a Firefox WebExtension.

Fission Milestone: --- → M6

Component: General → Networking

Product: Firefox → Core

Summary: [Fission] Unable to use Google Doc's Code-Blocks extension → [Fission] Google Docs' "Code Blocks" Add-on fails with "NetworkError: Connection failure due to HTTP 0"

Kris Maglione [:kmag]

Assignee

Updated

•

4 years ago

Assignee: nobody → kmaglione+bmo

Component: Networking → DOM: Navigation

Olli Pettay [:smaug][bugs@pettay.fi]

Updated

•

4 years ago

Priority: -- → P3

Kris Maglione [:kmag]

Assignee

Comment 4

•

4 years ago

Attached file Bug 1626404: Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika — Details

Kris Maglione [:kmag]

Assignee

Comment 5

•

4 years ago

Attached file Bug 1626404: Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika — Details

Kris Maglione [:kmag]

Assignee

Comment 6

•

4 years ago

Attached file Bug 1626404: Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika — Details

Kris Maglione [:kmag]

Assignee

Comment 7

•

4 years ago

Attached file Bug 1626404: Part 2b - Serialize source BC with LoadState. r=nika — Details

Kris Maglione [:kmag]

Assignee

Comment 8

•

4 years ago

Attached file Bug 1626404: Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika — Details

Kris Maglione [:kmag]

Assignee

Comment 9

•

4 years ago

Attached file Bug 1626404: Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika — Details

Pulsebot

Comment 10

•

4 years ago

Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/51120d617459
Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika
https://hg.mozilla.org/integration/autoland/rev/e255e4c9e665
Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika
https://hg.mozilla.org/integration/autoland/rev/df37f57cfa1d
Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/322c170e8301
Part 2b - Serialize source BC with LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/ad3272abd830
Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika
https://hg.mozilla.org/integration/autoland/rev/a399dfff2725
Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika

Narcis Beleuzu [:NarcisB]

Comment 11

•

4 years ago

Backed out for bustages on LocationBase.

Backout link: https://hg.mozilla.org/integration/autoland/rev/0635660942b60968c915eb55d1668c39cf8bdccf
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=296221444&repo=autoland&lineNumber=44499

Flags: needinfo?(kmaglione+bmo)

Pulsebot

Comment 12

•

4 years ago

Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/440b9193338a
Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika
https://hg.mozilla.org/integration/autoland/rev/52892d053ff1
Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika
https://hg.mozilla.org/integration/autoland/rev/a248fc78ab0c
Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/c065bd4594b4
Part 2b - Serialize source BC with LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/ad87dcb98637
Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika
https://hg.mozilla.org/integration/autoland/rev/aed006ab4c20
Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika

Narcis Beleuzu [:NarcisB]

Comment 13

•

4 years ago

Backed out for assertion failures on BrowsingContext.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/28e3a51b173eb939f72f4740ea72735832ecad3b
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=296243338&repo=autoland&lineNumber=2609

Kris Maglione [:kmag]

Assignee

Updated

•

4 years ago

Flags: needinfo?(kmaglione+bmo)

Pulsebot

Comment 14

•

4 years ago

Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/bf32b30efa49
Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika
https://hg.mozilla.org/integration/autoland/rev/c0c2e85e33da
Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika
https://hg.mozilla.org/integration/autoland/rev/6fb384961b99
Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/2ffdab0072e7
Part 2b - Serialize source BC with LoadState. r=nika
https://hg.mozilla.org/integration/autoland/rev/1a5a559ec6d5
Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika
https://hg.mozilla.org/integration/autoland/rev/4a3b391e08c4
Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika

Kris Maglione [:kmag]

Assignee

Comment 15

•

4 years ago

Attached file Bug 1626404: Follow-up: Fix missing source BC in some window.open calls. — Details

Pulsebot

Comment 16

•

4 years ago

Backout by opoprus@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c5e4e7f80049
Backed out 6 changesets for mochitest failures in BrowsingContext.cpp on a CLOSED TREE

Oana Pop-Rus

Comment 17

•

4 years ago

Backed out 6 changesets (bug 1626404) for mochitest failures in BrowsingContext.cpp

Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=296842740&resultStatus=success%2Csuperseded%2Ctestfailed%2Cbusted%2Cexception%2Crunnable&fromchange=f39a87ff23d8be6d86e919748f4585397d52e374&searchStr=linux%2C18.04%2Cx64%2Cquantumrender%2Cdebug%2Cmochitests%2Cwith%2Cfission%2Cenabled%2Ctest-linux1804-64-qr%2Fdebug-mochitest-plain-fis-e10s-7%2Cm-fis%287%29

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=296842740&repo=autoland&lineNumber=1860

Backout: https://hg.mozilla.org/integration/autoland/rev/c5e4e7f80049dc1f761ae8eeb08cbf95e79bcb9d

Flags: needinfo?(kmaglione+bmo)

Pulsebot

Comment 18

•

4 years ago

Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5349e5225018
Follow-up: Fix missing source BC in some window.open calls.

Pulsebot

Comment 19

•

4 years ago

Pushed by opoprus@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/45b68d43bdbb
Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/af763de00e9d
Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/b0d4b213b819
Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/ec63b18da4b9
Part 2b - Serialize source BC with LoadState. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/f54ce6a36c75
Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/453b858f4462
Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika CLOSED TREE

Bogdan Tara[:bogdan_tara | bogdant]

Comment 20

•

4 years ago

Backed out 6 changesets (bug 1626404) for frequent timeouts on test_postMessage_onOther.html

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&searchStr=linux%2C18.04%2Cx64%2Cquantumrender%2Cdebug%2Cmochitests%2Cwith%2Cfission%2Cenabled%2Ctest-linux1804-64-qr%2Fdebug-mochitest-plain-fis-e10s-8%2Cm-fis%288%29&fromchange=453b858f4462272af20b61cdddc99b6c8e90bee9&tochange=eb0fdd0ea21ac5a3f8c172194b98ff084243385e&selectedJob=296905564

Backout link: https://hg.mozilla.org/integration/autoland/rev/eb0fdd0ea21ac5a3f8c172194b98ff084243385e

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=296905564&repo=autoland&lineNumber=8473

[task 2020-04-09T07:14:50.409Z] 07:14:50     INFO - TEST-START | dom/tests/mochitest/whatwg/test_postMessage_onOther.html
[task 2020-04-09T07:14:50.627Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.628Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.629Z] 07:14:50     INFO - GECKO(4142) | [GPU 4177, Compositor] WARNING: bad Shmem: file /builds/worker/checkouts/gecko/ipc/glue/ProtocolUtils.cpp, line 434
[task 2020-04-09T07:14:50.824Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /builds/worker/checkouts/gecko/netwerk/base/nsNetUtil.cpp, line 2307
[task 2020-04-09T07:14:50.828Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/caps/ContentPrincipal.cpp, line 398
[task 2020-04-09T07:14:50.828Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /builds/worker/checkouts/gecko/caps/ContentPrincipal.cpp, line 423
[task 2020-04-09T07:14:50.828Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /builds/worker/checkouts/gecko/netwerk/base/nsNetUtil.cpp, line 2307
[task 2020-04-09T07:14:50.828Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/caps/ContentPrincipal.cpp, line 398
[task 2020-04-09T07:14:50.829Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /builds/worker/checkouts/gecko/caps/ContentPrincipal.cpp, line 423
[task 2020-04-09T07:14:50.849Z] 07:14:50     INFO - GECKO(4142) | [Child 4518, Main Thread] WARNING: Extra shutdown CC: 'i < NORMAL_SHUTDOWN_COLLECTIONS', file /builds/worker/checkouts/gecko/xpcom/base/nsCycleCollector.cpp, line 3352
[task 2020-04-09T07:14:50.946Z] 07:14:50     INFO - GECKO(4142) | nsStringStats
[task 2020-04-09T07:14:50.947Z] 07:14:50     INFO - GECKO(4142) |  => mAllocCount:           8662
[task 2020-04-09T07:14:50.948Z] 07:14:50     INFO - GECKO(4142) |  => mReallocCount:            0
[task 2020-04-09T07:14:50.948Z] 07:14:50     INFO - GECKO(4142) |  => mFreeCount:            8662
[task 2020-04-09T07:14:50.948Z] 07:14:50     INFO - GECKO(4142) |  => mShareCount:           4828
[task 2020-04-09T07:14:50.948Z] 07:14:50     INFO - GECKO(4142) |  => mAdoptCount:            231
[task 2020-04-09T07:14:50.948Z] 07:14:50     INFO - GECKO(4142) |  => mAdoptFreeCount:        241
[task 2020-04-09T07:14:50.949Z] 07:14:50     INFO - GECKO(4142) |  => Process ID: 4518, Thread ID: 140432070252416
[task 2020-04-09T07:14:50.987Z] 07:14:50     INFO - GECKO(4142) | ### XPCOM_MEM_BLOAT_LOG defined -- logging bloat/leaks to /tmp/tmpHBNC2c.mozrunner/runtests_leaks_tab_pid4547.log
[task 2020-04-09T07:14:50.987Z] 07:14:50     INFO - GECKO(4142) | ### XPCOM_MEM_BLOAT_LOG defined -- logging bloat/leaks to /tmp/tmpHBNC2c.mozrunner/runtests_leaks_tab_pid4546.log
[task 2020-04-09T07:14:51.188Z] 07:14:51     INFO - GECKO(4142) | [Child 4403, Main Thread] WARNING: NS_ENSURE_TRUE(request) failed: file /builds/worker/checkouts/gecko/netwerk/base/nsLoadGroup.cpp, line 591
[task 2020-04-09T07:14:51.196Z] 07:14:51     INFO - GECKO(4142) | [Child 4403, Main Thread] WARNING: NS_ENSURE_TRUE(request) failed: file /builds/worker/checkouts/gecko/netwerk/base/nsLoadGroup.cpp, line 591
[task 2020-04-09T07:14:51.313Z] 07:14:51     INFO - GECKO(4142) | Couldn't convert chrome URL: chrome://branding/locale/brand.properties
[task 2020-04-09T07:14:51.457Z] 07:14:51     INFO - GECKO(4142) | Couldn't convert chrome URL: chrome://branding/locale/brand.properties
[task 2020-04-09T07:14:51.622Z] 07:14:51     INFO - GECKO(4142) | [Child 4547, Main Thread] WARNING: could not set real-time limit at process startup: file /builds/worker/checkouts/gecko/dom/ipc/ContentChild.cpp, line 1677
[task 2020-04-09T07:14:51.622Z] 07:14:51     INFO - GECKO(4142) | [Child 4546, Main Thread] WARNING: could not set real-time limit at process startup: file /builds/worker/checkouts/gecko/dom/ipc/ContentChild.cpp, line 1677
[task 2020-04-09T07:14:51.657Z] 07:14:51     INFO - GECKO(4142) | [Child 4546, Main Thread] WARNING: NS_ENSURE_TRUE(mPresShell) failed: file /builds/worker/checkouts/gecko/layout/base/nsPresContext.cpp, line 844
[task 2020-04-09T07:14:51.678Z] 07:14:51     INFO - GECKO(4142) | [Child 4547, Main Thread] WARNING: NS_ENSURE_TRUE(mPresShell) failed: file /builds/worker/checkouts/gecko/layout/base/nsPresContext.cpp, line 844
[task 2020-04-09T07:14:51.948Z] 07:14:51     INFO - GECKO(4142) | JavaScript error: http://example.com/tests/dom/tests/mochitest/whatwg/postMessage_onOther.html, line 88: SecurityError: Permission denied to access property "testSiblingPostMessage" on cross-origin object
[task 2020-04-09T07:20:06.730Z] 07:20:06     INFO - TEST-INFO | started process screentopng
[task 2020-04-09T07:20:06.886Z] 07:20:06     INFO - TEST-INFO | screentopng: exit 0
[task 2020-04-09T07:20:06.886Z] 07:20:06     INFO - TEST-UNEXPECTED-FAIL | dom/tests/mochitest/whatwg/test_postMessage_onOther.html | Test timed out. 
[task 2020-04-09T07:20:06.887Z] 07:20:06     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:299:16
[task 2020-04-09T07:20:06.887Z] 07:20:06     INFO -     reportError@SimpleTest/TestRunner.js:128:22
[task 2020-04-09T07:20:06.888Z] 07:20:06     INFO -     TestRunner._checkForHangs@SimpleTest/TestRunner.js:150:18
[task 2020-04-09T07:20:06.888Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.888Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.888Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.889Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.889Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.889Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.890Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.890Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.890Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.890Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.890Z] 07:20:06     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:184:15
[task 2020-04-09T07:20:06.891Z] 07:20:06     INFO -     TestRunner.runTests/<@SimpleTest/TestRunner.js:420:16
[task 2020-04-09T07:20:06.891Z] 07:20:06     INFO -     promise callback*TestRunner.runTests@SimpleTest/TestRunner.js:407:48
[task 2020-04-09T07:20:06.891Z] 07:20:06     INFO -     RunSet.runtests@SimpleTest/setup.js:218:14
[task 2020-04-09T07:20:06.891Z] 07:20:06     INFO -     RunSet.runall@SimpleTest/setup.js:197:12
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO -     hookupTests@SimpleTest/setup.js:294:12
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:48:13
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:61:28
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:57:3
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO -     hookup@SimpleTest/setup.js:270:20
[task 2020-04-09T07:20:06.892Z] 07:20:06     INFO - EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp&cleanupCrashes=true:11:1
[task 2020-04-09T07:20:07.743Z] 07:20:07     INFO - GECKO(4142) | MEMORY STAT | vsize 2537MB | residentFast 133MB | heapAllocated 11MB
[task 2020-04-09T07:20:07.760Z] 07:20:07     INFO - TEST-OK | dom/tests/mochitest/whatwg/test_postMessage_onOther.html | took 317349ms

Pulsebot

Comment 21

•

4 years ago

Backout by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/25d1c9d1e7e2
Backed out changeset 5349e5225018 frequent timeouts on test_postMessage_onOther.html CLOSED TREE

Pulsebot

Comment 22

•

4 years ago

Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/466821af996e
Follow-up: Fix missing source BC in some window.open calls.
https://hg.mozilla.org/integration/autoland/rev/cdd400bbf5ba
Part 0 - Add operators `bool` and `->` to MaybeDiscarded to match Maybe. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/acc4a7da3b21
Part 1 - Move IsSandboxedFrom to BrowsingContext. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/633fc86926f3
Part 2a - Store source BrowsingContext rather than DocShell in LoadState. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/d7bb3fe6b03c
Part 2b - Serialize source BC with LoadState. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/43355e9523c4
Part 3 - Do sandbox flag checks at both ends of cross-process loads. r=nika CLOSED TREE
https://hg.mozilla.org/integration/autoland/rev/f85190eebb8e
Part 4 - Remove now-redundant accessor args from BrowsingContext load methods. r=nika CLOSED TREE

Bogdan Tara[:bogdan_tara | bogdant]

Comment 23

•

4 years ago

Sorry for the backout.

Daniel Varga [:dvarga]

Comment 24

•

4 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/466821af996e
https://hg.mozilla.org/mozilla-central/rev/cdd400bbf5ba
https://hg.mozilla.org/mozilla-central/rev/acc4a7da3b21
https://hg.mozilla.org/mozilla-central/rev/633fc86926f3
https://hg.mozilla.org/mozilla-central/rev/d7bb3fe6b03c
https://hg.mozilla.org/mozilla-central/rev/43355e9523c4
https://hg.mozilla.org/mozilla-central/rev/f85190eebb8e

Status: NEW → RESOLVED

Closed: 4 years ago

status-firefox77: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla77

Kris Maglione [:kmag]

Assignee

Updated

•

4 years ago

Flags: needinfo?(kmaglione+bmo)

Cristian Baica [:cbaica], Release Desktop QA

Updated

•

4 years ago

Flags: qe-verify+

Bogdan Maris, Desktop QA

Comment 25

•

4 years ago

Reproduced the initial issue using an old Nightly build from 2020-03-31. Verified that using latest Nightly 78.0a1 and Nightly 77.0a1 after the fix is indeed fixed across platforms (Windows 10 64bit, macOS 10.15.5 beta 3 and Ubuntu 18.04 64bit).

Status: RESOLVED → VERIFIED

status-firefox77: fixed → verified

Flags: qe-verify+

You need to log in before you can comment on or make changes to this bug.