Disambiguate use of "samples" by introducing "frames" in a number of places

RESOLVED FIXED in mozilla10

Status

()

Core
Audio/Video
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: kinetik, Assigned: kinetik)

Tracking

Trunk
mozilla10
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 562651 [details] [diff] [review]
patch v0

Refer to "frames" rather than "samples" in most places, so it's clearer which one is really meant.  This is mostly a rename patch, but there are a couple of small logic changes where samples need to be calculated instead of frames.

Also renames ns{Local,Remote}AudioStream to ns{Native,Remoted}AudioStream to make their purpose clearer.
Attachment #562651 - Flags: review?(chris.double)
(Assignee)

Updated

6 years ago
Blocks: 623444

Updated

6 years ago
Attachment #562651 - Flags: review?(chris.double) → review+
(Assignee)

Comment 2

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/89b5ec4cacc1
Target Milestone: --- → mozilla10
https://hg.mozilla.org/mozilla-central/rev/89b5ec4cacc1
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.