Closed Bug 1437991 Opened 2 years ago Closed 7 months ago

327,100 instances of "'!mMainThread'" emitted from xpcom/threads/nsThreadManager.cpp during linux64 debug testing

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: erahm, Assigned: erahm)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

> 41562 WARNING: '!mMainThread', file xpcom/threads/nsThreadManager.cpp, line 517

This warning [1] shows up in the following test suites:

>   2752 - test-linux64/debug-web-platform-tests-wdspec-e10s Wd
>   1832 - test-linux64/debug-test-verify-e10s TV
>   1832 - test-linux64/debug-web-platform-tests-e10s-9 wpt9
>   1760 - test-linux64/debug-web-platform-tests-e10s-10 wpt10
>   1705 - test-linux64/debug-web-platform-tests-e10s-6 wpt6
>   1640 - test-linux64/debug-web-platform-tests-e10s-8 wpt8
>   1600 - test-linux64/debug-web-platform-tests-e10s-1 wpt1
>   1473 - test-linux64/debug-web-platform-tests-e10s-7 wpt7
>   1424 - test-linux64/debug-web-platform-tests-e10s-12 wpt12
>   1408 - test-linux64/debug-web-platform-tests-e10s-11 wpt11
>   1353 - test-linux64/debug-web-platform-tests-e10s-3 wpt3
>   1352 - test-linux64/debug-web-platform-tests-e10s-5 wpt5
>   1328 - test-linux64/debug-web-platform-tests-e10s-4 wpt4
>   1304 - test-linux64/debug-web-platform-tests-e10s-2 wpt2
>    896 - test-linux64/debug-web-platform-tests-reftests-e10s-1 Wr1
>    866 - test-linux64/debug-web-platform-tests-reftests-e10s-3 Wr3
>    832 - test-linux64/debug-firefox-ui-functional-local-e10s en-US
>    765 - test-linux64/debug-web-platform-tests-reftests-e10s-5 Wr5
>    738 - test-linux64/debug-web-platform-tests-reftests-e10s-4 Wr4
>    736 - test-linux64/debug-mochitest-plain-headless-e10s-15 h15
>    704 - test-linux64/debug-mochitest-chrome-3 c3
>    700 - test-linux64/debug-web-platform-tests-reftests-e10s-6 Wr6
>    672 - test-linux64/debug-mochitest-e10s-15 15
>    640 - test-linux64/debug-mochitest-chrome-1 c1
>    562 - test-linux64/debug-web-platform-tests-reftests-e10s-2 Wr2
>    560 - test-linux64/debug-test-verify-wpt-e10s TVw
>    512 - test-linux64/debug-mochitest-chrome-2 c2
>    464 - test-linux64/debug-marionette-e10s Mn
>    456 - test-linux64/debug-marionette-headless-e10s MnH
>    344 - test-linux64/debug-mochitest-clipboard-e10s cl
>    264 - test-linux64/debug-mochitest-browser-chrome-e10s-4 bc4
>    216 - test-linux64/debug-mochitest-e10s-16 16
>    208 - test-linux64/debug-mochitest-e10s-12 12
>    208 - test-linux64/debug-mochitest-plain-headless-e10s-12 h12
>    184 - test-linux64/debug-mochitest-e10s-8 8
>    184 - test-linux64/debug-mochitest-plain-headless-e10s-8 h8
>    176 - test-linux64/debug-mochitest-e10s-5 5
>    176 - test-linux64/debug-mochitest-plain-headless-e10s-5 h5
>    168 - test-linux64/debug-mochitest-browser-chrome-e10s-7 bc7
>    168 - test-linux64/debug-mochitest-plain-headless-e10s-16 h16
>    160 - test-linux64/debug-mochitest-plain-headless-e10s-1 h1
>    160 - test-linux64/debug-mochitest-e10s-1 1
>    160 - test-linux64/debug-mochitest-e10s-10 10
>    152 - test-linux64/debug-mochitest-browser-chrome-e10s-3 bc3
>    152 - test-linux64/debug-mochitest-plain-headless-e10s-4 h4
>    152 - test-linux64/debug-mochitest-e10s-4 4
>    152 - test-linux64/debug-mochitest-plain-headless-e10s-10 h10
>    152 - test-linux64/debug-mochitest-browser-chrome-e10s-12 bc12
>    144 - test-linux64/debug-mochitest-devtools-chrome-e10s-1 dt1
>    128 - test-linux64/debug-mochitest-e10s-11 11
>    128 - test-linux64/debug-mochitest-e10s-6 6
>    128 - test-linux64/debug-mochitest-plain-headless-e10s-11 h11
>    128 - test-linux64/debug-mochitest-e10s-9 9
>    128 - test-linux64/debug-mochitest-browser-chrome-e10s-2 bc2
>    128 - test-linux64/debug-mochitest-browser-chrome-e10s-8 bc8
>    128 - test-linux64/debug-mochitest-browser-chrome-e10s-10 bc10
>    128 - test-linux64/debug-mochitest-plain-headless-e10s-9 h9
>    120 - test-linux64/debug-mochitest-e10s-7 7
>    120 - test-linux64/debug-mochitest-plain-headless-e10s-6 h6
>    120 - test-linux64/debug-mochitest-plain-headless-e10s-7 h7
>    120 - test-linux64/debug-mochitest-browser-chrome-e10s-9 bc9
>    112 - test-linux64/debug-mochitest-e10s-2 2
>    112 - test-linux64/debug-mochitest-e10s-3 3
>    112 - test-linux64/debug-mochitest-plain-headless-e10s-2 h2
>    112 - test-linux64/debug-mochitest-e10s-14 14
>    112 - test-linux64/debug-mochitest-plain-headless-e10s-3 h3
>    112 - test-linux64/debug-mochitest-plain-headless-e10s-14 h14
>    104 - test-linux64/debug-mochitest-browser-chrome-e10s-5 bc5
>    104 - test-linux64/debug-firefox-ui-functional-remote-e10s en-US
>    104 - test-linux64/debug-mochitest-browser-chrome-e10s-1 bc1
>    104 - test-linux64/debug-mochitest-plain-headless-e10s-13 h13
>    104 - test-linux64/debug-mochitest-e10s-13 13
>    104 - test-linux64/debug-mochitest-gpu-e10s gpu
>     96 - test-linux64/debug-mochitest-browser-chrome-e10s-6 bc6
>     80 - test-linux64/debug-mochitest-media-e10s-3 mda3
>     64 - test-linux64/debug-mochitest-devtools-chrome-e10s-3 dt3
>     56 - test-linux64/debug-reftest-stylo-e10s-1 Rs1
>     56 - test-linux64/debug-reftest-e10s-7 R7
>     56 - test-linux64/debug-reftest-stylo-e10s-8 Rs8
>     56 - test-linux64/debug-reftest-e10s-5 R5
>     56 - test-linux64/debug-jsreftest-e10s-3 J3
>     56 - test-linux64/debug-reftest-no-accel-e10s-1 Ru1
>     56 - test-linux64/debug-reftest-e10s-8 R8
>     56 - test-linux64/debug-reftest-no-accel-e10s-8 Ru8
>     56 - test-linux64/debug-reftest-no-accel-e10s-2 Ru2
>     56 - test-linux64/debug-reftest-stylo-e10s-4 Rs4
>     56 - test-linux64/debug-jsreftest-e10s-2 J2
>     56 - test-linux64/debug-jsreftest-e10s-1 J1
>     56 - test-linux64/debug-reftest-stylo-e10s-3 Rs3
>     56 - test-linux64/debug-reftest-e10s-6 R6
>     56 - test-linux64/debug-reftest-stylo-e10s-5 Rs5
>     56 - test-linux64/debug-reftest-no-accel-e10s-4 Ru4
>     56 - test-linux64/debug-reftest-no-accel-e10s-6 Ru6
>     56 - test-linux64/debug-reftest-no-accel-e10s-5 Ru5
>     56 - test-linux64/debug-reftest-stylo-e10s-7 Rs7
>     56 - test-linux64/debug-reftest-e10s-3 R3
>     56 - test-linux64/debug-reftest-no-accel-e10s-7 Ru7
>     56 - test-linux64/debug-reftest-e10s-4 R4
>     56 - test-linux64/debug-reftest-stylo-e10s-2 Rs2
>     56 - test-linux64/debug-crashtest-e10s C
>     56 - test-linux64/debug-reftest-e10s-1 R1
>     56 - test-linux64/debug-reftest-e10s-2 R2
>     56 - test-linux64/debug-reftest-stylo-e10s-6 Rs6
>     56 - test-linux64/debug-reftest-no-accel-e10s-3 Ru3
>     48 - test-linux64/debug-mochitest-devtools-chrome-e10s-2 dt2
>     48 - test-linux64/debug-mochitest-devtools-chrome-e10s-4 dt4
>     40 - test-linux64/debug-mochitest-devtools-chrome-e10s-5 dt5
>     40 - test-linux64/debug-mochitest-devtools-chrome-e10s-6 dt6
>     32 - test-linux64/debug-mochitest-media-e10s-2 mda2
>     32 - test-linux64/debug-mochitest-devtools-chrome-e10s-7 dt7
>     24 - test-linux64/debug-mochitest-browser-chrome-e10s-11 bc11
>     24 - test-linux64/debug-mochitest-a11y a11y
>     24 - test-linux64/debug-mochitest-webgl-e10s-1 gl1
>     24 - test-linux64/debug-mochitest-media-e10s-1 mda1
>     16 - test-linux64/debug-mochitest-webgl-e10s-3 gl3
>     16 - test-linux64/debug-mochitest-browser-chrome-e10s-14 bc14
>      8 - test-linux64/debug-mochitest-webgl-e10s-2 gl2
>      8 - test-linux64/debug-mochitest-browser-chrome-e10s-16 bc16
>      8 - test-linux64/debug-mochitest-devtools-chrome-e10s-8 dt8
>      8 - test-linux64/debug-mochitest-browser-chrome-e10s-15 bc15
>      8 - test-linux64/debug-mochitest-browser-chrome-e10s-13 bc13

