1386644 - Fix and enable the mochitest for bug 1053321 on Android

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P2)

Description

[Honza Bambas (:mayhemer)]

Before we have cycles to fix bug 1386194 it would be good to land the test at https://bugzilla.mozilla.org/attachment.cgi?id=8892597&action=edit disabled on android.  See https://bugzilla.mozilla.org/show_bug.cgi?id=1053321#c23 and c25 for details.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Note that we don't run wpt on Android yet anyway, so mochitest coverage is beneficial in that respect too.

Comment 2

[Honza Bambas (:mayhemer)]

I believe this has been fixed with Bug 1392382 and can now be enabled.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4943c9c6243b6f5875eb65f47a0b71dc2f2235e5

Comment 3

[Honza Bambas (:mayhemer)]

Still failing, trying more:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=315c1f1fc0a265089ef9fb82c34a079b78140b17&filter-searchStr=tc-M

Comment 4

[Honza Bambas (:mayhemer)]

The problem here is that from some reason we do the "unblock" requests two times on Android, always.  No idea why, but the sjs code has to count with that.

Comment 5

[Honza Bambas (:mayhemer)]

Attachment: v1

- there is no guarantee that an async scripts will run before onload in this case, fixed with explicit wait for both scripts to execute ; this was the root cause
- the sjs script is protected against duplicate requests for unblocking ; was another problem, but not root cause of the perma-failure

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac23623d6a5a0ff0e0c02d71db60e27804d9f337

Comment 6

[Honza Bambas (:mayhemer)]

thanks!

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7fba47d2119b
Enable test for bug 1053321 on Android, r=bkelly

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/7fba47d2119b