Finish the base::Bind migration for webkit/media.

webkit/media/webmediaplayer_proxy.cc

 // Copyright (c) 2011 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 "webkit/media/webmediaplayer_proxy.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "media/base/pipeline_status.h"
@@ skipping 22 matching lines @@
 
 WebMediaPlayerProxy::~WebMediaPlayerProxy() {
   Detach();
 }
 
 void WebMediaPlayerProxy::Repaint() {
   base::AutoLock auto_lock(lock_);
   if (outstanding_repaints_ < kMaxOutstandingRepaints) {
     ++outstanding_repaints_;
 
-    render_loop_->PostTask(FROM_HERE,
-        NewRunnableMethod(this, &WebMediaPlayerProxy::RepaintTask));
+    render_loop_->PostTask(
+        FROM_HERE,

[scherkus (not reviewing), 2011/12/01 17:42:18]

nit: we seemed to have settled on having:

message_loop_->PostTask(FROM_HERE,
    base::Bind(...));

[Ami GONE FROM CHROMIUM, 2011/12/01 18:11:26]

Dunno about "settled on" but that's a violation of the style guide.  Part of my
base::Bind migrations has been fixing these violations.
The post-migration commonest pattern is
loop->PostTask(FROM_HERE, base::Bind(
    blah blah blah
which *is* legit in the style guide.  Changed this to that.

+        base::Bind(&WebMediaPlayerProxy::RepaintTask, this));
   }
 }
 
 void WebMediaPlayerProxy::SetVideoRenderer(
     const scoped_refptr<VideoRendererImpl>& video_renderer) {
   video_renderer_ = video_renderer;
 }
 
 WebDataSourceBuildObserverHack WebMediaPlayerProxy::GetBuildObserver() {
   return base::Bind(&WebMediaPlayerProxy::AddDataSource, this);
@@ skipping 37 matching lines @@
   video_renderer_ = NULL;
 
   {
     base::AutoLock auto_lock(data_sources_lock_);
     data_sources_.clear();
   }
 }
 
 void WebMediaPlayerProxy::PipelineInitializationCallback(
     PipelineStatus status) {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::PipelineInitializationTask, status));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::PipelineInitializationTask, this, status));
 }
 
 void WebMediaPlayerProxy::PipelineSeekCallback(PipelineStatus status) {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::PipelineSeekTask, status));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::PipelineSeekTask, this, status));
 }
 
 void WebMediaPlayerProxy::PipelineEndedCallback(PipelineStatus status) {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::PipelineEndedTask, status));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::PipelineEndedTask, this, status));
 }
 
 void WebMediaPlayerProxy::PipelineErrorCallback(PipelineStatus error) {
   DCHECK_NE(error, media::PIPELINE_OK);
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::PipelineErrorTask, error));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::PipelineErrorTask, this, error));
 }
 
 void WebMediaPlayerProxy::NetworkEventCallback(NetworkEvent type) {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::NetworkEventTask, type));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::NetworkEventTask, this, type));
 }
 
 void WebMediaPlayerProxy::AddDataSource(WebDataSource* data_source) {
   base::AutoLock auto_lock(data_sources_lock_);
   data_sources_.push_back(make_scoped_refptr(data_source));
 }
 
 void WebMediaPlayerProxy::RepaintTask() {
   DCHECK(MessageLoop::current() == render_loop_);
   {
@@ skipping 42 matching lines @@
     video_renderer_->GetCurrentFrame(frame_out);
 }
 
 void WebMediaPlayerProxy::PutCurrentFrame(
     scoped_refptr<media::VideoFrame> frame) {
   if (video_renderer_)
     video_renderer_->PutCurrentFrame(frame);
 }
 
 void WebMediaPlayerProxy::DemuxerOpened(media::ChunkDemuxer* demuxer) {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::DemuxerOpenedTask,
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::DemuxerOpenedTask, this,
       scoped_refptr<media::ChunkDemuxer>(demuxer)));
 }
 
 void WebMediaPlayerProxy::DemuxerClosed() {
-  render_loop_->PostTask(FROM_HERE, NewRunnableMethod(
-      this, &WebMediaPlayerProxy::DemuxerClosedTask));
+  render_loop_->PostTask(FROM_HERE, base::Bind(
+      &WebMediaPlayerProxy::DemuxerClosedTask, this));
 }
 
 void WebMediaPlayerProxy::DemuxerFlush() {
   if (chunk_demuxer_.get())
     chunk_demuxer_->FlushData();
 }
 
 bool WebMediaPlayerProxy::DemuxerAppend(const uint8* data, size_t length) {
   if (chunk_demuxer_.get())
     return chunk_demuxer_->AppendData(data, length);
@@ skipping 16 matching lines @@
   chunk_demuxer_ = demuxer;
   if (webmediaplayer_)
     webmediaplayer_->OnDemuxerOpened();
 }
 
 void WebMediaPlayerProxy::DemuxerClosedTask() {
   chunk_demuxer_ = NULL;
 }
 
 }  // namespace webkit_media