CSSCalcOperation constructor wastes 6KB of Vector capacity on cnn.com
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Nov 2018 16:49:50 +0000 (16:49 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Nov 2018 16:49:50 +0000 (16:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=190839

Patch by Rob Buis <rbuis@igalia.com> on 2018-11-12
Reviewed by Frédéric Wang.

The CSSCalcOperation ctor that takes a leftSide and rightSide parameter
wastes memory since it will always have size 2 but claims the
default Vector size. So make sure to reserve an initial capacity of 2.

* css/CSSCalculationValue.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@238089 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/CSSCalculationValue.cpp

index fe1a094..7fb8ac9 100644 (file)
@@ -1,3 +1,16 @@
+2018-11-12  Rob Buis  <rbuis@igalia.com>
+
+        CSSCalcOperation constructor wastes 6KB of Vector capacity on cnn.com
+        https://bugs.webkit.org/show_bug.cgi?id=190839
+
+        Reviewed by Frédéric Wang.
+
+        The CSSCalcOperation ctor that takes a leftSide and rightSide parameter
+        wastes memory since it will always have size 2 but claims the
+        default Vector size. So make sure to reserve an initial capacity of 2.
+
+        * css/CSSCalculationValue.cpp:
+
 2018-11-12  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
 
         WTFMove(xxx) is used in arguments while other arguments touch xxx
index 9a94a2f..05872e2 100644 (file)
@@ -660,8 +660,9 @@ private:
         : CSSCalcExpressionNode(category, isIntegerResult(op, leftSide.get(), rightSide.get()))
         , m_operator(op)
     {
-        m_children.append(WTFMove(leftSide));
-        m_children.append(WTFMove(rightSide));
+        m_children.reserveInitialCapacity(2);
+        m_children.uncheckedAppend(WTFMove(leftSide));
+        m_children.uncheckedAppend(WTFMove(rightSide));
     }
 
     CSSCalcOperation(CalculationCategory category, CalcOperator op, Vector<Ref<CSSCalcExpressionNode>>&& children)