Move code in src/remoting to the new callbacks.

remoting/client/plugin/pepper_view.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 "remoting/client/plugin/pepper_view.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
+#include "ppapi/cpp/completion_callback.h"
 #include "ppapi/cpp/graphics_2d.h"
 #include "ppapi/cpp/image_data.h"
 #include "ppapi/cpp/point.h"
 #include "ppapi/cpp/rect.h"
 #include "ppapi/cpp/size.h"
 #include "remoting/base/util.h"
 #include "remoting/client/chromoting_stats.h"
 #include "remoting/client/client_context.h"
 #include "remoting/client/plugin/chromoting_instance.h"
 #include "remoting/client/plugin/pepper_util.h"
@@ skipping 283 matching lines @@
 
 double PepperView::GetVerticalScaleRatio() const {
   if (instance_->DoScaling()) {
     DCHECK(!host_size_.isEmpty());
     return 1.0 * plugin_size_.height() / host_size_.height();
   }
   return 1.0;
 }
 
 void PepperView::AllocateFrame(media::VideoFrame::Format format,
-                               size_t width,
-                               size_t height,
-                               base::TimeDelta timestamp,
-                               base::TimeDelta duration,
+                               const SkISize& size,
                                scoped_refptr<media::VideoFrame>* frame_out,
-                               Task* done) {
+                               const base::Closure& done) {
   DCHECK(context_->main_message_loop()->BelongsToCurrentThread());
 
-  *frame_out = media::VideoFrame::CreateFrame(media::VideoFrame::RGB32,
-                                              width, height,
-                                              base::TimeDelta(),
-                                              base::TimeDelta());
+  *frame_out = media::VideoFrame::CreateFrame(
+      media::VideoFrame::RGB32, size.width(), size.height(),
+      base::TimeDelta(), base::TimeDelta());
   (*frame_out)->AddRef();
-  done->Run();
-  delete done;
+  done.Run();
 }
 
 void PepperView::ReleaseFrame(media::VideoFrame* frame) {
   DCHECK(context_->main_message_loop()->BelongsToCurrentThread());
 
   if (frame)
     frame->Release();
 }
 
 void PepperView::OnPartialFrameOutput(media::VideoFrame* frame,
                                       RectVector* rects,
-                                      Task* done) {
+                                      const base::Closure& done) {
   DCHECK(context_->main_message_loop()->BelongsToCurrentThread());
 
   // TODO(ajwong): Clean up this API to be async so we don't need to use a
   // member variable as a hack.
   PaintFrame(frame, rects);
-  done->Run();
-  delete done;
+  done.Run();
 }
 
 void PepperView::OnPaintDone(base::Time paint_start) {
   DCHECK(context_->main_message_loop()->BelongsToCurrentThread());
   instance_->GetStats()->video_paint_ms()->Record(
       (base::Time::Now() - paint_start).InMilliseconds());
 
   // If the last flush failed because there was already another one in progress
   // then we perform the flush now.
   if (flush_blocked_)
     FlushGraphics(base::Time::Now());
   return;
 }
 
 }  // namespace remoting