cc: Fix accelerated video freezing with commit aborts

cc/layers/video_layer.cc

 // Copyright 2010 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/layers/video_layer.h"
 
 #include "cc/layers/video_layer_impl.h"
 
 namespace cc {
 
 scoped_refptr<VideoLayer> VideoLayer::Create(VideoFrameProvider* provider) {
   return make_scoped_refptr(new VideoLayer(provider));
 }
 
 VideoLayer::VideoLayer(VideoFrameProvider* provider) : provider_(provider) {
   DCHECK(provider_);
 }
 
 VideoLayer::~VideoLayer() {}
 
 scoped_ptr<LayerImpl> VideoLayer::CreateLayerImpl(LayerTreeImpl* tree_impl) {
   return VideoLayerImpl::Create(tree_impl, id(), provider_).PassAs<LayerImpl>();
 }
 
+bool VideoLayer::Update(ResourceUpdateQueue* queue,
+                        const OcclusionTracker* occlusion) {
+  // Video layer doesn't update any resources from the main thread side,
+  // but repaint rects need to be sent to the VideoLayerImpl via commit.
+  //
+  // TODO(enne): Have the video frame provider communicate this directly

[danakj, 2013/08/01 20:22:01]

While this will be good, I don't think we'll want to delete this line once we
have that WMP impl working on the compositor thread only. So I am not sure about
this todo. It kinda belongs in WMPClientImpl.

[danakj, 2013/08/01 20:22:55]

I guess we could make this a DCHECK, but there is a 3rd WMP impl at least still.

+  // to the compositor thread rather than via commit.  http://crbug.com/267077
+  return !update_rect_.IsEmpty();
+}
+
 }  // namespace cc