888303 - Please disable the coordinate logging

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Zac C (:zac)]

The coordinate logged introduced in this bug[0] is printing directly into the console and polluting our test results because it is enabled by default.

It's fairly unanimous that we want this turned off. 

I think the original intention may have been for it to be printed into logcat which would make more sense as it could be compared directly with the Marionette JSON request that triggered it and in realtime, too. 

[0] https://bugzilla.mozilla.org/show_bug.cgi?id=883299

Comment 1

[Bob Silverberg [:bsilverberg]]

The logging is also going to logcat, which I think is fine. I believe this request is just to disable it being output directly to the console. Do you agree Zac?

Comment 2

[Zac C (:zac)]

Yes just console-level logging :)

Comment 3

[Jonathan Griffin (:jgriffin)]

Canada is on holiday today, so I'll take care of this.

Comment 4

[Jonathan Griffin (:jgriffin)]

Pushed without review: https://hg.mozilla.org/integration/mozilla-inbound/rev/5d89e929edd9

Comment 5

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Thanks Jonathan. I'll raise a bug to reduce log output to stdout, and another to re-enable this logging once the former is resolved.

Comment 6

[Dave Hunt [:davehunt] [he/him] ⌚BST]

(In reply to Dave Hunt (:davehunt) from comment #5)
> Thanks Jonathan. I'll raise a bug to reduce log output to stdout, and
> another to re-enable this logging once the former is resolved.

See bug 889288 and bug 889296

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/5d89e929edd9

Comment 8

[Jonathan Griffin (:jgriffin)]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/865fc7bd88ed

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/865fc7bd88ed