Add OMTA test API support in PWebRenderBridge

RESOLVED FIXED in Firefox 56

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

Other Branch
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [gfx-noted])

Attachments

(6 attachments)

The OMTA mochitests use some functions in DOMWindowUtils that currently only work over PLayerTransaction. We should make them work over PWebRenderBridge as well.
Without these patches: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2f61202ad0246f2fe281b044f92d0ab8c2eb501d&selectedJob=108980831

With these patches: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e6b4a8fc5c0f48fc15e6d5549f8b651977507cd5&filter-searchStr=mochitest&selectedJob=109220230

Most of the OMTA mochitests are running in tc-M(9) so that's the job I highlighted in the links above. There are other failures in both pushes that are unrelated to OMTA.
Comment on attachment 8880398 [details]
Bug 1375497 - Add the new IPDL messages to sync-messages.ini.

https://reviewboard.mozilla.org/r/151780/#review157334
Attachment #8880398 - Flags: review?(dvander) → review+
Comment on attachment 8880394 [details]
Bug 1375497 - Modify the SetTestSampleTime and LeaveTestMode functions to take a layers id instead of a LayerTransactionParent.

https://reviewboard.mozilla.org/r/151772/#review157862
Attachment #8880394 - Flags: review?(howareyou322) → review+
Comment on attachment 8880395 [details]
Bug 1375497 - Make sure the WebRenderBridgeParent respects the testing time set in the CompositorBridgeParent.

https://reviewboard.mozilla.org/r/151774/#review157868
Attachment #8880395 - Flags: review?(howareyou322) → review+
Comment on attachment 8880396 [details]
Bug 1375497 - Extract helper methods to read animation values.

https://reviewboard.mozilla.org/r/151776/#review157872
Attachment #8880396 - Flags: review?(howareyou322) → review+
Comment on attachment 8880397 [details]
Bug 1375497 - Implement some more PLayerTransaction testing APIs in PWebRenderBridge.

https://reviewboard.mozilla.org/r/151778/#review157874
Attachment #8880397 - Flags: review?(howareyou322) → review+
Comment on attachment 8880399 [details]
Bug 1375497 - Hook up the testing APIs in DOMWindowUtils.

https://reviewboard.mozilla.org/r/151782/#review157876
Attachment #8880399 - Flags: review?(howareyou322) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8d6925751571
Modify the SetTestSampleTime and LeaveTestMode functions to take a layers id instead of a LayerTransactionParent. r=pchang
https://hg.mozilla.org/integration/autoland/rev/cd7792e85914
Make sure the WebRenderBridgeParent respects the testing time set in the CompositorBridgeParent. r=pchang
https://hg.mozilla.org/integration/autoland/rev/a85e3788225a
Extract helper methods to read animation values. r=pchang
https://hg.mozilla.org/integration/autoland/rev/0f5c926c958e
Implement some more PLayerTransaction testing APIs in PWebRenderBridge. r=pchang
https://hg.mozilla.org/integration/autoland/rev/1eb44b686c34
Add the new IPDL messages to sync-messages.ini. r=dvander
https://hg.mozilla.org/integration/autoland/rev/95c6960a133a
Hook up the testing APIs in DOMWindowUtils. r=pchang
You need to log in before you can comment on or make changes to this bug.