1457326 - Add coverage status to getBuildConfiguration

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement)

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Attachment: patch

It will be nice to know if a js shell was compiled with --enable-coverage. I based this patch off bug 998596.

Jan, do you mind checking if this patch works as expected?

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Also, I'm not sure whether the ordering of items in getBuildConfiguration matters or not...

Comment 2

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8971428 [details] [diff] [review]
patch

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

Seems straightforward enough. Looks good to me.

Comment 3

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Attachment: patch v2

Bringing over r+. This version sets the right commit message.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d9515d63a7cf
Add coverage status to getBuildConfiguration, r=sfink

Comment 5

[Natalia Csoregi [:nataliaCs]]

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