Commit 6d9f25a8 authored by Ian Kilpatrick's avatar Ian Kilpatrick Committed by Commit Bot

[cleanup] Remove unused members from LayoutSVGRoot.

There should be no behaviour change.

Change-Id: Id077c3928c977f6eabc70d6849970d1809445019
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2468942Reviewed-by: default avatarFredrik Söderquist <fs@opera.com>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#816777}
parent 51d021d2
...@@ -52,11 +52,9 @@ namespace blink { ...@@ -52,11 +52,9 @@ namespace blink {
LayoutSVGRoot::LayoutSVGRoot(SVGElement* node) LayoutSVGRoot::LayoutSVGRoot(SVGElement* node)
: LayoutReplaced(node), : LayoutReplaced(node),
object_bounding_box_valid_(false),
is_layout_size_changed_(false), is_layout_size_changed_(false),
did_screen_scale_factor_change_(false), did_screen_scale_factor_change_(false),
needs_boundaries_or_transform_update_(true), needs_boundaries_or_transform_update_(true),
has_box_decoration_background_(false),
has_non_isolated_blending_descendants_(false), has_non_isolated_blending_descendants_(false),
has_non_isolated_blending_descendants_dirty_(false), has_non_isolated_blending_descendants_dirty_(false),
has_descendant_with_compositing_reason_(false), has_descendant_with_compositing_reason_(false),
...@@ -276,10 +274,6 @@ void LayoutSVGRoot::UpdateLayout() { ...@@ -276,10 +274,6 @@ void LayoutSVGRoot::UpdateLayout() {
} }
UpdateAfterLayout(); UpdateAfterLayout();
has_box_decoration_background_ = IsDocumentElement()
? StyleRef().HasBoxDecorationBackground()
: HasBoxDecorationBackground();
ClearNeedsLayout(); ClearNeedsLayout();
} }
...@@ -374,10 +368,6 @@ void LayoutSVGRoot::StyleDidChange(StyleDifference diff, ...@@ -374,10 +368,6 @@ void LayoutSVGRoot::StyleDidChange(StyleDifference diff,
NOT_DESTROYED(); NOT_DESTROYED();
if (diff.NeedsFullLayout()) if (diff.NeedsFullLayout())
SetNeedsBoundariesUpdate(); SetNeedsBoundariesUpdate();
if (diff.NeedsPaintInvalidation()) {
// Box decorations may have appeared/disappeared - recompute status.
has_box_decoration_background_ = StyleRef().HasBoxDecorationBackground();
}
if (old_style && StyleChangeAffectsIntrinsicSize(*old_style)) if (old_style && StyleChangeAffectsIntrinsicSize(*old_style))
IntrinsicSizingInfoChanged(); IntrinsicSizingInfoChanged();
...@@ -534,9 +524,10 @@ const LayoutObject* LayoutSVGRoot::PushMappingToContainer( ...@@ -534,9 +524,10 @@ const LayoutObject* LayoutSVGRoot::PushMappingToContainer(
void LayoutSVGRoot::UpdateCachedBoundaries() { void LayoutSVGRoot::UpdateCachedBoundaries() {
NOT_DESTROYED(); NOT_DESTROYED();
SVGLayoutSupport::ComputeContainerBoundingBoxes(this, object_bounding_box_, bool ignore;
object_bounding_box_valid_, SVGLayoutSupport::ComputeContainerBoundingBoxes(
stroke_bounding_box_); this, object_bounding_box_,
/* object_bounding_box_valid */ ignore, stroke_bounding_box_);
} }
bool LayoutSVGRoot::NodeAtPoint(HitTestResult& result, bool LayoutSVGRoot::NodeAtPoint(HitTestResult& result,
......
...@@ -222,13 +222,11 @@ class CORE_EXPORT LayoutSVGRoot final : public LayoutReplaced { ...@@ -222,13 +222,11 @@ class CORE_EXPORT LayoutSVGRoot final : public LayoutReplaced {
LayoutObjectChildList children_; LayoutObjectChildList children_;
LayoutSize container_size_; LayoutSize container_size_;
FloatRect object_bounding_box_; FloatRect object_bounding_box_;
bool object_bounding_box_valid_;
FloatRect stroke_bounding_box_; FloatRect stroke_bounding_box_;
AffineTransform local_to_border_box_transform_; AffineTransform local_to_border_box_transform_;
bool is_layout_size_changed_ : 1; bool is_layout_size_changed_ : 1;
bool did_screen_scale_factor_change_ : 1; bool did_screen_scale_factor_change_ : 1;
bool needs_boundaries_or_transform_update_ : 1; bool needs_boundaries_or_transform_update_ : 1;
bool has_box_decoration_background_ : 1;
mutable bool has_non_isolated_blending_descendants_ : 1; mutable bool has_non_isolated_blending_descendants_ : 1;
mutable bool has_non_isolated_blending_descendants_dirty_ : 1; mutable bool has_non_isolated_blending_descendants_dirty_ : 1;
mutable bool has_descendant_with_compositing_reason_ : 1; mutable bool has_descendant_with_compositing_reason_ : 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment