1027372 - runNetworkTests break steeplechase tests

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Description

[Nils Ohlmeier [:drno]]

With the use of runNetworkTest() steeplechase test execution fails, because info() is missing.

Comment 1

[Nils Ohlmeier [:drno]]

Attachment: bug_1027372_fix_info.patch

Added a wrapper around info() which first checks if that function is available.

Try run: https://tbpl.mozilla.org/?tree=Try&rev=9330138aebcf

Comment 2

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

Comment on attachment 8442490 [details] [diff] [review]
bug_1027372_fix_info.patch

Review of attachment 8442490 [details] [diff] [review]:
-----------------------------------------------------------------

Would prefer safeInfo() over infoSafe(), along with a function comment explaining why the wrapper's there. However, r+ either way.

Comment 3

[Nils Ohlmeier [:drno]]

Attachment: bug_1027372_fix_info.patch

Updated according to geo's feedback.

Carrying forward r+=geo

Comment 4

[Nils Ohlmeier [:drno]]

No new problem in the  try run.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3b965d005fb3

Comment 6

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

https://hg.mozilla.org/mozilla-central/rev/3b965d005fb3