Closed Bug 859595 Opened 12 years ago Closed 12 years ago

Implement the new alternate names

Categories

(Core :: Web Audio, defect)

Product:
Core
Component:
Web Audio
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla23

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

Patch (v0)
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
We need to implement the alternate names previously known as deprecated names: <https://dvcs.w3.org/hg/audio/rev/005ecbd7710c>
As part of this, we're going to have to support attributes which accept either Web IDL enums or numerical constants. Boris, any idea what the correct way of doing that is?
Flags: needinfo?(bzbarsky)
There isn't one (short of using "any" and doing it all by hand). Right now WebIDL does not allow distinguishing between strings and numbers. Cameron, this keeps coming up....
Flags: needinfo?(bzbarsky) → needinfo?(cam)
(In reply to Boris Zbarsky (:bz) from comment #3) > There isn't one (short of using "any" and doing it all by hand). Right now > WebIDL does not allow distinguishing between strings and numbers. > > Cameron, this keeps coming up.... It would be really great if we fix it. Cameron? :-)
Attached patch Patch (v0) (deleted) — — Splinter Review
This doesn't deserve to be called "Patch (v1)" even...
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #736636 - Flags: review?(bzbarsky)
Will do.
Flags: needinfo?(cam)
Comment on attachment 736636 [details] [diff] [review] Patch (v0) >+partial interface AudioBufferSourceNode { Maybe document this with a link to the relevant part of the spec? Similar for the others. r=me.
Attachment #736636 - Flags: review?(bzbarsky) → review+
(In reply to Cameron McCormack (:heycam) from comment #6) > Will do. Can you please let me know once that's done? Thanks!
To be fair, the part you'll want to know is when we implement it, not when Cameron updates the spec... ;) I'll make sure to cc you on that bug if you're not the one who files it.
https://hg.mozilla.org/mozilla-central/rev/fb81cc5b400e
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
(In reply to comment #10) > To be fair, the part you'll want to know is when we implement it, not when > Cameron updates the spec... ;) I'll make sure to cc you on that bug if you're > not the one who files it. Filed bug 861371.
Mass moving Web Audio bugs to the Web Audio component. Filter on duckityduck.
Component: Video/Audio → Web Audio
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: