[LayoutNG] Avoid creating NGPhysicalTextFragment::RareData

I am trying to figure out why laying out a large table takes up twice
as much memory in NG.

My test case is a 100K cells table (attached to bug).

When profiling, I've noticed that
NGPhysicalTextFragment had rare data 96340 times.
NGPaintFragment NGInkOverflowModel was being created 88360 times

This patch is my attempt to improve this. With the patch:
NGPhysicalTextFragment rare data is down to 82084
NGInkOverflowModel is down to 74080

The reason RareData was being created was that

ink_overflow.ExpandEdgesToPixelBoundaries()
local_ink_overflow = ConvertToLocal(ink_overflow)

local_rect.Contains(local_ink_overflow) was false whenever ink_overflow
was fractional (because it expanded).

Expanding edges only if local_rect does not contain local_ink_overflow
without expansion reduced the number of RareData created > 15%.

There is another problem with ascents/location that causes more cache
misses. I'll need @kojii's help with it.

Bug: 931663
Change-Id: I586ca26024338a97263f9d324d62a4a50d53895a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1535294
Commit-Queue: Aleks Totic <atotic@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#649745}

third_party/blink/renderer/core/layout/ng/geometry/ng_physical_offset_rect.cc

@@ -73,6 +73,17 @@ void NGPhysicalOffsetRect::Expand(const NGPhysicalBoxStrut& strut) {
   }
 }
 
+void NGPhysicalOffsetRect::ExpandEdgesToPixelBoundaries() {
+  int left = FloorToInt(offset.left);
+  int top = FloorToInt(offset.top);
+  int max_right = (offset.left + size.width).Ceil();
+  int max_bottom = (offset.top + size.height).Ceil();
+  offset.left = LayoutUnit(left);
+  offset.top = LayoutUnit(top);
+  size.width = LayoutUnit(max_right - left);
+  size.height = LayoutUnit(max_bottom - top);
+}
+
 NGPhysicalOffsetRect::NGPhysicalOffsetRect(const LayoutRect& source)
     : NGPhysicalOffsetRect({source.X(), source.Y()},
                            {source.Width(), source.Height()}) {}

third_party/blink/renderer/core/layout/ng/geometry/ng_physical_offset_rect.h

@@ -43,6 +43,7 @@ struct CORE_EXPORT NGPhysicalOffsetRect {
   void UniteEvenIfEmpty(const NGPhysicalOffsetRect&);
 
   void Expand(const NGPhysicalBoxStrut&);
+  void ExpandEdgesToPixelBoundaries();
 
   // Conversions from/to existing code. New code prefers type safety for
   // logical/physical distinctions.

third_party/blink/renderer/core/layout/ng/inline/ng_physical_text_fragment.cc

@@ -267,8 +267,6 @@ void NGPhysicalTextFragment::UpdateSelfInkOverflow() {
     ink_overflow.Expand(text_shadow_logical_outsets);
   }
 
-  ink_overflow.ExpandEdgesToPixelBoundaries();
-
   // Uniting the frame rect ensures that non-ink spaces such side bearings, or
   // even space characters, are included in the visual rect for decorations.
   NGPhysicalOffsetRect local_ink_overflow = ConvertToLocal(ink_overflow);
@@ -278,6 +276,7 @@ void NGPhysicalTextFragment::UpdateSelfInkOverflow() {
     return;
   }
   local_ink_overflow.Unite(local_rect);
+  local_ink_overflow.ExpandEdgesToPixelBoundaries();
   EnsureRareData()->self_ink_overflow_ = local_ink_overflow;
 }
 

third_party/blink/renderer/core/layout/ng/inline/ng_physical_text_fragment.h

@@ -144,6 +144,9 @@ class CORE_EXPORT NGPhysicalTextFragment final : public NGPhysicalFragment {
                          scoped_refptr<const ShapeResultView> shape_result);
 
   struct RareData {
+    USING_FAST_MALLOC(RareData);
+
+   public:
     NGPhysicalOffsetRect self_ink_overflow_;
     scoped_refptr<const ComputedStyle> style_;  // Used only for ellipsis.
   };

