Closed
Bug 1204397
Opened 9 years ago
Closed 9 years ago
NIGHTLY vs. !RELEASE mismatch enabling ServiceWorkers on Android
Categories
(Core :: DOM: Service Workers, defect)
Core
DOM: Service Workers
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: philor, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file, 1 obsolete file)
Bug 1203160 enabled ServiceWorkers for Fennec #ifdef NIGHTLY_BUILD, which is true if the version number ends in a1, and told the interface tests to expect them to be enabled if !isRelease, and isRelease is true when there is no a in the version number. So on aurora, where the version number ends in a2, they are not enabled but the two interface tests expect them to be enabled, https://treeherder.mozilla.org/#/jobs?repo=try&revision=6a5c1cfcea73&filter-searchStr=android%20mochitest The merge is September 21st.
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8660756 -
Flags: review?(amarchesini)
Updated•9 years ago
|
Status: NEW → ASSIGNED
Updated•9 years ago
|
Attachment #8660756 -
Flags: review?(amarchesini) → review+
Reporter | ||
Comment 4•9 years ago
|
||
Maybe I've screwed up the patch to do central-as-aurora, since I haven't been the one to do it for years, but https://treeherder.mozilla.org/#/jobs?repo=try&revision=e8cdb09267b6 still fails with that.
Reporter | ||
Comment 5•9 years ago
|
||
Oh, no, that patch is broken, because it changed from saying "not enabled on release" to saying "not enabled on nightly" instead of "not enabled on non-nightly"
Both backed out for bustage: https://treeherder.mozilla.org/logviewer.html#?job_id=14076757&repo=mozilla-inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/165e0853088e
Flags: needinfo?(ehsan)
Assignee | ||
Comment 8•9 years ago
|
||
Ah, the logic here is pretty complex. I think we should look for nonNightlyAndroid, and make these interfaces nonNightlyAndroid: false. That way they are enabled everywhere except for nonNightlyAndroid, which matches what we are trying to achieve here, I think. Trying that here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4d9fc219c1c3
Flags: needinfo?(ehsan)
Assignee | ||
Comment 9•9 years ago
|
||
Looks like I finally got the logic right! <https://treeherder.mozilla.org/#/jobs?repo=try&revision=2a1108b66a9e>
Assignee | ||
Comment 10•9 years ago
|
||
The semantics of the new attributes are as follows: * nightlyAndroid only takes effect on Android. * If both nightlyAndroid and android are specified, nightlyAndroid takes * precedence. These two rules allow us to check for the current situation.
Attachment #8660756 -
Attachment is obsolete: true
Attachment #8661764 -
Flags: review?(amarchesini)
Updated•9 years ago
|
Attachment #8661764 -
Flags: review?(amarchesini) → review+
Comment 12•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/65cec42b60fa
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•