Title: [233636] trunk/Source/WebCore
- Revision
- 233636
- Author
- [email protected]
- Date
- 2018-07-08 22:57:01 -0700 (Sun, 08 Jul 2018)
Log Message
Optimize packing of RenderSVGViewportContainer
https://bugs.webkit.org/show_bug.cgi?id=187458
Reviewed by Zalan Bujtas.
Shrink RenderSVGViewportContainer from 296 bytes to 280 bytes, saving about 6KB on nytimes.com.
* rendering/svg/RenderSVGContainer.cpp:
(WebCore::RenderSVGContainer::RenderSVGContainer):
* rendering/svg/RenderSVGContainer.h:
* rendering/svg/RenderSVGViewportContainer.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (233635 => 233636)
--- trunk/Source/WebCore/ChangeLog 2018-07-09 05:35:38 UTC (rev 233635)
+++ trunk/Source/WebCore/ChangeLog 2018-07-09 05:57:01 UTC (rev 233636)
@@ -1,5 +1,19 @@
2018-07-08 Simon Fraser <[email protected]>
+ Optimize packing of RenderSVGViewportContainer
+ https://bugs.webkit.org/show_bug.cgi?id=187458
+
+ Reviewed by Zalan Bujtas.
+
+ Shrink RenderSVGViewportContainer from 296 bytes to 280 bytes, saving about 6KB on nytimes.com.
+
+ * rendering/svg/RenderSVGContainer.cpp:
+ (WebCore::RenderSVGContainer::RenderSVGContainer):
+ * rendering/svg/RenderSVGContainer.h:
+ * rendering/svg/RenderSVGViewportContainer.h:
+
+2018-07-08 Simon Fraser <[email protected]>
+
Shrink RenderTableSection
https://bugs.webkit.org/show_bug.cgi?id=187457
Modified: trunk/Source/WebCore/rendering/svg/RenderSVGContainer.cpp (233635 => 233636)
--- trunk/Source/WebCore/rendering/svg/RenderSVGContainer.cpp 2018-07-09 05:35:38 UTC (rev 233635)
+++ trunk/Source/WebCore/rendering/svg/RenderSVGContainer.cpp 2018-07-09 05:57:01 UTC (rev 233636)
@@ -44,8 +44,6 @@
RenderSVGContainer::RenderSVGContainer(SVGElement& element, RenderStyle&& style)
: RenderSVGModelObject(element, WTFMove(style))
- , m_objectBoundingBoxValid(false)
- , m_needsBoundariesUpdate(true)
{
}
Modified: trunk/Source/WebCore/rendering/svg/RenderSVGContainer.h (233635 => 233636)
--- trunk/Source/WebCore/rendering/svg/RenderSVGContainer.h 2018-07-09 05:35:38 UTC (rev 233635)
+++ trunk/Source/WebCore/rendering/svg/RenderSVGContainer.h 2018-07-09 05:57:01 UTC (rev 233636)
@@ -73,10 +73,11 @@
bool isSVGContainer() const final { return true; }
FloatRect m_objectBoundingBox;
- bool m_objectBoundingBoxValid;
FloatRect m_strokeBoundingBox;
FloatRect m_repaintBoundingBox;
- bool m_needsBoundariesUpdate : 1;
+
+ bool m_objectBoundingBoxValid { false };
+ bool m_needsBoundariesUpdate { true };
};
} // namespace WebCore
Modified: trunk/Source/WebCore/rendering/svg/RenderSVGViewportContainer.h (233635 => 233636)
--- trunk/Source/WebCore/rendering/svg/RenderSVGViewportContainer.h 2018-07-09 05:35:38 UTC (rev 233635)
+++ trunk/Source/WebCore/rendering/svg/RenderSVGViewportContainer.h 2018-07-09 05:57:01 UTC (rev 233636)
@@ -60,11 +60,12 @@
void applyViewportClip(PaintInfo&) override;
bool pointIsInsideViewportClip(const FloatPoint& pointInParent) override;
- FloatRect m_viewport;
- mutable AffineTransform m_localToParentTransform;
bool m_didTransformToRootUpdate : 1;
bool m_isLayoutSizeChanged : 1;
bool m_needsTransformUpdate : 1;
+
+ FloatRect m_viewport;
+ mutable AffineTransform m_localToParentTransform;
};
} // namespace WebCore
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
