Summary: | REGRESSION (r135972): WebCore::TextIterator::plainText() discards value from Document::displayStringModifiedByEncoding() when isDisplayString == true | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> | ||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | NEW --- | ||||||
Severity: | Normal | CC: | ahmad.saleem792, ap, cdumez, darin, rniwa, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=103295 https://bugs.webkit.org/show_bug.cgi?id=166569 |
||||||
Attachments: |
|
Description
David Kilzer (:ddkilzer)
2019-03-30 09:42:25 PDT
Created attachment 366364 [details]
Patch (no tests; checking EWS)
(In reply to David Kilzer (:ddkilzer) from comment #2) > Created attachment 366364 [details] > Patch (no tests to check EWS) Interesting...the style bot doesn't complain about a missing ChangeLog. We need a test for this... I agree. And also, maybe soon we can remove the special yen sign and backslash handling if it’s not needed in practice. (In reply to Darin Adler from comment #5) > I agree. > > And also, maybe soon we can remove the special yen sign and backslash > handling if it’s not needed in practice. Unfortunately, I don't think the situation is changing anytime soon about the yen mark :( There is simply too much legacy content out there... Backslash handling was removed by Blink in this commit: https://src.chromium.org/viewvc/blink?view=revision&revision=159589 It's certainly interesting that this bug hasn't been fixed, and we don't seem to be seeing any complaints. That said, I'd be very cautious about putting too much weight Blink decisions in this area. |