cc: Set read lock fences required for YUV native video frames.

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 <algorithm>
 
 #include "base/bind.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 67 matching lines @@
 
 void VideoResourceUpdater::SetPlaneResourceUniqueId(
     const media::VideoFrame* video_frame,
     size_t plane_index,
     PlaneResource* plane_resource) {
   plane_resource->frame_ptr = video_frame;
   plane_resource->plane_index = plane_index;
   plane_resource->timestamp = video_frame->timestamp();
 }
 
-VideoFrameExternalResources::VideoFrameExternalResources() : type(NONE) {}
+VideoFrameExternalResources::VideoFrameExternalResources()
+    : type(NONE), read_lock_fences_enabled(false) {
+}
 
 VideoFrameExternalResources::~VideoFrameExternalResources() {}
 
 VideoResourceUpdater::VideoResourceUpdater(ContextProvider* context_provider,
                                            ResourceProvider* resource_provider)
     : context_provider_(context_provider),
       resource_provider_(resource_provider) {
 }
 
 VideoResourceUpdater::~VideoResourceUpdater() {
@@ skipping 292 matching lines @@
         case GL_TEXTURE_RECTANGLE_ARB:
           external_resources.type = VideoFrameExternalResources::IO_SURFACE;
           break;
         default:
           NOTREACHED();
           return VideoFrameExternalResources();
       }
       break;
     case media::VideoFrame::I420:
       external_resources.type = VideoFrameExternalResources::YUV_RESOURCE;
+      external_resources.read_lock_fences_enabled = true;

[reveman, 2015/06/17 16:59:02]

We'll need this for all formats. Maybe just set it before it switch statement.

[Daniele Castagna, 2015/06/17 17:02:25]

Done.

       break;
 #if defined(OS_MACOSX) || defined(OS_CHROMEOS)
     case media::VideoFrame::NV12:
 #endif
     case media::VideoFrame::YV12:
     case media::VideoFrame::YV16:
     case media::VideoFrame::YV24:
     case media::VideoFrame::YV12A:
     case media::VideoFrame::UNKNOWN:
       DLOG(ERROR) << "Unsupported Texture format"
@@ skipping 44 matching lines @@
     resource_it->ref_count = 0;
     updater->DeleteResource(resource_it);
     return;
   }
 
   --resource_it->ref_count;
   DCHECK_GE(resource_it->ref_count, 0);
 }
 
 }  // namespace cc