RESOLVED FIXED284831
HTML parser phone number handling should better account for MathML
https://bugs.webkit.org/show_bug.cgi?id=284831
Summary HTML parser phone number handling should better account for MathML
Anne van Kesteren
Reported 2024-12-17 11:01:14 PST
HTMLTreeBuilder::processCharacterBufferForInBody avoids calling phone number handling for several elements, but MathML is not part of the list. It probably should be. As should SVG. What's also weird is that HTMLTreeBuilder::linkifyPhoneNumbers does a tree walk, but its caller does one too through shouldParseTelephoneNumbersInNode. Those should probably be deduplicated.
Attachments
Anne van Kesteren
Comment 1 2024-12-17 11:09:12 PST
Also, all credit to https://leancrew.com/all-this/2024/12/mathml-problems-in-mobile-safari/ for making me investigate this to begin with. And Ricky for pointing out the post.
Radar WebKit Bug Importer
Comment 2 2024-12-17 12:14:29 PST
Anne van Kesteren
Comment 3 2025-01-09 10:18:25 PST
https://github.com/WebKit/WebKit/commit/51965fbac8b7d44d52321c3bb81a881a9a59923a explains the reasoning behind the duplication quite well.
Anne van Kesteren
Comment 4 2025-01-09 23:41:49 PST
EWS
Comment 5 2025-01-10 07:37:04 PST
Committed 288706@main (e13876c09f11): <https://commits.webkit.org/288706@main> Reviewed commits have been landed. Closing PR #38831 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.