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

Unified Diff: cc/trees/layer_tree_impl.h

Issue 2051013002: cc : Push layer lists instead of layer tree at commit and activation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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
Index: cc/trees/layer_tree_impl.h
diff --git a/cc/trees/layer_tree_impl.h b/cc/trees/layer_tree_impl.h
index ff7a4f84256627fbcbcc63b5e1131fa6944f2572..2b0e4a473f17c8ef7007524372da697ebb7c829a 100644
--- a/cc/trees/layer_tree_impl.h
+++ b/cc/trees/layer_tree_impl.h
@@ -135,6 +135,7 @@ class CC_EXPORT LayerTreeImpl {
// ---------------------------------------------------------------------------
LayerImpl* root_layer() const { return root_layer_; }
void SetRootLayer(std::unique_ptr<LayerImpl>);
+ void SetRootLayerFromLayerList();
bool IsRootLayer(const LayerImpl* layer) const;
std::unique_ptr<OwnedLayerImplList> DetachLayers();
@@ -147,10 +148,10 @@ class CC_EXPORT LayerTreeImpl {
void MoveChangeTrackingToLayers();
- LayerListIterator<LayerImpl> begin() const;
- LayerListIterator<LayerImpl> end() const;
- LayerListReverseIterator<LayerImpl> rbegin();
- LayerListReverseIterator<LayerImpl> rend();
+ std::vector<LayerImpl*>::const_iterator begin() const;
+ std::vector<LayerImpl*>::const_iterator end() const;
+ std::vector<LayerImpl*>::reverse_iterator rbegin();
+ std::vector<LayerImpl*>::reverse_iterator rend();
ajuma 2016/06/09 15:15:19 There's a LayerImplList type defined (as vector<La
jaydasika 2016/06/09 16:56:25 Done.
struct CC_EXPORT ElementLayers {
// Transform and opacity mutations apply to this layer.
@@ -474,6 +475,12 @@ class CC_EXPORT LayerTreeImpl {
void ResetAllChangeTracking();
+ void AddToLayerList(LayerImpl* layer);
+
+ void ClearLayerList();
+
+ void BuildLayerListForTesting();
+
protected:
explicit LayerTreeImpl(
LayerTreeHostImpl* layer_tree_host_impl,
@@ -579,7 +586,6 @@ class CC_EXPORT LayerTreeImpl {
std::unique_ptr<PendingPageScaleAnimation> pending_page_scale_animation_;
private:
- void BuildLayerListForTesting();
DISALLOW_COPY_AND_ASSIGN(LayerTreeImpl);
};

Powered by Google App Engine
This is Rietveld 408576698