media: Optimize HW Video to 2D Canvas copy.

cc/resources/video_resource_updater.cc

 // Copyright 2013 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 "cc/resources/video_resource_updater.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "cc/output/gl_renderer.h"
 #include "cc/resources/resource_provider.h"
@@ skipping 234 matching lines @@
 
     if (!video_renderer_)
       video_renderer_.reset(new media::SkCanvasVideoRenderer);
 
     {
       ResourceProvider::ScopedWriteLockSoftware lock(
           resource_provider_, plane_resources[0].resource_id);
       video_renderer_->Paint(video_frame.get(),
                              lock.sk_canvas(),
                              video_frame->visible_rect(),
-                             0xff);
+                             0xff,
+                             0);

[danakj, 2014/08/07 14:07:31]

This should be NULL right? It's a pointer?

[dshwang, 2014/08/07 15:10:43]

Done. I was confused with Blink coding style.

     }
 
     RecycleResourceData recycle_data = {
       plane_resources[0].resource_id,
       plane_resources[0].resource_size,
       plane_resources[0].resource_format,
       gpu::Mailbox()
     };
     external_resources.software_resources.push_back(
         plane_resources[0].resource_id);
@@ skipping 125 matching lines @@
   }
 
   PlaneResource recycled_resource(data.resource_id,
                                   data.resource_size,
                                   data.resource_format,
                                   data.mailbox);
   updater->recycled_resources_.push_back(recycled_resource);
 }
 
 }  // namespace cc