1193365 - Nightly spams lots of output to terminal about PushService.jsm, PushServiceWebSocket.jsm, PushDB.jsm (now that dom.push.debug is enabled)

Status: RESOLVED FIXED

(Core :: DOM: Push Subscriptions, defect)

Description

[Daniel Holbert [:dholbert]]

STR:
 1. Start Firefox with a fresh profile, e.g. run this:
      mkdir /tmp/foo1; firefox -profile /tmp/foo1 -no-remote
 2. Watch your terminal.

ACTUAL RESULTS:
A few addons.xpi logging messages, followed by this new output:
{
-*- PushService.jsm: init()
-*- PushService.jsm: new state: 2 old state: 0
-*- PushService.jsm: init()
-*- PushService.jsm: changeServerURL
-*- PushService.jsm: startService
-*- PushServiceWebSocket.jsm: init()
-*- PushService.jsm: startObservers
-*- PushService.jsm: changeStateConnectionEnabledEvent: true
-*- PushService.jsm: changeStateOfflineEvent: false
-*- PushDB.jsm: getAllUnexpired()
-*- PushDB.jsm: getAllByPushQuota()
-*- PushService.jsm: new state: 5 old state: 2
-*- PushService.jsm: receiveMessage(): Push:RegisterEventNotificationListener
-*- PushService.jsm: Adding child listener
}

EXPECTED RESULTS: No such logging (by default, in nightly builds) unless I opted in to see it with a pref.

It looks like this logging is controlled by the pref dom.push.debug, which was recently enabled in bug 1153499, with the other Push-related prefs.

:nsm, do we really need to have this "debug" pref enabled by default? (in opt nightly builds)

Comment 1

[Daniel Holbert [:dholbert]]

(Note that Nightly builds are generally pretty silent in terms of logging/terminal-spew, which is why this stood out to me.)

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Attachment: MozReview Request: Bug 1193365 - Disable push debug. r?kitcambridge

Bug 1193365 - Disable push debug. r?kitcambridge

Comment 3

[Lina Butler [:lina]]

Comment on attachment 8646544 [details]
MozReview Request: Bug 1193365 - Disable push debug. r?kitcambridge

https://reviewboard.mozilla.org/r/15795/#review14075

Ship It!

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4fd08002000bf48aaa6b0113566ea55b83c39f6c
changeset:  4fd08002000bf48aaa6b0113566ea55b83c39f6c
user:       Nikhil Marathe <[e-mail]>
date:       Tue Aug 11 12:53:14 2015 -0700
description:
Bug 1193365 - Disable push debug. r=kitcambridge

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/4fd08002000b

Comment 6

[Daniel Holbert [:dholbert]]

Looks like this affects DevEdition 42 -- e.g. my current DevEdition version, which is 42.0a2 (2015-08-17).

(Looks like bug 1153499 landed before the aurora merge, vs. the fix on this bug here landed after the merge.)

nsm, can you request backport approval to reduce default-terminalspew on DevEdition?

Comment 7

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8646544 [details]
MozReview Request: Bug 1193365 - Disable push debug. r?kitcambridge

Approval Request Comment
[Feature/regressing bug #]: Bug 1153499
[User impact if declined]: A lot of unnecessary logging information is printed on the console by default. This is an annoyance to developers and interferes with investigating other issues. There is no end-user impact.
[Describe test coverage new/current, TreeHerder]: None, this just switches off logging.
[Risks and why]: None
[String/UUID change made/needed]: None

Comment 8

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8646544 [details]
MozReview Request: Bug 1193365 - Disable push debug. r?kitcambridge

Less debug messages is better, taking it.

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/044e019a13a9