1047572 - Mulet exposed Intl ECMA support when b2g does not

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Runtime, defect)

Description

[Ben Kelly [:bkelly, not reviewing]]

After enabling mulet builds today we say this:

1565 INFO TEST-UNEXPECTED-FAIL | /tests/dom/workers/test/test_worker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface Intl to all webpages as a property on the worker? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals)

Presumably this would fail in /tests/dom/tests/mochitest/general/test_interfaces.html as well for the main thread if it wasn't disabled for mulet.

According to #jsapi the Intl support is disabled in b2g.  The test checks for this by looking for mobile user agent strings.

If Mulet is enabling Intl support when b2g is not, then this is a bug in Mulet's emulation of the b2g platform.

As this is a new build just working out the kinks, we plan to hide Mulet M4 until this can be fixed.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Aldo, didn't know what category to put this in.  If anyone knows, please fix.

Comment 2

[Alexandre Poirot [:ochameau]]

Mulet is a hybrid product between b2g and Firefox, so that there is some side effects like this.
There is multiple issues here.
The user agent for mulet is: "Mozilla/5.0 (X11; Mobile; rv:33.0) Gecko/20100101 Firefox/33.0"
So that isDesktop and isB2G are both false.
We would need to tweak these tests to behave sanely on Mulet.

Otherwise, we can try to disable Intl on Mulet, if Firefox still works without it.

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Disable intl on mulet

Doesn't seem to break anything...
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=33f7f3d6545b

Comment 4

[Alexandre Poirot [:ochameau]]

Comment on attachment 8538515 [details] [diff] [review]
Disable intl on mulet

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

It doesn't seem to break anything on tests, nor when using it...
So just do whatever we do for b2g.

Comment 5

[Andrew Halberstadt [:ahal]]

Comment on attachment 8538515 [details] [diff] [review]
Disable intl on mulet

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

Looks trivial enough to r+, but I'm not a build peer so don't think I'm technically supposed to do that.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/9c81cf0a3ebe

Comment 7

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

https://hg.mozilla.org/mozilla-central/rev/9c81cf0a3ebe