1026987 - Give the MOZ_DISABLE_NONLOCAL_CONNECTIONS error a TBPL parsable prefix

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Ed Morley [:emorley]]

Currently the error printed for MOZ_DISABLE_NONLOCAL_CONNECTIONS transgressions ends up being hard to find on some platforms - particularly those where it's only buried in the logcat some lines away from the crash (or in the case of some suites; hang since they don't handle crashes properly... facepalm):

eg
https://tbpl.mozilla.org/php/getParsedLog.php?id=41944900&full=1&branch=try

02:29:23     INFO -  06-18 02:28:15.453 I/Gecko   ( 2177): Non-local network connections are disabled and a connection attempt to www.google.com (74.125.239.144) was made.  You should only access hostnames available via the test networking proxy (if running mochitests) or from a test-specific httpd.js server (if running xpcshell tests).  Browser services should be disabled or redirected to a local server.

We should use one of the strings recognised by the parser:
https://hg.mozilla.org/webtools/tbpl/file/tip/php/inc/GeneralErrorFilter.php#l37
...which is soon hopefully going to have "ERROR: " added (in bug 1018910) - which is perhaps more suitable than something too test-centric like "TEST-UNEXPECTED-FAIL" or "Automation Error:".

Comment 1

[Ryan VanderMeulen [:RyanVM]]

I could be talked into writing a patch borrowing "fatal error" from line 44.

Comment 2

[Ed Morley [:emorley]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #1)
> I could be talked into writing a patch borrowing "fatal error" from line 44.

Waiting to see how bug 1035773 and thus bug 1018910 pans out :-)

Comment 3

[Ed Morley [:emorley]]

Attachment: Give the MOZ_DISABLE_NONLOCAL_CONNECTIONS error a TBPL-parsable prefix

Bug 1018910 added TBPL support for the new "FATAL ERROR" style prefixes that are now being used by failure cases where an automation related string would be inappropriate (eg bug 1035773), so let's just use that. I've also broken the error string across two lines, so that the bugzilla search term only includes the first, more relevant line.

Comment 4

[Ed Morley [:emorley]]

Ty :-)

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/8dd44718c3c5

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/8dd44718c3c5

Comment 6

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

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

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/b91fc106f82b
https://hg.mozilla.org/releases/mozilla-beta/rev/92aead6bd5fb
https://hg.mozilla.org/releases/mozilla-esr31/rev/5c56c57043a2
https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/79a3790888d2