Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(235)

Issue 7745029: Merge 92408 - Make sure that AudioArray is 16-byte aligned (Closed)

Created:
9 years, 4 months ago by Chris Rogers
Modified:
9 years, 4 months ago
Reviewers:
Chris Rogers
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/835/
Visibility:
Public.

Description

Merge 92408 - Make sure that AudioArray is 16-byte aligned https://bugs.webkit.org/show_bug.cgi?id=65651 Reviewed by Kenneth Russell. No new tests - this does not change JavaScript API. * platform/audio/AudioArray.h: (WebCore::AudioArray::AudioArray): (WebCore::AudioArray::~AudioArray): (WebCore::AudioArray::allocate): (WebCore::AudioArray::data): (WebCore::AudioArray::size): (WebCore::AudioArray::at): (WebCore::AudioArray::operator[]): (WebCore::AudioArray::alignedAddress): * platform/audio/Biquad.cpp: (WebCore::Biquad::Biquad): * platform/audio/ReverbConvolverStage.cpp: (WebCore::ReverbConvolverStage::ReverbConvolverStage): * webaudio/DelayDSPKernel.cpp: (WebCore::DelayDSPKernel::DelayDSPKernel): * webaudio/RealtimeAnalyser.cpp: (WebCore::RealtimeAnalyser::setFftSize): TBR=crogers@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=93810

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+75 lines, -10 lines) Patch
M Source/WebCore/platform/audio/AudioArray.h View 2 chunks +68 lines, -3 lines 0 comments Download
M Source/WebCore/platform/audio/Biquad.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/WebCore/platform/audio/ReverbConvolverStage.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/WebCore/webaudio/DelayDSPKernel.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/WebCore/webaudio/RealtimeAnalyser.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
Chris Rogers
9 years, 4 months ago (2011-08-25 19:53:17 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698