Bug 95074

Summary: Datalist RTL test fails on ports that have progress indicator on the range groove
Product: WebKit Reporter: Thiago Marcos P. Santos <tmpsantos>
Component: Tools / TestsAssignee: Thiago Marcos P. Santos <tmpsantos>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, keishi, rakuco, tkent, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 94595    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Problem with the current test implementation
none
Patch none

Thiago Marcos P. Santos
Reported 2012-08-27 05:02:26 PDT
One solution might be set the initial value to somewhere in the corners. Test case: fast/forms/datalist/input-appearance-range-with-datalist-rtl.html
Attachments
Patch (3.39 KB, patch)
2012-08-27 06:52 PDT, Thiago Marcos P. Santos 		no flags
DetailsFormatted DiffDiff
Problem with the current test implementation (36.41 KB, image/png)
2012-08-27 07:08 PDT, Thiago Marcos P. Santos 		no flags
Details
Patch (3.35 KB, patch)
2012-08-28 04:43 PDT, Thiago Marcos P. Santos 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Thiago Marcos P. Santos
Comment 1 2012-08-27 06:49:24 PDT
(In reply to comment #0) > One solution might be set the initial value to somewhere in the corners. > > Test case: > fast/forms/datalist/input-appearance-range-with-datalist-rtl.html I tried doing this plus applying a horizontal flip transformation but it didn't work. The only solution I see now is applying a mask on top of the slider, but keep the ticks visible.
Thiago Marcos P. Santos
Comment 2 2012-08-27 06:52:18 PDT
Created attachment 160709 [details] Patch
Thiago Marcos P. Santos
Comment 3 2012-08-27 07:08:40 PDT
Created attachment 160713 [details] Problem with the current test implementation The problem should affect EFL and Qt at least.
Kent Tamura
Comment 4 2012-08-27 21:15:06 PDT
(In reply to comment #3) > Created an attachment (id=160713) [details] > Problem with the current test implementation > > The problem should affect EFL and Qt at least. Ah, I didn't know we have such slider implementations. I think the patch is ok. Converting the test from a ref test to a pixel test is also acceptable.
Kent Tamura
Comment 5 2012-08-27 21:15:35 PDT
Comment on attachment 160709 [details] Patch cq- because of purple EWS.
Thiago Marcos P. Santos
Comment 6 2012-08-28 04:43:21 PDT
Created attachment 160954 [details] Patch Re-uploading. The dependency for this patch just landed.
WebKit Review Bot
Comment 7 2012-08-28 05:49:41 PDT
Comment on attachment 160954 [details] Patch Clearing flags on attachment: 160954 Committed r126869: <http://trac.webkit.org/changeset/126869>
WebKit Review Bot
Comment 8 2012-08-28 05:49:45 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.