Changes subtreeShouldRenderToSeparateSurface logic to account for explicit clipping

cc/layer.h

 // 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.
 
 #ifndef CC_LAYER_H_
 #define CC_LAYER_H_
 
 #include <public/WebFilterOperations.h>
 #include <string>
 #include <vector>
@@ skipping 264 matching lines @@
 
     virtual ScrollbarLayer* toScrollbarLayer();
 
     gfx::Rect layerRectToContentRect(const gfx::RectF& layerRect) const;
 
     // In impl-side painting, this returns true if this layer type is not
     // compatible with the main thread running freely, such as a double-buffered
     // canvas that doesn't want to be triple-buffered across all three trees.
     virtual bool blocksPendingCommit() const;
 
+    virtual bool canClipSelf() const { return false; }

[danakj, 2012/12/15 18:49:12]

Even if this is meant to be short lived, we should not leave the code in a weird
state. This name should be the accessor for m_drawProps.can_clip_self, and a
different named thing should be the virtual method here.

+
 protected:
     friend class LayerImpl;
     friend class TreeSynchronizer;
     virtual ~Layer();
 
     Layer();
 
     void setNeedsCommit();
     void setNeedsFullTreeSync();
 
@@ skipping 89 matching lines @@
     WebKit::WebLayerScrollClient* m_layerScrollClient;
 
     DrawProperties<Layer, RenderSurface> m_drawProperties;
 };
 
 void sortLayers(std::vector<scoped_refptr<Layer> >::iterator, std::vector<scoped_refptr<Layer> >::iterator, void*);
 
 }  // namespace cc
 
 #endif  // CC_LAYER_H_