Closed Bug 1148230 Opened 5 years ago Closed 5 years ago

Duplicated size_t(destinationBus->mDuration) >= framesToProcess when computing isCopySafe

Categories

(Core :: Web Audio, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: khuey, Assigned: ehsan)

References

Details

This is leftover from bug 815643.  In that bug, |destinationChannelL->length()| and |destinationChannelR->length()| both end up being |destinationBus->mDuration| and I forgot to eliminate the common subexpression.

<https://github.com/mozilla/gecko-dev/commit/573f70e55#diff-1fd72b36e1fb1e77e797377f6755dbacL180>
https://hg.mozilla.org/integration/mozilla-inbound/rev/73d6926f4912
Assignee: nobody → ehsan
Blocks: 815643
Flags: needinfo?(ehsan)
https://hg.mozilla.org/mozilla-central/rev/73d6926f4912
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.