It shows up in 1194 tests. A few of the most prevalent:

>   7592 - [e10s] Shutdown
>   1544 -        Shutdown
>    760 - [e10s] file:///builds/worker/workspace/build/tests/jsreftest/tests/jsreftest.html?test=NNNNNN/regress/regress-352208.js
>    736 - [e10s] file:///builds/worker/workspace/build/tests/jsreftest/tests/jsreftest.html?test=NNNNNN/extensions/unterminated-literal-error-location.js
>    552 - [e10s] /webdriver/tests/sessions/new_session/create_alwaysMatch.py
>    528 - [e10s] /webdriver/tests/sessions/new_session/create_firstMatch.py
>    400 - [e10s] /html/semantics/document-metadata/the-meta-element/the-lang-attribute-012.html
>    288 - [e10s] /service-workers/service-worker/detached-context.https.html
>    288 - [e10s] js/xpconnect/tests/marionette/test_loader_global_sharing.py TestLoaderGlobalSharing.test_global_sharing_settings
>    272 - [e10s] /service-workers/service-worker/activation.https.html

[1] https://hg.mozilla.org/mozilla-central/annotate/38b3c1d03a59/xpcom/threads/nsThreadManager.cpp#l517
This is similar to bug 1369002, we should probably just remove it.
Attached patch Remove verbose thread warning (obsolete) — Splinter Review
This warning doesn't seem to have much value, it just spews during shutdown.
Attachment #8950701 - Flags: review?(nfroyd)
Assignee: nobody → erahm
Status: NEW → ASSIGNED
This continues to be by far the most verbose warning during testing.
Flags: needinfo?(nfroyd)
Summary: 41,600 instances of "'!mMainThread'" emitted from xpcom/threads/nsThreadManager.cpp during linux64 debug testing → 58,400 instances of "'!mMainThread'" emitted from xpcom/threads/nsThreadManager.cpp during linux64 debug testing
Comment on attachment 8950701 [details] [diff] [review]
Remove verbose thread warning

