Fix remoting to use scoped_refptr.

remoting/client/plugin/pepper_view.cc

Index: remoting/client/plugin/pepper_view.cc
diff --git a/remoting/client/plugin/pepper_view.cc b/remoting/client/plugin/pepper_view.cc
index 9a10d1a5ab91367329f37216aaba71e640992dc5..f941b08e5639e482cc262ba2d64c2e139c8cf322 100644
--- a/remoting/client/plugin/pepper_view.cc
+++ b/remoting/client/plugin/pepper_view.cc
@@ -220,9 +220,9 @@ void PepperView::OnPartialFrameOutput(media::VideoFrame* frame,
                                       UpdatedRects* rects,
                                       Task* done) {
   if (!instance_->CurrentlyOnPluginThread()) {
-    RunTaskOnPluginThread(NewTracedMethod(this,
-                                          &PepperView::OnPartialFrameOutput,
-                                          frame, rects, done));
+    RunTaskOnPluginThread(
+        NewTracedMethod(this, &PepperView::OnPartialFrameOutput,

[Nico, 2010/10/24 19:51:39]

Hi Will,

OnPartialFrameOutput takes a raw |frame| pointer, not a scoped_refptr. The bug
is only about adding make_scoped_refptr if the function does take a
scoped_refptr. Should OnPartialFrameOutput's parameter be made a scoped_refptr?

Nico

+                        make_scoped_refptr(frame), rects, done));
     return;
   }