Closed
Bug 1183894
Opened 9 years ago
Closed 9 years ago
2,000 instances of "Called close() before start()!: 'mStarted'" emitted from dom/workers/MessagePort.cpp during linux64 debug testing
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: erahm, Assigned: erahm)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #643325 +++ > 1997 [NNNNN] WARNING: Called close() before start()!: 'mStarted', file dom/workers/MessagePort.cpp, line 216 This warning [1], introduced in bug 643325, shows up in the following test suites: > mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-4-bm122-tests1-linux64-build12.txt:780 > mozilla-central_ubuntu64_vm-debug_test-mochitest-4-bm121-tests1-linux64-build0.txt:653 > mozilla-central_ubuntu64_vm-debug_test-mochitest-3-bm115-tests1-linux64-build21.txt:345 > mozilla-central_ubuntu64_vm-debug_test-mochitest-1-bm117-tests1-linux64-build1.txt:77 > mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-1-bm123-tests1-linux64-build25.txt:65 > mozilla-central_ubuntu64_vm-debug_test-mochitest-push-bm68-tests1-linux64-build20.txt:45 > mozilla-central_ubuntu64_vm-debug_test-mochitest-other-bm53-tests1-linux64-build0.txt:7 > mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-browser-chrome-3-bm118-tests1-linux64-build0.txt:5 > mozilla-central_ubuntu64_vm-debug_test-mochitest-devtools-chrome-2-bm120-tests1-linux64-build0.txt:5 > mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-2-bm53-tests1-linux64-build12.txt:4 > mozilla-central_ubuntu64_vm-debug_test-mochitest-browser-chrome-1-bm122-tests1-linux64-build19.txt:4 > mozilla-central_ubuntu64_vm-debug_test-mochitest-2-bm67-tests1-linux64-build1.txt:4 > mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-3-bm51-tests1-linux64-build0.txt:3 It shows up in 240 tests. A few of the most prevalent: > 178 - dom/tests/mochitest/fetch/test_fetch_cors_sw_reroute.html > 88 - dom/workers/test/serviceworkers/test_fetch_event.html > 88 - dom/workers/test/serviceworkers/test_fetch_event.html > 53 - dom/workers/test/serviceworkers/test_match_all_client_properties.html > 39 - dom/workers/test/test_blobWorkers.html > 39 - dom/workers/test/serviceworkers/test_request_context_nestedworkerinsharedworker.html > 38 - dom/workers/test/serviceworkers/test_third_party_iframes.html > 35 - dom/tests/mochitest/general/test_497898.html > 35 - dom/push/test/test_unregister.html > 32 - dom/workers/test/serviceworkers/test_sanitize.html [1] https://hg.mozilla.org/mozilla-central/annotate/e786406bc683/dom/workers/MessagePort.cpp#l216
Assignee | ||
Comment 1•9 years ago
|
||
This seems like something that can legitimately happen (closing before starting), I propose we just remove the warning.
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8633824 -
Flags: review?(khuey)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → erahm
Status: NEW → ASSIGNED
Attachment #8633824 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/90f284794775
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/90f284794775
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•