Closed Bug 1183894 Opened 5 years ago Closed 5 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)

x86
All
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: erahm, Assigned: erahm)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ 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
This seems like something that can legitimately happen (closing before starting), I propose we just remove the warning.
Assignee: nobody → erahm
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/90f284794775
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.