diff options
author | fbarchard@chromium.org <fbarchard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-04 22:25:18 +0000 |
---|---|---|
committer | fbarchard@chromium.org <fbarchard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-04 22:25:18 +0000 |
commit | 3ee84160f356ad78f48639b4e26eebee9a88b9c1 (patch) | |
tree | 41cdcce0762674b9eb5893d0585673d91bea53e2 /chrome/browser/renderer_host/audio_renderer_host.cc | |
parent | 3da33567fb3c69351d74ccb11d884e4ea9f9a21e (diff) | |
download | chromium_src-3ee84160f356ad78f48639b4e26eebee9a88b9c1.zip chromium_src-3ee84160f356ad78f48639b4e26eebee9a88b9c1.tar.gz chromium_src-3ee84160f356ad78f48639b4e26eebee9a88b9c1.tar.bz2 |
SetVolume and GetVolume take one volume instead of separate left and right volumes.
BUG=26660
TEST=no visible difference. Make sure volume still works. Code size should go down marginally.
Review URL: http://codereview.chromium.org/357004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31018 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/renderer_host/audio_renderer_host.cc')
-rw-r--r-- | chrome/browser/renderer_host/audio_renderer_host.cc | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/chrome/browser/renderer_host/audio_renderer_host.cc b/chrome/browser/renderer_host/audio_renderer_host.cc index 7ea64e7..a5425be 100644 --- a/chrome/browser/renderer_host/audio_renderer_host.cc +++ b/chrome/browser/renderer_host/audio_renderer_host.cc @@ -211,12 +211,12 @@ void AudioRendererHost::IPCAudioSource::Close() { state_ = kClosed; } -void AudioRendererHost::IPCAudioSource::SetVolume(double left, double right) { +void AudioRendererHost::IPCAudioSource::SetVolume(double volume) { // TODO(hclam): maybe send an error message back to renderer if this object // is in a wrong state. if (!stream_) return; - stream_->SetVolume(left, right); + stream_->SetVolume(volume); } void AudioRendererHost::IPCAudioSource::GetVolume() { @@ -224,10 +224,10 @@ void AudioRendererHost::IPCAudioSource::GetVolume() { // is in a wrong state. if (!stream_) return; - double left_channel, right_channel; - stream_->GetVolume(&left_channel, &right_channel); + double volume; + stream_->GetVolume(&volume); host_->Send(new ViewMsg_NotifyAudioStreamVolume(route_id_, stream_id_, - left_channel, right_channel)); + volume)); } size_t AudioRendererHost::IPCAudioSource::OnMoreData(AudioOutputStream* stream, @@ -315,12 +315,12 @@ void AudioRendererHost::IPCAudioSource::SubmitPacketRequest_Locked() { } void AudioRendererHost::IPCAudioSource::SubmitPacketRequest(AutoLock* alock) { - if (alock) { - SubmitPacketRequest_Locked(); - } else { - AutoLock auto_lock(lock_); - SubmitPacketRequest_Locked(); - } + if (alock) { + SubmitPacketRequest_Locked(); + } else { + AutoLock auto_lock(lock_); + SubmitPacketRequest_Locked(); + } } void AudioRendererHost::IPCAudioSource::StartBuffering() { @@ -465,11 +465,11 @@ void AudioRendererHost::OnCloseStream(const IPC::Message& msg, int stream_id) { } void AudioRendererHost::OnSetVolume(const IPC::Message& msg, int stream_id, - double left_channel, double right_channel) { + double volume) { DCHECK(ChromeThread::CurrentlyOn(ChromeThread::IO)); IPCAudioSource* source = Lookup(msg.routing_id(), stream_id); if (source) { - source->SetVolume(left_channel, right_channel); + source->SetVolume(volume); } else { SendErrorMessage(msg.routing_id(), stream_id); } |