[webvr] Implement Reftest support for WebVR functionality

RESOLVED FIXED in Firefox 55

Status

()

defect
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: kip, Assigned: daoshengmu)

Tracking

(Blocks 1 bug)

Trunk
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 affected, firefox55 fixed)

Details

(Whiteboard: [gfx-noted][webvr])

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Description

4 years ago
No description provided.
Reporter

Updated

4 years ago
Depends on: 1229480
Whiteboard: [gfx-noted]
Whiteboard: [gfx-noted] → [gfx-noted][webvr]
Reporter

Updated

2 years ago
Component: Graphics → WebVR
Assignee

Updated

2 years ago
Assignee: nobody → dmu
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Updated

2 years ago
Attachment #8865385 - Attachment is obsolete: true
Assignee

Comment 7

2 years ago
(In reply to Daosheng Mu[:daoshengmu] from comment #6)
> Comment on attachment 8865386 [details]
> Bug 1229481 - Part 2: Add draw rect reftest for WebVR;
> 
> Review request updated; see interdiff:
> https://reviewboard.mozilla.org/r/137058/diff/2-3/

The VRSimulationDriver.js can't be included from mochitest/ because our reftest toolchain can not get it when we run it globally. So, I copy it mochitest/ Moreover, I notice draw_rect.html will wait lots of time for finishing when running in reftest Nightly Debug, but running it without reftest, it is good. I wanna skip it for now and keep investigating how to solve it in reftest Nightly Debug environment.

Try looks good,
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7fc01cfdc0d2f7972843c9e79a34b11b4d51a78e
Assignee

Comment 8

2 years ago
(In reply to Daosheng Mu[:daoshengmu] from comment #7)

> Try looks good,
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=7fc01cfdc0d2f7972843c9e79a34b11b4d51a78e

BTW, the try result for VR is at (R) and (R-e10s).
Reporter

Comment 9

2 years ago
mozreview-review
Comment on attachment 8865384 [details]
Bug 1229481 - Part 1: Move VR mochitest to its subfolder;

https://reviewboard.mozilla.org/r/137054/#review148062

Looks good, thanks!
Attachment #8865384 - Flags: review?(kgilbert) → review+
Reporter

Comment 10

2 years ago
mozreview-review
Comment on attachment 8865386 [details]
Bug 1229481 - Part 2: Add draw rect reftest for WebVR;

https://reviewboard.mozilla.org/r/137058/#review148100

This looks good, except for the readability of reftest.list.

r=me if you can move the image from the base64 data to a file.

::: dom/vr/test/reftest/reftest.list:7
(Diff revision 3)
> +default-preferences pref(dom.vr.puppet.enabled,true) pref(dom.vr.test.enabled,true) pref(dom.vr.require-gesture,false) pref(dom.vr.puppet.submitframe,1)
> +
> +# VR SubmitFrame is only implemented for D3D11 now.
> +# We need to continue to investigate why these reftests can be run well in local,
> +# but will be suspended until terminating on reftest debug build.
> +skip-if(!winWidget||!layersGPUAccelerated||isDebugBuild) == draw_rect.html wrapper.html?data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAQAAAAEACAYAAABccqhmAAAGmklEQVR4nO3RsQ0AMAzDsPz/dHsGBwuEdiO5e0lm8QFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckcfiAJA4fkMThA5I4fEAShw9I4vABSRw+IInDByRx+IAkDh+QxOEDkjh8QBKHD0ji8AFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckcfiAJA4fkMThA5I4fEAShw9I4vABSRw+IInDByRx+IAkDh+QxOEDkjh8QBKHD0ji8AFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckcfiAJA4fkMThA5I4fEASRw94t906fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j9AD9AN06fX8dpwfoB+jW6fvrOD1AP0C3Tt9fx+kB+gG6dfr+Ok4P0A/QrdP313F6gH6Abp2+v47TA/QDdOv0/XWcHqAfoFun76/j+IAkDh+QxOEDkjh8QBKHD0ji8AFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckcfiAJA4fkMThA5I4fEAShw9I4vABSRw+IInDByRx+IAkDh+QxOEDkjh8QBKHD0ji8AFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckcfiAJA4fkMThA5I4fEAShw9I4vABSRw+IInDByRx+IAkDh+QxOEDkjh8QBKHD0ji8AFJHD4gicMHJHH4gCQOH5DE4QOSOHxAEocPSOLwAUkcPiCJwwckYT4olR3j8YEJmwAAAABJRU5ErkJggg==

This could become difficult to read once more tests are added.

Perhaps we could move the image to a file rather than passing it as base64 data?
Attachment #8865386 - Flags: review?(kgilbert) → review+
Comment hidden (mozreview-request)
Assignee

Comment 12

2 years ago
mozreview-review-reply
Comment on attachment 8865386 [details]
Bug 1229481 - Part 2: Add draw rect reftest for WebVR;

https://reviewboard.mozilla.org/r/137058/#review148100

> This could become difficult to read once more tests are added.
> 
> Perhaps we could move the image to a file rather than passing it as base64 data?

Good suggestion. Thank!

Comment 13

2 years ago
Pushed by dmu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c13bf6be43e
Part 1: Move VR mochitest to its subfolder; r=kip
https://hg.mozilla.org/integration/autoland/rev/bce41e33bffd
Part 2: Add draw rect reftest for WebVR; r=kip

Comment 14

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7c13bf6be43e
https://hg.mozilla.org/mozilla-central/rev/bce41e33bffd
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.