I would really like to have some idea of who's calling this and why so we can evaluate whether this is a legitimate operation or whether we should fix the calling code to be smarter.
Flags: needinfo?(nfroyd)
Attachment #8950701 - Flags: review?(nfroyd)
(In reply to Nathan Froyd [:froydnj] from comment #4)
> Comment on attachment 8950701 [details] [diff] [review]
> Remove verbose thread warning
> 
> I would really like to have some idea of who's calling this and why so we
> can evaluate whether this is a legitimate operation or whether we should fix
> the calling code to be smarter.

This spews a lot during cycle collection shutdown [1]. I think maybe just loosening the warning to only warn if we're not the main thread might get rid of a fair amount of the warning output.

[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=a3bafd22b86ede5fd18681c609275aec2745cbf6
This is a lot noisier now.

> 327119 WARNING: '!mMainThread', file xpcom/threads/nsThreadManager.cpp, line 457

This warning [1] shows up in the following test suites:

>   7884 - test-linux64/debug-web-platform-tests-wdspec-sw-e10s-1 Wd1
>   7882 - test-linux64/debug-web-platform-tests-wdspec-e10s-1 Wd1
>   5535 - test-linux64/debug-web-platform-tests-wdspec-sw-e10s-2 Wd2
>   5535 - test-linux64/debug-web-platform-tests-wdspec-e10s-2 Wd2
>   4046 - test-linux64/debug-web-platform-tests-e10s-1 wpt1
>   4037 - test-linux64/debug-web-platform-tests-sw-e10s-1 wpt1
>   3987 - test-linux64/debug-web-platform-tests-e10s-14 wpt14
>   3978 - test-linux64/debug-web-platform-tests-sw-e10s-14 wpt14
>   3852 - test-linux64/debug-web-platform-tests-e10s-5 wpt5
>   3843 - test-linux64/debug-web-platform-tests-sw-e10s-5 wpt5
>   3807 - test-linux64/debug-web-platform-tests-sw-e10s-12 wpt12
>   3807 - test-linux64/debug-web-platform-tests-e10s-12 wpt12
>   3762 - test-linux64/debug-web-platform-tests-e10s-9 wpt9
>   3744 - test-linux64/debug-web-platform-tests-sw-e10s-9 wpt9
>   3735 - test-linux64/debug-web-platform-tests-sw-e10s-16 wpt16
>   3726 - test-linux64/debug-web-platform-tests-e10s-16 wpt16
>   3717 - test-linux64/debug-web-platform-tests-e10s-10 wpt10
>   3717 - test-linux64/debug-web-platform-tests-sw-e10s-10 wpt10
>   3710 - test-linux64/debug-web-platform-tests-sw-e10s-7 wpt7
>   3692 - test-linux64/debug-web-platform-tests-e10s-7 wpt7
>   3663 - test-linux64/debug-web-platform-tests-sw-e10s-3 wpt3
>   3645 - test-linux64/debug-web-platform-tests-e10s-3 wpt3
>   3636 - test-linux64/debug-web-platform-tests-sw-e10s-15 wpt15
>   3627 - test-linux64/debug-web-platform-tests-e10s-15 wpt15
>   3447 - test-linux64/debug-web-platform-tests-sw-e10s-4 wpt4
>   3447 - test-linux64/debug-web-platform-tests-e10s-4 wpt4
>   3425 - test-linux64/debug-web-platform-tests-sw-e10s-6 wpt6
>   3411 - test-linux64/debug-web-platform-tests-sw-e10s-11 wpt11
>   3411 - test-linux64/debug-web-platform-tests-e10s-6 wpt6
>   3411 - test-linux64/debug-web-platform-tests-e10s-11 wpt11
>   3216 - test-linux64/debug-web-platform-tests-sw-e10s-18 wpt18
>   3214 - test-linux64/debug-web-platform-tests-e10s-18 wpt18
>   3195 - test-linux64/debug-web-platform-tests-e10s-17 wpt17
>   3186 - test-linux64/debug-web-platform-tests-sw-e10s-17 wpt17
>   3168 - test-linux64/debug-web-platform-tests-e10s-2 wpt2
>   3159 - test-linux64/debug-web-platform-tests-sw-e10s-2 wpt2
>   3115 - test-linux64/debug-web-platform-tests-e10s-13 wpt13
>   3105 - test-linux64/debug-web-platform-tests-sw-e10s-13 wpt13
>   2918 - test-linux64/debug-reftest-e10s-6 R6
>   2918 - test-linux64/debug-reftest-sw-e10s-6 R6
>   2916 - test-linux64/debug-reftest-no-accel-e10s-6 Ru6
>   2916 - test-linux64/debug-reftest-no-accel-sw-e10s-6 Ru6
>   2864 - test-linux64/debug-reftest-no-accel-sw-e10s-7 Ru7
>   2864 - test-linux64/debug-reftest-no-accel-e10s-7 Ru7
>   2862 - test-linux64/debug-reftest-e10s-7 R7
>   2862 - test-linux64/debug-reftest-no-accel-sw-e10s-3 Ru3
>   2862 - test-linux64/debug-reftest-sw-e10s-1 R1
>   2862 - test-linux64/debug-reftest-e10s-1 R1
>   2862 - test-linux64/debug-reftest-sw-e10s-7 R7
>   2862 - test-linux64/debug-reftest-no-accel-sw-e10s-1 Ru1
>   2862 - test-linux64/debug-reftest-sw-e10s-3 R3
>   2862 - test-linux64/debug-reftest-e10s-3 R3
>   2862 - test-linux64/debug-reftest-no-accel-e10s-3 Ru3
>   2861 - test-linux64/debug-reftest-no-accel-e10s-1 Ru1
>   2754 - test-linux64/debug-reftest-e10s-5 R5
>   2754 - test-linux64/debug-reftest-e10s-2 R2
>   2754 - test-linux64/debug-reftest-no-accel-e10s-2 Ru2
>   2754 - test-linux64/debug-reftest-no-accel-sw-e10s-5 Ru5
>   2754 - test-linux64/debug-reftest-sw-e10s-5 R5
>   2754 - test-linux64/debug-reftest-no-accel-sw-e10s-2 Ru2
>   2754 - test-linux64/debug-reftest-no-accel-e10s-5 Ru5
>   2754 - test-linux64/debug-reftest-sw-e10s-2 R2
>   2709 - test-linux64/debug-web-platform-tests-e10s-8 wpt8
>   2709 - test-linux64/debug-web-platform-tests-sw-e10s-8 wpt8
>   2598 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-1 Wr1
>   2592 - test-linux64/debug-web-platform-tests-reftests-e10s-1 Wr1
>   2538 - test-linux64/debug-reftest-no-accel-sw-e10s-4 Ru4
>   2538 - test-linux64/debug-reftest-no-accel-e10s-4 Ru4
>   2538 - test-linux64/debug-reftest-sw-e10s-4 R4
>   2538 - test-linux64/debug-reftest-e10s-4 R4
>   2187 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-3 Wr3
>   2172 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-4 Wr4
>   2171 - test-linux64/debug-web-platform-tests-reftests-e10s-4 Wr4
>   2169 - test-linux64/debug-mochitest-chrome-1 c1
>   2169 - test-linux64/debug-web-platform-tests-reftests-e10s-3 Wr3
>   2164 - test-linux64/debug-web-platform-tests-reftests-e10s-6 Wr6
>   2152 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-6 Wr6
>   2151 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-5 Wr5
>   2142 - test-linux64/debug-mochitest-chrome-sw-1 c1
>   2139 - test-linux64/debug-web-platform-tests-reftests-e10s-5 Wr5
>   2100 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-2 Wr2
>   2099 - test-linux64/debug-web-platform-tests-reftests-e10s-2 Wr2
>   2088 - test-linux64/debug-mochitest-chrome-sw-2 c2
>   2061 - test-linux64/debug-mochitest-chrome-2 c2
>   2043 - test-linux64/debug-mochitest-chrome-sw-3 c3
>   2043 - test-linux64/debug-mochitest-chrome-3 c3
>   1908 - test-linux64/debug-marionette-headless-e10s MnH
>   1908 - test-linux64/debug-marionette-e10s Mn
>   1224 - test-linux64/debug-mochitest-clipboard-e10s cl
>   1215 - test-linux64/debug-mochitest-clipboard-sw-e10s cl
>   1053 - test-linux64/debug-mochitest-plain-headless-sw-e10s-15 h15
>   1044 - test-linux64/debug-mochitest-sw-e10s-15 15
>   1035 - test-linux64/debug-mochitest-plain-headless-e10s-15 h15
>    990 - test-linux64/debug-mochitest-e10s-15 15
>    756 - test-linux64/debug-firefox-ui-functional-local-e10s en-US
>    719 - test-linux64/debug-mochitest-browser-chrome-e10s-11 bc11
>    711 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-10 bc10
>    711 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-6 bc6
>    702 - test-linux64/debug-mochitest-browser-chrome-e10s-8 bc8
>    603 - test-linux64/debug-mochitest-browser-chrome-e10s-7 bc7
>    603 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-5 bc5
>    594 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-3 bc3
>    576 - test-linux64/debug-mochitest-browser-chrome-e10s-2 bc2
>    575 - test-linux64/debug-mochitest-browser-chrome-e10s-5 bc5
>    558 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-1 bc1
>    558 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-4 bc4
>    540 - test-linux64/debug-mochitest-browser-chrome-e10s-1 bc1
>    531 - test-linux64/debug-mochitest-browser-chrome-e10s-10 bc10
>    522 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-7 bc7
>    522 - test-linux64/debug-mochitest-browser-chrome-e10s-6 bc6
>    522 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-8 bc8
>    504 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-11 bc11
>    504 - test-linux64/debug-mochitest-browser-chrome-e10s-4 bc4
>    486 - test-linux64/debug-mochitest-sw-e10s-1 1
>    486 - test-linux64/debug-mochitest-plain-headless-e10s-1 h1
>    486 - test-linux64/debug-mochitest-plain-headless-sw-e10s-8 h8
>    486 - test-linux64/debug-mochitest-e10s-1 1
>    477 - test-linux64/debug-mochitest-e10s-8 8
>    477 - test-linux64/debug-mochitest-browser-chrome-e10s-3 bc3
>    477 - test-linux64/debug-mochitest-plain-headless-e10s-11 h11
>    477 - test-linux64/debug-mochitest-sw-e10s-5 5
>    477 - test-linux64/debug-mochitest-plain-headless-sw-e10s-11 h11
>    477 - test-linux64/debug-mochitest-e10s-5 5
>    477 - test-linux64/debug-mochitest-plain-headless-e10s-5 h5
>    477 - test-linux64/debug-mochitest-sw-e10s-11 11
>    468 - test-linux64/debug-mochitest-plain-headless-sw-e10s-4 h4
>    468 - test-linux64/debug-mochitest-sw-e10s-9 9
>    468 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-9 bc9
>    450 - test-linux64/debug-mochitest-plain-headless-e10s-9 h9
>    450 - test-linux64/debug-mochitest-e10s-16 16
>    450 - test-linux64/debug-mochitest-plain-headless-e10s-8 h8
>    441 - test-linux64/debug-mochitest-plain-headless-sw-e10s-1 h1
>    441 - test-linux64/debug-mochitest-sw-e10s-8 8
>    432 - test-linux64/debug-mochitest-plain-headless-sw-e10s-5 h5
>    432 - test-linux64/debug-mochitest-sw-e10s-4 4
>    432 - test-linux64/debug-mochitest-plain-headless-e10s-4 h4
>    432 - test-linux64/debug-mochitest-e10s-14 14
>    432 - test-linux64/debug-mochitest-e10s-4 4
>    432 - test-linux64/debug-test-verify-e10s TV
>    432 - test-linux64/debug-mochitest-e10s-10 10
>    423 - test-linux64/debug-mochitest-e10s-11 11
>    423 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-7 dt7
>    423 - test-linux64/debug-mochitest-sw-e10s-7 7
>    423 - test-linux64/debug-mochitest-e10s-7 7
>    414 - test-linux64/debug-mochitest-e10s-9 9
>    414 - test-linux64/debug-mochitest-plain-headless-e10s-7 h7
>    414 - test-linux64/debug-mochitest-plain-headless-sw-e10s-9 h9
>    414 - test-linux64/debug-mochitest-devtools-chrome-e10s-6 dt6
>    408 - test-linux64/debug-mochitest-plain-headless-sw-e10s-16 h16
>    405 - test-linux64/debug-mochitest-plain-headless-e10s-3 h3
>    405 - test-linux64/debug-mochitest-plain-headless-e10s-16 h16
>    405 - test-linux64/debug-mochitest-e10s-3 3
>    405 - test-linux64/debug-mochitest-plain-headless-sw-e10s-6 h6
>    405 - test-linux64/debug-mochitest-sw-e10s-16 16
>    405 - test-linux64/debug-mochitest-sw-e10s-3 3
>    396 - test-linux64/debug-mochitest-plain-headless-sw-e10s-10 h10
>    396 - test-linux64/debug-mochitest-plain-headless-e10s-10 h10
>    387 - test-linux64/debug-mochitest-plain-headless-sw-e10s-14 h14
>    387 - test-linux64/debug-mochitest-sw-e10s-10 10
>    387 - test-linux64/debug-mochitest-sw-e10s-14 14
>    387 - test-linux64/debug-mochitest-e10s-12 12
>    387 - test-linux64/debug-mochitest-plain-headless-e10s-14 h14
>    378 - test-linux64/debug-mochitest-plain-headless-sw-e10s-13 h13
>    378 - test-linux64/debug-mochitest-plain-headless-e10s-13 h13
>    369 - test-linux64/debug-mochitest-sw-e10s-13 13
>    369 - test-linux64/debug-mochitest-plain-headless-sw-e10s-3 h3
>    369 - test-linux64/debug-mochitest-plain-headless-sw-e10s-7 h7
>    360 - test-linux64/debug-mochitest-e10s-6 6
>    360 - test-linux64/debug-mochitest-sw-e10s-6 6
>    360 - test-linux64/debug-mochitest-plain-headless-e10s-6 h6
>    360 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-2 bc2
>    359 - test-linux64/debug-mochitest-browser-chrome-e10s-9 bc9
>    351 - test-linux64/debug-mochitest-gpu-sw-e10s gpu
>    351 - test-linux64/debug-mochitest-gpu-e10s gpu
>    342 - test-linux64/debug-mochitest-plain-headless-sw-e10s-12 h12
>    342 - test-linux64/debug-mochitest-plain-headless-e10s-12 h12
>    342 - test-linux64/debug-mochitest-sw-e10s-12 12
>    333 - test-linux64/debug-mochitest-e10s-13 13
>    324 - test-linux64/debug-mochitest-plain-headless-sw-e10s-2 h2
>    279 - test-linux64/debug-mochitest-plain-headless-e10s-2 h2
>    270 - test-linux64/debug-mochitest-e10s-2 2
>    270 - test-linux64/debug-mochitest-sw-e10s-2 2
>    252 - test-linux64/debug-mochitest-media-sw-e10s-3 mda3
>    252 - test-linux64/debug-mochitest-media-e10s-3 mda3
>    252 - test-linux64/debug-mochitest-devtools-chrome-e10s-7 dt7
>    216 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-1 dt1
>    207 - test-linux64/debug-mochitest-webgl1-core-sw-e10s gl1c
>    207 - test-linux64/debug-mochitest-webgl1-core-e10s gl1c
>    198 - test-linux64/debug-mochitest-devtools-chrome-e10s-1 dt1
>    198 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-2 dt2
>    189 - test-linux64/debug-mochitest-devtools-chrome-e10s-2 dt2
>    189 - test-linux64/debug-mochitest-devtools-chrome-e10s-4 dt4
>    189 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-5 dt5
>    189 - test-linux64/debug-firefox-ui-functional-remote-e10s en-US
>    144 - test-linux64/debug-mochitest-devtools-chrome-e10s-3 dt3
>    144 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-6 dt6
>    144 - test-linux64/debug-mochitest-devtools-chrome-e10s-8 dt8
>    144 - test-linux64/debug-mochitest-devtools-chrome-e10s-5 dt5
>    144 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-4 dt4
>    144 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-8 dt8
>    119 - test-linux64/debug-mochitest-media-sw-e10s-2 mda2
>    118 - test-linux64/debug-mochitest-media-e10s-2 mda2
>    108 - test-linux64/debug-reftest-e10s-8 R8
>    108 - test-linux64/debug-reftest-no-accel-sw-e10s-8 Ru8
>    108 - test-linux64/debug-reftest-no-accel-e10s-8 Ru8
>    108 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-3 dt3
>    108 - test-linux64/debug-reftest-sw-e10s-8 R8
>    108 - test-linux64/debug-mochitest-media-sw-e10s-1 mda1
>    108 - test-linux64/debug-mochitest-media-e10s-1 mda1
>     81 - test-linux64/debug-telemetry-tests-client-e10s c
>     72 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-15 bc15
>     72 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-16 bc16
>     72 - test-linux64/debug-mochitest-browser-chrome-e10s-16 bc16
>     72 - test-linux64/debug-mochitest-browser-chrome-e10s-15 bc15
>     54 - test-linux64/debug-jsreftest-sw-e10s-1 J1
>     54 - test-linux64/debug-jsreftest-sw-e10s-3 J3
>     54 - test-linux64/debug-jsreftest-e10s-3 J3
>     54 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-12 bc12
>     54 - test-linux64/debug-jsreftest-e10s-1 J1
>     54 - test-linux64/debug-mochitest-a11y-sw a11y
>     54 - test-linux64/debug-mochitest-a11y a11y
>     54 - test-linux64/debug-jsreftest-e10s-2 J2
>     54 - test-linux64/debug-crashtest-sw-e10s C
>     54 - test-linux64/debug-jsreftest-sw-e10s-2 J2
>     54 - test-linux64/debug-mochitest-browser-chrome-e10s-12 bc12
>     54 - test-linux64/debug-crashtest-e10s C
>     45 - test-linux64/debug-mochitest-browser-chrome-e10s-13 bc13
>     45 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-13 bc13
>     36 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-9 dt9
>     36 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-12 dt12
>     36 - test-linux64/debug-mochitest-browser-chrome-e10s-14 bc14
>     36 - test-linux64/debug-mochitest-devtools-chrome-e10s-9 dt9
>     36 - test-linux64/debug-mochitest-webgl1-ext-e10s gl1e
>     36 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-14 bc14
>     36 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-10 dt10
>     36 - test-linux64/debug-mochitest-devtools-chrome-e10s-12 dt12
>     36 - test-linux64/debug-mochitest-devtools-chrome-e10s-10 dt10
>     36 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-11 dt11
>     36 - test-linux64/debug-mochitest-devtools-chrome-e10s-11 dt11

It shows up in 2159 tests. A few of the most prevalent:

>  47040 - [e10s] Shutdown
>  10584 -        Shutdown
>   2430 - [e10s] /webdriver/tests/new_session/create_firstMatch.py
>   1548 -        browser/components/shell/test/test_headless_screenshot.html
>    540 - [e10s] /webdriver/tests/get_page_source/user_prompts.py
>    540 - [e10s] /webdriver/tests/find_elements_from_element/user_prompts.py
>    540 - [e10s] /webdriver/tests/find_element_from_element/user_prompts.py
>    540 - [e10s] /webdriver/tests/get_element_css_value/user_prompts.py
>    540 - [e10s] /webdriver/tests/get_element_tag_name/user_prompts.py
>    540 - [e10s] /webdriver/tests/element_clear/user_prompts.py

[1] https://hg.mozilla.org/mozilla-central/annotate/619d084f0891/xpcom/threads/nsThreadManager.cpp#l457
Summary: 58,400 instances of "'!mMainThread'" emitted from xpcom/threads/nsThreadManager.cpp during linux64 debug testing → 327,100 instances of "'!mMainThread'" emitted from xpcom/threads/nsThreadManager.cpp during linux64 debug testing
This limits the warning to the !NS_IsMainThread case.
Attachment #9034514 - Flags: review?(nfroyd)
Attachment #8950701 - Attachment is obsolete: true
Attachment #9034514 - Flags: review?(nfroyd) → review+
Pushed by erahm@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a70165566e3b
Reduce scope of GetMainThread warning. r=froydnj
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.