Summary: | computation in InlineTextBox::positionForOffset() might be wrong. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xiaomei Ji <xji> | ||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | NEW --- | ||||||
Severity: | Normal | CC: | ahmad.saleem792, enrica, mitz, simon.fraser, zalan | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Xiaomei Ji
2011-03-22 12:02:44 PDT
Created attachment 88331 [details]
patch w/ layout test
Dan, Enrica,
Could you please help on verifying the layout test result on editing/selection/mixed-editability-10.html and platform/mac/editing/input/caret-primary-bidi.html?
I tried moving caret in caret-primary-bidi.html, and I did not see any where that the caret becomes invisible. Seems the result is acceptable.
But I am not sure on mixed-editability-10.html. The result changed for the first <div> with one regression and one progression.
The patch is not completed. It breaks mixed-editability-10.html in the sense that when clicking left of the span, the caret overlaps with the span, the caret should be placed before the span. The code in the patch itself seems reasonable. But there are probably some other code pieces that do float/int conversion or float rounding or other float hacking that need to be taken care of. Comment on attachment 88331 [details] patch w/ layout test View in context: https://bugs.webkit.org/attachment.cgi?id=88331&action=review > LayoutTests/platform/mac/editing/input/caret-primary-bidi-expected.txt:3 > -0: 124,508,0,28 > +0: 125,508,0,28 > 1: 21,564,0,28 > 2: 36,564,0,28 These output makes no sense to me. We should come up with a better output here. @Alan - I looked into it and it seems that it is in IFC part and I am sure that you might have already fixed it and improved it. Anything to do it? https://searchfox.org/wubkat/source/Source/WebCore/layout/integration/inline/InlineIteratorTextBox.cpp#93 |