Summary: | Long text lines are rendered as empty for the first 2**16 characters | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Martin <dtm> | ||||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | NEW --- | ||||||||
Severity: | Normal | CC: | ap, arpitabahuguna, austin.cheney, brettw, dglazkov, eric, jimmy.selgen, mitz, pkasting, vijayan.bits, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Daniel Martin
2009-09-22 04:35:40 PDT
I think this is Windows' text drawing command rejecting a string that's "too long." The way to fix it would be to get a conservative estimate of what text actually appears in the current clip, and only draw those characters. I'm also seeing this bug on Safari running on MacOS, so I don't think it's a windows bug. *** Bug 88996 has been marked as a duplicate of this bug. *** Created attachment 152227 [details]
Patch
Comment on attachment 152227 [details] Patch Attachment 152227 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13239079 Comment on attachment 152227 [details] Patch Attachment 152227 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13240083 Okay.. I understand the problem that the typecasting will bring here. Need to check further on this. Created attachment 152498 [details]
Patch
Changing the datatype of m_len can obviously have its many typecasting ramifications. Have currently made m_len as an integer (similar to m_start). This would probably have issues too (typecasting to unsigned) but since we don't expect length to be negative, this can perhaps be ignored.(?) Also, this would have an additional overhead of increasing the size of our InlineTextBox by 2bytes. Comment on attachment 152498 [details]
Patch
I would expect this to be a large increase in memory usage for WebKit. I'm surprised there is not a COMPILE_ASSERT to verify the size of InlineTextBox.
*** Bug 90739 has been marked as a duplicate of this bug. *** *** Bug 51168 has been marked as a duplicate of this bug. *** |