Closed
Bug 1426500
Opened 6 years ago
Closed 6 years ago
Enable WebVR for macOS only on Nightly and Dev-Edition
Categories
(Core :: WebVR, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla58
People
(Reporter: kip, Assigned: kip)
References
Details
Attachments
(2 files)
59 bytes,
text/x-review-board-request
|
daoshengmu
:
review+
gchang
:
approval-mozilla-beta+
gchang
:
approval-mozilla-release+
|
Details |
3.65 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
The webvr pref, dom.vr.enabled, needs to be disabled for beta and release for the macOS platform. This is due to inability to QA macOS support for WebVR on currently available hardware. macOS support for WebVR will remain enabled on Firefox nightly and dev-edition. Support for WebVR on Windows will remain enabled in all 64-bit desktop Firefox builds, including release.
Assignee | ||
Comment 1•6 years ago
|
||
Note that the macOS support will be enabled for dev-edition, but not Beta. These changes will be reverted once QA with supported configurations becomes possible. It will also be possible to manually enable WebVR in release builds in about:config by flipping dom.vr.enabled
Assignee | ||
Comment 2•6 years ago
|
||
SoftVision has not yet been able to verify for QA due to technical issues with the macOS devkit hardware. As FF58 is about to merge, I am prioritizing this.
Priority: -- → P1
Target Milestone: --- → mozilla58
Version: 59 Branch → 58 Branch
Comment hidden (mozreview-request) |
Assignee | ||
Updated•6 years ago
|
Attachment #8941669 -
Flags: review?(dmu)
Assignee | ||
Comment 4•6 years ago
|
||
This patch applies cleanly on Beta and can be used for beta-uplift
Assignee | ||
Comment 5•6 years ago
|
||
I will request beta-uplift as soon as the try run passes with green and have r+.
Comment 6•6 years ago
|
||
mozreview-review |
Comment on attachment 8941669 [details] Bug 1426500 - Disable WebVR for macOS on Beta and Release https://reviewboard.mozilla.org/r/211908/#review217700 lgtm. Thanks
Attachment #8941669 -
Flags: review?(dmu) → review+
Pushed by kgilbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/78533a254124 Disable WebVR for macOS on Beta and Release r=daoshengmu
Assignee | ||
Comment 8•6 years ago
|
||
Comment on attachment 8941669 [details] Bug 1426500 - Disable WebVR for macOS on Beta and Release Approval Request Comment [Feature/Bug causing the regression]: Bug 1374399 - Enable WebVR by default on macOS [User impact if declined]: WebVR will be enabled on macOS in release Firefox 59. Although this is working well on our development machines, it has not been verified by SoftVision due to hardware issues (eGPU DevKit enclosure supplied by Apple is crashing their test machines.) [Is this code covered by automated tests?]: Yes. The automated reftets, mochitests, and web-platform-tests for WebVR will continue to run. [Has the fix been verified in Nightly?]: WebVR will continue to be enabled in Nightly and Dev Edition, but will be disabled in Release and Beta until the SoftVision QA can be completed. [Needs manual test from QE? If yes, steps to reproduce]: N/A - Simply ensuring that functionality does not become enabled in release. [List of other uplifts needed for the feature/fix]: N/A [Is the change risky?]: Low Risk. [Why is the change risky/not risky?]: Change reduces risk by delaying functionality that has not been shipped in release until QA verification. [String changes made/needed]: N/A
Attachment #8941669 -
Flags: approval-mozilla-beta?
Comment 9•6 years ago
|
||
Comment on attachment 8941669 [details] Bug 1426500 - Disable WebVR for macOS on Beta and Release Take this to enable WebVR for macOS only on Nightly and Dev-Edition. Beta58+.
Attachment #8941669 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•6 years ago
|
status-firefox58:
--- → affected
Comment 10•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/76192bc7ded1
Comment 11•6 years ago
|
||
Backed out from beta for failing mochitest dom/tests/mochitest/general/test_interfaces.html | VRDisplay should be defined on the global scope: https://hg.mozilla.org/releases/mozilla-beta/rev/8bb8f895a740ba60d587337e920f30ec3000c4ca Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=76192bc7ded10f07dd8eb314af1b268d2b0af248&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=155596907&repo=mozilla-beta TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | VRDisplay should be defined on the global scope
Flags: needinfo?(kgilbert)
Comment 12•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/78533a254124
Assignee | ||
Comment 13•6 years ago
|
||
This patch should fix the test failure that caused the beta-uplift to be rolled back. It will need to be applied to central as well to ensure that the same tests don't fail after merging central to beta.
Comment 14•6 years ago
|
||
Comment on attachment 8941669 [details] Bug 1426500 - Disable WebVR for macOS on Beta and Release As we passed the first merge, this needs to be release branch, too.
Attachment #8941669 -
Flags: approval-mozilla-release+
Updated•6 years ago
|
Attachment #8942006 -
Flags: review+
Comment 16•6 years ago
|
||
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/mozilla-central/rev/2a78b2a40796 Part 2: Update test_interfaces mochitest to expect WebVR interfaces to be disabled on release builds for macOS. r=smaug, a=me
Assignee | ||
Comment 17•6 years ago
|
||
It appears beta and release uplift approvals have been set, thanks! Please be certain to include both patches to avoid test failures in beta and release.
Flags: needinfo?(kgilbert)
Comment 18•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/52f3fbce290687befb442e913faf57763bdf6814 (FIREFOX_58b_RELBRANCH) https://hg.mozilla.org/releases/mozilla-release/rev/ace563efaa99 https://hg.mozilla.org/releases/mozilla-beta/rev/50afadf44520a92711c6e65ff424a300aa0cda4b (FIREFOX_58b_RELBRANCH) https://hg.mozilla.org/releases/mozilla-release/rev/1f97fc614f9e
Comment 19•6 years ago
|
||
I verified the pref code (dom.vr.enable) accross some builds on macOS X 10.13.3 Beta. Here are the results: - Fx 58.0 - disabled - Fx 58.0b16 - enabled (the uplift was after the merge) - Fx 59.0b1 (Developer Edition) - disabled (the aurora channel should be enabled) - Fx 59.0a1 - enabled Will the pref code be disabled on the aurora channel until Fx 59.0b3(non-developer edition) will be compiled ? Thank you!
Flags: needinfo?(kgilbert)
Assignee | ||
Comment 20•6 years ago
|
||
(In reply to Cristian Comorasu [:CristiComo] from comment #19) > I verified the pref code (dom.vr.enable) accross some builds on macOS X > 10.13.3 Beta. Here are the results: > > - Fx 58.0 - disabled > - Fx 58.0b16 - enabled (the uplift was after the merge) > - Fx 59.0b1 (Developer Edition) - disabled (the aurora channel should be > enabled) > - Fx 59.0a1 - enabled > > Will the pref code be disabled on the aurora channel until Fx > 59.0b3(non-developer edition) will be compiled ? > > Thank you! Thanks for verifying, Cristian! The enabled state for macOS would be the union of dom.vr.enabled and dom.vr.openvr.enabled. Priority is to disable in Fx 58 for release, so should be okay there. For Fx 59, intention was to have it enabled for DevEdition but not on Beta or release. (The patch looks for RELEASE_OR_BETA and disables on macOS if set).
Flags: needinfo?(kgilbert)
You need to log in
before you can comment on or make changes to this bug.
Description
•