oops

Author: LongCatIsLooong

lib/web_ui/lib/src/engine/text/layout_fragmenter.dart

@@ -686,6 +686,7 @@ mixin _FragmentBox on _CombinedFragment, _FragmentMetrics, _FragmentPosition {
         end: start + _graphemeStarts[endIndex],
       ),
     };
+    assert(!range.isCollapsed, '$range must not be collapsed. $_graphemeStarts[$startIndex, $endIndex]');
     final ui.TextBox box = toTextBox(start: range.start, end: range.end);
     return ui.GlyphInfo(box.toRect(), range, box.direction);
   }

lib/web_ui/skwasm/text/paragraph.cpp

@@ -70,7 +70,6 @@ SKWASM_EXPORT bool paragraph_getClosestGlyphInfoAtCoordinate(
     bool* booleanFlags) {
   Paragraph::GlyphInfo glyphInfo;
   if (!paragraph->getClosestUTF16GlyphInfoAt(offsetX, offsetY, &glyphInfo)) {
-    delete glyphInfo;
     return false;
   }
   // This is more verbose than memcpying the whole struct but ideally we don't
@@ -91,7 +90,6 @@ SKWASM_EXPORT bool paragraph_getGlyphInfoAt(Paragraph* paragraph,
                                             bool* booleanFlags) {
   Paragraph::GlyphInfo glyphInfo;
   if (!paragraph->getGlyphInfoAtUTF16Offset(index, &glyphInfo)) {
-    delete glyphInfo;
     return false;
   }
   std::memcpy(graphemeLayoutBounds, &glyphInfo.fGraphemeLayoutBounds,

lib/web_ui/test/html/text_test.dart

@@ -117,7 +117,7 @@ Future<void> testMain() async {
 
   test('Basic glyph metrics', () {
     const double fontSize = 10;
-    final ParagraphBuilder builder = ParagraphBuilder(CkParagraphStyle(
+    final ParagraphBuilder builder = ParagraphBuilder(ParagraphStyle(
       fontStyle: FontStyle.normal,
       fontWeight: FontWeight.normal,
       fontSize: fontSize,
@@ -138,7 +138,7 @@ Future<void> testMain() async {
 
   test('Basic glyph metrics - hit test', () {
     const double fontSize = 10.0;
-    final ParagraphBuilder builder = ParagraphBuilder(CkParagraphStyle(
+    final ParagraphBuilder builder = ParagraphBuilder(ParagraphStyle(
       fontSize: fontSize,
       fontFamily: 'FlutterTest',
     ))..addText('Test\nTest');