third_party/blink/renderer/core/paint/ng/ng_paint_fragment.h

@@ -348,6 +348,9 @@ class CORE_EXPORT NGPaintFragment : public RefCounted<NGPaintFragment>,
 
   // The ink overflow storage for when |InkOverflowOwnerBox()| is nullptr.
   struct NGInkOverflowModel {
+    USING_FAST_MALLOC(NGInkOverflowModel);
+
+   public:
     NGInkOverflowModel(const NGPhysicalOffsetRect& self_ink_overflow,
                        const NGPhysicalOffsetRect& contents_ink_overflow);
 

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-1-expected.txt

@@ -133,33 +133,33 @@
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
           "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
@@ -169,7 +169,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-10-expected.txt

@@ -102,16 +102,6 @@
           "rect": [65, 360, 356, 39],
           "reason": "geometry"
         },
-        {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
         {
           "object": "NGPhysicalTextFragment ' was in a furious passion, and went stamping'",
           "rect": [14, 460, 355, 59],
@@ -142,6 +132,16 @@
           "rect": [14, 440, 355, 39],
           "reason": "geometry"
         },
+        {
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
+        {
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
         {
           "object": "NGPhysicalTextFragment 'The chief difficulty Alice found at first was in managing'",
           "rect": [14, 80, 354, 119],
@@ -174,12 +174,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'the'",
-          "rect": [302, 440, 68, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-2-expected.txt

@@ -192,26 +192,6 @@
           "rect": [65, 360, 356, 39],
           "reason": "geometry"
         },
-        {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 421, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 421, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
         {
           "object": "NGPhysicalTextFragment 'about once in a minute.'",
           "rect": [14, 461, 355, 59],
@@ -242,6 +222,26 @@
           "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
+        {
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 421, 355, 39],
+          "reason": "geometry"
+        },
+        {
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 421, 355, 39],
+          "reason": "geometry"
+        },
+        {
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
+        {
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
         {
           "object": "NGPhysicalTextFragment 'The chief difficulty Alice found at first was in managing'",
           "rect": [14, 80, 354, 120],
@@ -274,22 +274,22 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 401, 305, 19],
+          "rect": [65, 401, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 441, 68, 19],
+          "rect": [302, 441, 67, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-3-expected.txt

@@ -164,37 +164,37 @@
         },
         {
           "object": "NGPhysicalTextFragment ' was in'",
-          "rect": [14, 440, 340, 79],
+          "rect": [14, 440, 339, 79],
           "reason": "appeared"
         },
         {
           "object": "NGPhysicalTextFragment 'a furious passion, and went stamping about, and'",
-          "rect": [14, 440, 340, 79],
+          "rect": [14, 440, 339, 79],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'about once in a minute.'",
-          "rect": [14, 440, 340, 79],
+          "rect": [14, 440, 339, 79],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
-          "rect": [14, 440, 340, 79],
+          "rect": [14, 440, 339, 79],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting '",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [242, 440, 67, 19],
+          "rect": [242, 440, 66, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-4-expected.txt

@@ -119,12 +119,12 @@
         },
         {
           "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
@@ -159,12 +159,12 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-5-expected.txt

@@ -103,38 +103,38 @@
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment ' was in a'",
-          "rect": [14, 440, 356, 79],
+          "object": "NGPhysicalTextFragment 'for'",
+          "rect": [14, 400, 356, 59],
           "reason": "appeared"
         },
         {
-          "object": "NGPhysicalTextFragment 'furious passion, and went stamping about, and shouting'",
-          "rect": [14, 440, 356, 79],
+          "object": "NGPhysicalTextFragment 'hedgehogs; and in a very short time '",
+          "rect": [14, 400, 356, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'in a minute.'",
-          "rect": [14, 440, 356, 79],
+          "object": "NGPhysicalTextFragment 'turns, quarrelling all the while, and fighting for the'",
+          "rect": [14, 400, 356, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment '\u2018Off with his head!\u2019 or \u2018Off with her head!\u2019 about once'",
-          "rect": [14, 440, 356, 79],
-          "reason": "geometry"
+          "object": "NGPhysicalTextFragment ' was in a'",
+          "rect": [14, 440, 355, 79],
+          "reason": "appeared"
         },
         {
-          "object": "NGPhysicalTextFragment 'for'",
-          "rect": [14, 400, 356, 59],
-          "reason": "appeared"
+          "object": "NGPhysicalTextFragment 'furious passion, and went stamping about, and shouting'",
+          "rect": [14, 440, 355, 79],
+          "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'hedgehogs; and in a very short time '",
-          "rect": [14, 400, 356, 59],
+          "object": "NGPhysicalTextFragment 'in a minute.'",
+          "rect": [14, 440, 355, 79],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'turns, quarrelling all the while, and fighting for the'",
-          "rect": [14, 400, 356, 59],
+          "object": "NGPhysicalTextFragment '\u2018Off with his head!\u2019 or \u2018Off with her head!\u2019 about once'",
+          "rect": [14, 440, 355, 79],
           "reason": "geometry"
         },
         {
@@ -169,7 +169,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting '",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
@@ -179,7 +179,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-6-expected.txt

@@ -103,28 +103,28 @@
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
           "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
@@ -159,12 +159,12 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-7-expected.txt

@@ -103,28 +103,28 @@
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
+          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
+          "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'about once in a minute.'",
+          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
           "rect": [14, 460, 355, 59],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'shouting \u2018Off with his head!\u2019 or \u2018Off with her head!\u2019'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
-          "object": "NGPhysicalTextFragment 'was in a furious passion, and went stamping about, and'",
-          "rect": [14, 460, 355, 59],
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
           "reason": "geometry"
         },
         {
@@ -159,12 +159,12 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [298, 440, 72, 19],
+          "rect": [298, 440, 71, 19],
           "reason": "style change"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-8-expected.txt

@@ -127,16 +127,6 @@
           "rect": [65, 360, 356, 39],
           "reason": "disappeared"
         },
-        {
-          "object": "NGPhysicalTextFragment 'and in a very short time '",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'quarrelling all the while, and fighting for the hedgehogs;'",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
         {
           "object": "NGPhysicalTextFragment 'and in a very short time '",
           "rect": [14, 400, 355, 59],
@@ -184,7 +174,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting '",
-          "rect": [65, 400, 305, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {
@@ -194,12 +184,12 @@
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [184, 440, 70, 19],
+          "rect": [184, 440, 69, 19],
           "reason": "geometry"
         },
         {
           "object": "NGPhysicalTextFragment 'the Queen'",
-          "rect": [302, 440, 68, 19],
+          "rect": [302, 440, 67, 19],
           "reason": "geometry"
         },
         {

third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/paint/invalidation/line-flow-with-floats-9-expected.txt

@@ -107,16 +107,6 @@
           "rect": [65, 360, 356, 39],
           "reason": "disappeared"
         },
-        {
-          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
-          "rect": [14, 420, 356, 39],
-          "reason": "geometry"
-        },
         {
           "object": "NGPhysicalTextFragment ' was in a furious passion, and went stamping'",
           "rect": [14, 460, 355, 59],
@@ -147,6 +137,16 @@
           "rect": [14, 440, 355, 39],
           "reason": "geometry"
         },
+        {
+          "object": "NGPhysicalTextFragment 'for the hedgehogs; and in a very short time '",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
+        {
+          "object": "NGPhysicalTextFragment 'for turns, quarrelling all the while, and fighting'",
+          "rect": [14, 420, 355, 39],
+          "reason": "geometry"
+        },
         {
           "object": "NGPhysicalTextFragment 'The chief difficulty Alice found at first was in managing'",
           "rect": [14, 80, 354, 119],
@@ -179,12 +179,7 @@
         },
         {
           "object": "NGPhysicalTextFragment 'The players all played at once without waiting'",
-          "rect": [65, 400, 305, 19],
-          "reason": "geometry"
-        },
-        {
-          "object": "NGPhysicalTextFragment 'the'",
-          "rect": [302, 440, 68, 19],
+          "rect": [65, 400, 304, 19],
           "reason": "geometry"
         },
         {