1320348 - "PresentationControlService.js: PresentationControlService - close" appears in all mochitest output

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Florian Quèze [:florian]]

When I run locally the browser chrome mochitest I'm currently writing, I have this line at the end of the output:

-*- PresentationControlService.js: PresentationControlService - close


Either http://searchfox.org/mozilla-central/rev/9a3ab17838f039aab023837d905115f8990e442e/dom/presentation/provider/PresentationControlService.js#319 should be removed (it doesn't seem very useful even for debugging), or dom.presentation.tcp_server.debug should only be set to true in the tests that are relevant to PresentationControlService.js instead of globally at http://searchfox.org/mozilla-central/rev/9a3ab17838f039aab023837d905115f8990e442e/testing/profiles/prefs_general.js#237

Comment 1

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

I can remove the global pref and only turn on it in related test cases.

Comment 2

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Attachment: Bug 1320348 - disable Presentation API logging on treeherder.

Review commit: https://reviewboard.mozilla.org/r/95878/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/95878/

Comment 3

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Comment on attachment 8814774 [details]
Bug 1320348 - disable Presentation API logging on treeherder.

https://reviewboard.mozilla.org/r/95878/#review95870

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cbd4fb9f368e
disable Presentation API logging on treeherder. r=schien

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/cbd4fb9f368e