RESOLVED FIXED 201471
-[WKContentView selectedText] returns an empty string when selecting more than 200 characters 
https://bugs.webkit.org/show_bug.cgi?id=201471
Summary -[WKContentView selectedText] returns an empty string when selecting more tha...
Wenson Hsieh
Reported 2019-09-04 14:56:01 PDT
<rdar://problem/55039227>
Attachments
Patch (4.46 KB, patch)
2019-09-04 15:38 PDT, Wenson Hsieh 		thorton: review+
DetailsFormatted DiffDiff
Fix non-internal iOS builds (5.31 KB, patch)
2019-09-04 15:56 PDT, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2019-09-04 15:38:38 PDT
Created attachment 378019 [details] Patch
Tim Horton
Comment 2 2019-09-04 15:43:56 PDT
Comment on attachment 378019 [details] Patch I feel like it also wouldn't be crazy to bump it to like... 10k or something. But whatever.
Wenson Hsieh
Comment 3 2019-09-04 15:51:21 PDT
(In reply to Tim Horton from comment #2) > Comment on attachment 378019 [details] > Patch > > I feel like it also wouldn't be crazy to bump it to like... 10k or > something. But whatever. Indeed! I also feel like we should see if we can get rid of this altogether. Callers of this really seem to be interested in either one of two things: - Is there any selected text? - What is the full selected text? It seems for (1), we could just make them check the length of the selected text instead, and for (2) refactor them to asynchronously grab the full text. Unfortunately, many of these call sites are in UIKit, so cleaning this up is a tad tricky :/
Wenson Hsieh
Comment 4 2019-09-04 15:56:57 PDT
Created attachment 378021 [details] Fix non-internal iOS builds
WebKit Commit Bot
Comment 5 2019-09-04 18:17:20 PDT
Comment on attachment 378021 [details] Fix non-internal iOS builds Clearing flags on attachment: 378021 Committed r249508: <https://trac.webkit.org/changeset/249508>
Note You need to log in before you can comment on or make changes to this bug.