Update APIs to expecte TextRange from getWordBoundary

Author: gspencergoog

lib/ui/text.dart

@@ -1863,12 +1863,19 @@ class Paragraph extends NativeFieldWrapperClass2 {
   }
   List<int> _getPositionForOffset(double dx, double dy) native 'Paragraph_getPositionForOffset';
 
-  /// Returns the [start, end] of the word at the given offset. Characters not
-  /// part of a word, such as spaces, symbols, and punctuation, have word breaks
-  /// on both sides. In such cases, this method will return [offset, offset+1].
+  /// Returns the [TextRange] of the word at the given [TextPosition.offset].
+  ///
+  /// Characters not part of a word, such as spaces, symbols, and punctuation,
+  /// have word breaks on both sides. In such cases, this method will return
+  /// `TextRange(begin: position.offset, end: position.offset+1)`.
+  ///
   /// Word boundaries are defined more precisely in Unicode Standard Annex #29
   /// http://www.unicode.org/reports/tr29/#Word_Boundaries
-  List<int> getWordBoundary(int offset) native 'Paragraph_getWordBoundary';
+  TextRange getWordBoundary(TextPosition position) {
+    final List<int> range = _getWordBoundary(position.offset);
+    return TextRange(start: range[0], end: range[1]);
+  }
+  List<int> _getWordBoundary(int offset) native 'Paragraph_getWordBoundary';
 
   // Redirecting the paint function in this way solves some dependency problems
   // in the C++ code. If we straighten out the C++ dependencies, we can remove

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

@@ -281,14 +281,14 @@ class EngineParagraph implements ui.Paragraph {
   }
 
   @override
-  List<int> getWordBoundary(int offset) {
+  ui.TextRange getWordBoundary(ui.TextPosition position) {
     if (_plainText == null) {
-      return <int>[offset, offset];
+      return ui.TextRange(start: position.offset, end: position.offset);
     }
 
-    final int start = WordBreaker.prevBreakIndex(_plainText, offset);
-    final int end = WordBreaker.nextBreakIndex(_plainText, offset);
-    return <int>[start, end];
+    final int start = WordBreaker.prevBreakIndex(_plainText, position.offset);
+    final int end = WordBreaker.nextBreakIndex(_plainText, position.offset);
+    return ui.TextRange(start: start, end: end);
   }
 
   @override

lib/web_ui/lib/src/ui/text.dart

@@ -1237,12 +1237,15 @@ abstract class Paragraph {
   /// within the text.
   TextPosition getPositionForOffset(Offset offset);
 
-  /// Returns the [start, end] of the word at the given offset. Characters not
-  /// part of a word, such as spaces, symbols, and punctuation, have word breaks
-  /// on both sides. In such cases, this method will return [offset, offset+1].
+  /// Returns the [TextRange] of the word at the given [TextPosition.offset].
+  ///
+  /// Characters not part of a word, such as spaces, symbols, and punctuation,
+  /// have word breaks on both sides. In such cases, this method will return
+  /// `TextRange(begin: position.offset, end: position.offset+1)`.
+  ///
   /// Word boundaries are defined more precisely in Unicode Standard Annex #29
   /// http://www.unicode.org/reports/tr29/#Word_Boundaries
-  List<int> getWordBoundary(int offset);
+  TextRange getWordBoundary(TextPosition position);
 
   /// Returns a list of text boxes that enclose all placeholders in the paragraph.
   ///