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

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

Issue 2873823002: Put PaintInvalidatorContext on the heap, to reduce stack overflows. (Closed)
Patch Set: Merge branch 'master' into fastmallocpaintinvalidator Created 3 years, 7 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/paint/PrePaintTreeWalk.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 PaintInvalidator_h 5 #ifndef PaintInvalidator_h
6 #define PaintInvalidator_h 6 #define PaintInvalidator_h
7 7
8 #include "core/layout/LayoutObject.h" 8 #include "core/layout/LayoutObject.h"
9 #include "core/paint/PaintPropertyTreeBuilder.h" 9 #include "core/paint/PaintPropertyTreeBuilder.h"
10 #include "platform/geometry/LayoutRect.h" 10 #include "platform/geometry/LayoutRect.h"
11 #include "platform/wtf/Vector.h" 11 #include "platform/wtf/Vector.h"
12 12
13 namespace blink { 13 namespace blink {
14 14
15 struct PaintInvalidatorContext { 15 struct PaintInvalidatorContext {
16 USING_FAST_MALLOC(PaintInvalidatorContext);
17
18 public:
16 PaintInvalidatorContext() : parent_context(nullptr) {} 19 PaintInvalidatorContext() : parent_context(nullptr) {}
17 20
18 PaintInvalidatorContext(const PaintInvalidatorContext& parent_context) 21 PaintInvalidatorContext(const PaintInvalidatorContext& parent_context)
19 : parent_context(&parent_context), 22 : parent_context(&parent_context),
20 forced_subtree_invalidation_flags( 23 forced_subtree_invalidation_flags(
21 parent_context.forced_subtree_invalidation_flags), 24 parent_context.forced_subtree_invalidation_flags),
22 paint_invalidation_container( 25 paint_invalidation_container(
23 parent_context.paint_invalidation_container), 26 parent_context.paint_invalidation_container),
24 paint_invalidation_container_for_stacked_contents( 27 paint_invalidation_container_for_stacked_contents(
25 parent_context.paint_invalidation_container_for_stacked_contents), 28 parent_context.paint_invalidation_container_for_stacked_contents),
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 PaintInvalidatorContext&); 151 PaintInvalidatorContext&);
149 ALWAYS_INLINE void UpdateVisualRect(const LayoutObject&, 152 ALWAYS_INLINE void UpdateVisualRect(const LayoutObject&,
150 PaintInvalidatorContext&); 153 PaintInvalidatorContext&);
151 154
152 Vector<const LayoutObject*> pending_delayed_paint_invalidations_; 155 Vector<const LayoutObject*> pending_delayed_paint_invalidations_;
153 }; 156 };
154 157
155 } // namespace blink 158 } // namespace blink
156 159
157 #endif // PaintInvalidator_h 160 #endif // PaintInvalidator_h
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/paint/PrePaintTreeWalk.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698