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

Side by Side Diff: third_party/WebKit/Source/core/paint/RarePaintData.h

Issue 2831683003: Refactor to centralize code which decides whether ObjectPaintProperties are needed. (Closed)
Patch Set: none Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef RarePaintData_h 5 #ifndef RarePaintData_h
6 #define RarePaintData_h 6 #define RarePaintData_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "platform/wtf/Allocator.h" 9 #include "platform/wtf/Allocator.h"
10 #include "platform/wtf/Noncopyable.h" 10 #include "platform/wtf/Noncopyable.h"
(...skipping 14 matching lines...) Expand all
25 RarePaintData(); 25 RarePaintData();
26 ~RarePaintData(); 26 ~RarePaintData();
27 27
28 PaintLayer* Layer() { return layer_.get(); } 28 PaintLayer* Layer() { return layer_.get(); }
29 void SetLayer(std::unique_ptr<PaintLayer>); 29 void SetLayer(std::unique_ptr<PaintLayer>);
30 30
31 ObjectPaintProperties* PaintProperties() const { 31 ObjectPaintProperties* PaintProperties() const {
32 return paint_properties_.get(); 32 return paint_properties_.get();
33 } 33 }
34 ObjectPaintProperties& EnsurePaintProperties(); 34 ObjectPaintProperties& EnsurePaintProperties();
35 void ClearPaintProperties();
35 36
36 PropertyTreeState* LocalBorderBoxProperties() const { 37 PropertyTreeState* LocalBorderBoxProperties() const {
37 return local_border_box_properties_.get(); 38 return local_border_box_properties_.get();
38 } 39 }
39 40
40 void ClearLocalBorderBoxProperties(); 41 void ClearLocalBorderBoxProperties();
41 void SetLocalBorderBoxProperties(PropertyTreeState&); 42 void SetLocalBorderBoxProperties(PropertyTreeState&);
42 43
43 // This is the complete set of property nodes that can be used to paint the 44 // This is the complete set of property nodes that can be used to paint the
44 // contents of this object. It is similar to local_border_box_properties_ but 45 // contents of this object. It is similar to local_border_box_properties_ but
(...skipping 18 matching lines...) Expand all
63 // The div's local border box properties would have an opacity 0.3 effect 64 // The div's local border box properties would have an opacity 0.3 effect
64 // node. Even though the div has no transform, its local border box 65 // node. Even though the div has no transform, its local border box
65 // properties would have a transform node that points to the div's 66 // properties would have a transform node that points to the div's
66 // ancestor transform space. 67 // ancestor transform space.
67 std::unique_ptr<PropertyTreeState> local_border_box_properties_; 68 std::unique_ptr<PropertyTreeState> local_border_box_properties_;
68 }; 69 };
69 70
70 } // namespace blink 71 } // namespace blink
71 72
72 #endif 73 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698