1443319 - Removing unnecessary generator on waitForAnimationTimelineRendering breaks browser_animation_timeline_rate_selector.js

Status: RESOLVED INVALID

(DevTools :: Inspector: Animations, defect, P2)

Description

[Alexandre Poirot [:ochameau]]

With the following patch:
 $ ./mach mochitest devtools/client/animationinspector/test/browser_animation_timeline_rate_selector.js

fails with:
Select just one animated node and change its rate only
Selecting the node for '.animated'
Console message: [JavaScript Error: "A promise chain failed to handle a rejection. Did you forget to '.catch', or did you forget to 'return'?
See https://developer.mozilla.org/Mozilla/JavaScript_code_modules/Promise.jsm/Promise

Date: Mon Mar 05 2018 14:10:40 GMT-0800 (PST)
Full Message: Protocol error (noSuchActor): No such actor for ID: server1.conn0.child1/animationplayer53

This block Task.jsm removal from animation inspector folder as switching all generators to async also breaks this test with the same error.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1443319 - Removing unnecessary generator on waitForAnimationTimelineRendering makes browser_animation_timeline_rate_selector.js fail

Review commit: https://reviewboard.mozilla.org/r/225156/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/225156/

Comment 2

[Yulia Startsev [:yulia]]

Attachment: Bug 1443319 - Remove unnecessary generator on waitForAnimationTimelineRendering

- Fixes failing test

Review commit: https://reviewboard.mozilla.org/r/233302/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233302/

Comment 3

[Alexandre Poirot [:ochameau]]

Comment on attachment 8964583 [details]
Bug 1443319 - Remove unnecessary generator on waitForAnimationTimelineRendering

https://reviewboard.mozilla.org/r/233302/#review238908

It looks like you were working on an old version of mozilla-central.
The fix looks good if it still works after rebase.

Thanks for looking into that.

::: devtools/client/animationinspector/test/head.js:361
(Diff revision 1)
>    // Simulate the right events to select the option in the drop-down.
>    EventUtils.synthesizeMouseAtCenter(select, {type: "mousedown"}, win);
>    EventUtils.synthesizeMouseAtCenter(option, {type: "mouseup"}, win);
>  
>    yield onUiUpdated;
> -  yield onRendered;
> +  yield waitForAnimationTimelineRendering(panel);

You would have to rebase as this method is now using await.
Does that still work on try after rebase?

Comment 4

[Yulia Startsev [:yulia]]

Looks like this change is no longer necessary as it is already in central: https://searchfox.org/mozilla-central/source/devtools/client/animationinspector/test/head.js#382