Bug 284831
| Summary: | HTML parser phone number handling should better account for MathML | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Anne van Kesteren <annevk> |
| Component: | DOM | Assignee: | Anne van Kesteren <annevk> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | darin, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Anne van Kesteren
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 | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Anne van Kesteren
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
<rdar://problem/141632782>
Anne van Kesteren
https://github.com/WebKit/WebKit/commit/51965fbac8b7d44d52321c3bb81a881a9a59923a explains the reasoning behind the duplication quite well.
Anne van Kesteren
Pull request: https://github.com/WebKit/WebKit/pull/38831
EWS
Committed 288706@main (e13876c09f11): <https://commits.webkit.org/288706@main>
Reviewed commits have been landed. Closing PR #38831 and removing active labels.