Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(401)

Unified Diff: cc/layers/video_layer.cc

Issue 21839004: cc: Push valid property values when CalcDrawProps skips layer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: pushpaintprops: Call SavePaintProps when needed in tests Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/layers/tiled_layer.cc ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/video_layer.cc
diff --git a/cc/layers/video_layer.cc b/cc/layers/video_layer.cc
index 6cba239e0226f9b9ccf1b0f4013cbf288bcd3e96..8d728e0a8e067a9298a06b427fffbee10df1771b 100644
--- a/cc/layers/video_layer.cc
+++ b/cc/layers/video_layer.cc
@@ -24,12 +24,16 @@ scoped_ptr<LayerImpl> VideoLayer::CreateLayerImpl(LayerTreeImpl* tree_impl) {
bool VideoLayer::Update(ResourceUpdateQueue* queue,
const OcclusionTracker* occlusion) {
+ bool updated = Layer::Update(queue, 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.
//
// This is the inefficient legacy redraw path for videos. It's better to
// communicate this directly to the VideoLayerImpl.
- return !update_rect_.IsEmpty();
+ updated |= !update_rect_.IsEmpty();
+
+ return updated;
}
} // namespace cc
« no previous file with comments | « cc/layers/tiled_layer.cc ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698