Remove size_t from audio IPC code.

The change got to this size because I had to modify the surrounding code (I didn't want to just cast at the last minute).
Review URL: http://codereview.chromium.org/577006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38192 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 33 insertions, 31 deletions

diff --git a/chrome/renderer/media/audio_renderer_impl.cc b/chrome/renderer/media/audio_renderer_impl.cc
index f10ff37..3aba1c1 100644
--- a/chrome/renderer/media/audio_renderer_impl.cc
+++ b/chrome/renderer/media/audio_renderer_impl.cc
@@ -1,12 +1,13 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.  Use of this
-// source code is governed by a BSD-style license that can be found in the
-// LICENSE file.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/renderer/media/audio_renderer_impl.h"
 
 #include <math.h>
 
 #include "chrome/common/render_messages.h"
 #include "chrome/renderer/audio_message_filter.h"
-#include "chrome/renderer/media/audio_renderer_impl.h"
 #include "chrome/renderer/render_view.h"
 #include "chrome/renderer/render_thread.h"
 #include "media/base/filter_host.h"
@@ -76,8 +77,8 @@ bool AudioRendererImpl::OnInitialize(const media::MediaFormat& media_format) {
 
   // Create the audio output stream in browser process.
   bytes_per_second_ = sample_rate_ * channels_ * sample_bits_ / 8;
-  size_t packet_size = bytes_per_second_ * kMillisecondsPerPacket / 1000;
-  size_t buffer_capacity = packet_size * kPacketsInBuffer;
+  uint32 packet_size = bytes_per_second_ * kMillisecondsPerPacket / 1000;
+  uint32 buffer_capacity = packet_size * kPacketsInBuffer;
 
   // Calculate the amount for prerolling.
   preroll_bytes_ = bytes_per_second_ * kMillisecondsPreroll / 1000;
@@ -161,7 +162,7 @@ void AudioRendererImpl::SetVolume(float volume) {
 }
 
 void AudioRendererImpl::OnCreated(base::SharedMemoryHandle handle,
-                                  size_t length) {
+                                  uint32 length) {
   DCHECK(MessageLoop::current() == io_loop_);
 
   AutoLock auto_lock(lock_);
@@ -173,7 +174,7 @@ void AudioRendererImpl::OnCreated(base::SharedMemoryHandle handle,
   shared_memory_size_ = length;
 }
 
-void AudioRendererImpl::OnRequestPacket(size_t bytes_in_buffer,
+void AudioRendererImpl::OnRequestPacket(uint32 bytes_in_buffer,
                                         const base::Time& message_timestamp) {
   DCHECK(MessageLoop::current() == io_loop_);
 
@@ -192,7 +193,8 @@ void AudioRendererImpl::OnRequestPacket(size_t bytes_in_buffer,
   OnNotifyPacketReady();
 }
 
-void AudioRendererImpl::OnStateChanged(ViewMsg_AudioStreamState state) {
+void AudioRendererImpl::OnStateChanged(
+    const ViewMsg_AudioStreamState_Params& state) {
   DCHECK(MessageLoop::current() == io_loop_);
 
   AutoLock auto_lock(lock_);
@@ -200,7 +202,7 @@ void AudioRendererImpl::OnStateChanged(ViewMsg_AudioStreamState state) {
     return;
 
   switch (state.state) {
-    case ViewMsg_AudioStreamState::kError:
+    case ViewMsg_AudioStreamState_Params::kError:
       // We receive this error if we counter an hardware error on the browser
       // side. We can proceed with ignoring the audio stream.
       // TODO(hclam): We need more handling of these kind of error. For example
@@ -209,8 +211,8 @@ void AudioRendererImpl::OnStateChanged(ViewMsg_AudioStreamState state) {
       host()->BroadcastMessage(media::kMsgDisableAudio);
       break;
     // TODO(hclam): handle these events.
-    case ViewMsg_AudioStreamState::kPlaying:
-    case ViewMsg_AudioStreamState::kPaused:
+    case ViewMsg_AudioStreamState_Params::kPlaying:
+    case ViewMsg_AudioStreamState_Params::kPaused:
       break;
     default:
       NOTREACHED();
@@ -225,7 +227,7 @@ void AudioRendererImpl::OnVolume(double volume) {
 
 void AudioRendererImpl::OnCreateStream(
     AudioManager::Format format, int channels, int sample_rate,
-    int bits_per_sample, size_t packet_size, size_t buffer_capacity) {
+    int bits_per_sample, uint32 packet_size, uint32 buffer_capacity) {
   DCHECK(MessageLoop::current() == io_loop_);
 
   AutoLock auto_lock(lock_);
@@ -237,7 +239,7 @@ void AudioRendererImpl::OnCreateStream(
   stream_id_ = filter_->AddDelegate(this);
   io_loop_->AddDestructionObserver(this);
 
-  ViewHostMsg_Audio_CreateStream params;
+  ViewHostMsg_Audio_CreateStream_Params params;
   params.format = format;
   params.channels = channels;
   params.sample_rate = sample_rate;
@@ -324,7 +326,7 @@ void AudioRendererImpl::OnNotifyPacketReady() {
                                   GetPlaybackRate())));
     }
 
-    size_t filled = FillBuffer(static_cast<uint8*>(shared_memory_->memory()),
+    uint32 filled = FillBuffer(static_cast<uint8*>(shared_memory_->memory()),
                                shared_memory_size_,
                                request_delay);
     // TODO(hclam): we should try to fill in the buffer as much as possible.
diff --git a/chrome/renderer/media/audio_renderer_impl.h b/chrome/renderer/media/audio_renderer_impl.h
index 01885be..2c4d990 100644
--- a/chrome/renderer/media/audio_renderer_impl.h
+++ b/chrome/renderer/media/audio_renderer_impl.h
@@ -1,6 +1,6 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.  Use of this
-// source code is governed by a BSD-style license that can be found in the
-// LICENSE file.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
 //
 // Audio rendering unit utilizing audio output stream provided by browser
 // process through IPC.
@@ -118,10 +118,10 @@ class AudioRendererImpl : public media::AudioRendererBase,
 
   // Methods called on IO thread ----------------------------------------------
   // AudioMessageFilter::Delegate methods, called by AudioMessageFilter.
-  void OnRequestPacket(size_t bytes_in_buffer,
+  void OnRequestPacket(uint32 bytes_in_buffer,
                        const base::Time& message_timestamp);
-  void OnStateChanged(ViewMsg_AudioStreamState state);
-  void OnCreated(base::SharedMemoryHandle handle, size_t length);
+  void OnStateChanged(const ViewMsg_AudioStreamState_Params& state);
+  void OnCreated(base::SharedMemoryHandle handle, uint32 length);
   void OnVolume(double volume);
 
   // Methods called on pipeline thread ----------------------------------------
@@ -163,7 +163,7 @@ class AudioRendererImpl : public media::AudioRendererBase,
   // sends IPC messages on that thread.
   void OnCreateStream(AudioManager::Format format, int channels,
                       int sample_rate, int bits_per_sample,
-                      size_t packet_size, size_t buffer_capacity);
+                      uint32 packet_size, uint32 buffer_capacity);
   void OnPlay();
   void OnPause();
   void OnSetVolume(double volume);
@@ -177,7 +177,7 @@ class AudioRendererImpl : public media::AudioRendererBase,
   int channels_;
   int sample_rate_;
   int sample_bits_;
-  size_t bytes_per_second_;
+  uint32 bytes_per_second_;
 
   scoped_refptr<AudioMessageFilter> filter_;
 
@@ -186,7 +186,7 @@ class AudioRendererImpl : public media::AudioRendererBase,
 
   // Memory shared by the browser process for audio buffer.
   scoped_ptr<base::SharedMemory> shared_memory_;
-  size_t shared_memory_size_;
+  uint32 shared_memory_size_;
 
   // Message loop for the IO thread.
   MessageLoop* io_loop_;
@@ -214,7 +214,7 @@ class AudioRendererImpl : public media::AudioRendererBase,
   bool prerolling_;
 
   // Remaining bytes for prerolling to complete.
-  size_t preroll_bytes_;
+  uint32 preroll_bytes_;
 
   DISALLOW_COPY_AND_ASSIGN(AudioRendererImpl);
 };
diff --git a/chrome/renderer/media/audio_renderer_impl_unittest.cc b/chrome/renderer/media/audio_renderer_impl_unittest.cc
index 1dda05d..5b823e8 100644
--- a/chrome/renderer/media/audio_renderer_impl_unittest.cc
+++ b/chrome/renderer/media/audio_renderer_impl_unittest.cc
@@ -100,12 +100,12 @@ TEST_F(AudioRendererImplTest, SetVolume) {
 
 TEST_F(AudioRendererImplTest, Stop) {
   // Declare some state messages.
-  const ViewMsg_AudioStreamState kError =
-      { ViewMsg_AudioStreamState::kError };
-  const ViewMsg_AudioStreamState kPlaying =
-      { ViewMsg_AudioStreamState::kPlaying };
-  const ViewMsg_AudioStreamState kPaused =
-      { ViewMsg_AudioStreamState::kPaused };
+  const ViewMsg_AudioStreamState_Params kError =
+      { ViewMsg_AudioStreamState_Params::kError };
+  const ViewMsg_AudioStreamState_Params kPlaying =
+      { ViewMsg_AudioStreamState_Params::kPlaying };
+  const ViewMsg_AudioStreamState_Params kPaused =
+      { ViewMsg_AudioStreamState_Params::kPaused };
 
   // Execute Stop() codepath to create an IPC message.
   renderer_->Stop();