Bug 93234 - [Chromium] Drawing issues in the rightmost part of RTL text
Summary: [Chromium] Drawing issues in the rightmost part of RTL text
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-06 01:38 PDT by Hironori Bono
Modified: 2012-08-06 08:29 PDT (History)
6 users (show)

See Also:


Attachments
A test case (1.16 KB, text/html)
2012-08-06 01:38 PDT, Hironori Bono
no flags Details
Manual roll out (6.18 KB, patch)
2012-08-06 03:31 PDT, Hironori Bono
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hironori Bono 2012-08-06 01:38:24 PDT
Created attachment 156621 [details]
A test case

(Copied from <http://crbug.com/139486>)

Reported by project member prog...@chromium.org, Jul 28, 2012
Version: 
Chromium 147542 (WebKit r123007): OK
Chromium 147553 (WebKit r123126): FAIL
Chromium 148889 (WebKit r123866): FAIL
OS: Windows 7

What steps will reproduce the problem?
1. Enter the following URL:
http://in.bgu.ac.il/acadsec/Pages/acadcalander.aspx
2. Using your mouse, hover one of the year links, for example 2011-2012
-The two rightmost numbers will not by underlined by the hover effect
3. Now try to select the link 2011-2012
4. Deselect it by pressing on the white background
- The two rightmost numbers will still look selected

Please provide any additional information below. Attach a screenshot if possible.

My WebKit r123067 <http://trac.webkit.org/changeset/123067> sometimes moves RTL elements right without changing its left position, i.e. it renders some RTL element to wrong places. (The attached file is the simple test case that visualize this bug. All values in its <input> element should be 500.) Such an RTL element with a wrong left position causes this problem. 

Regards,

Hironori Bono
Comment 1 Hironori Bono 2012-08-06 03:31:53 PDT
Created attachment 156646 [details]
Manual roll out

Greetings,

Apologies for disturbing you with my manual change. Unfortunately, I notice it is not so easy to fix this issue without any more regressions right now. So, I would like to manually roll out my WebKit changes (r123067 and r123572) except their layout tests. I will re-open WebKit Bug 85856 and upload another change when I can fix this bug without regressions.

Regards,

Hironori Bono
Comment 2 WebKit Review Bot 2012-08-06 08:29:47 PDT
Comment on attachment 156646 [details]
Manual roll out

Clearing flags on attachment: 156646

Committed r124773: <http://trac.webkit.org/changeset/124773>
Comment 3 WebKit Review Bot 2012-08-06 08:29:51 PDT
All reviewed patches have been landed.  Closing bug.