Closed Bug 1375497 Opened 7 years ago Closed 7 years ago

Add OMTA test API support in PWebRenderBridge

Categories

(Core :: Graphics: WebRender, enhancement, P3)

Other Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(6 files)

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+
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.

Attachment

General

Created:
Updated:
Size: