requestAnimationFrame callback function name should be "sample", not "onBeforePaint"

RESOLVED FIXED in mozilla11

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

({addon-compat, dev-doc-needed})

Trunk
mozilla11
x86
Mac OS X
addon-compat, dev-doc-needed
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Per current draft at http://dvcs.w3.org/hg/webperf/raw-file/tip/specs/RequestAnimationFrame/Overview.html
Blocks: 704063
https://mxr.mozilla.org/addons/search?string=onBeforePaint says we have no consumers of this in addons.  We do have some in our own tree.
Keywords: addon-compat
Keywords: dev-doc-needed
I wonder whether I should rename nsIAnimationFrameListener to nsIFrameRequestCallback in the process...
Created attachment 575895 [details] [diff] [review]
Rename onBeforePaint to sample for animation frame callbacks.
Attachment #575895 - Flags: review?(roc)
Whiteboard: [need review]
Version: 9 Branch → Trunk
Attachment #575895 - Flags: review?(roc) → review+
Er, you need to rev the IID too.
Hmm..  I guess I do to make JS that does iid-detection work.  :(  Will do.

Thoughts on comment 2?
Is iid rev really required? Method name change is binary compatible.
"I guess I do to make JS that does iid-detection work".
(In reply to Boris Zbarsky (:bz) from comment #5)
> Thoughts on comment 2?

Seems like a good idea.
OK, I'll add a patch to rename the interface, and change its iid at the same time.
Blocks: 704171
Created attachment 576541 [details] [diff] [review]
part 2.  Rename nsIAnimationFrameListener to nsIFrameRequestCallback.
Attachment #576541 - Flags: review?(roc)
Attachment #576541 - Flags: review?(roc) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/befef8dfe35e
https://hg.mozilla.org/integration/mozilla-inbound/rev/e05a25fcfec6
Flags: in-testsuite+
Whiteboard: [need review]
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/befef8dfe35e
https://hg.mozilla.org/mozilla-central/rev/e05a25fcfec6
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.