Bug 109440

Summary: [EFL] fast/forms/number/number-l10n-input.html is failing
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit EFLAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, kenneth, laszlo.gombos, lucas.de.marchi, paroga, rakuco, rwlbuis, tmpsantos, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2013-02-11 08:16:35 PST
fast/forms/number/number-l10n-input.html is failing on EFL port with the following diff: --- /home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/layout-test-results/fast/forms/number/number-l10n-input-expected.txt +++ /home/buildslave-1/webkit-buildslave/efl-linux-64-release-wk2/build/layout-test-results/fast/forms/number/number-l10n-input-actual.txt @@ -5,7 +5,7 @@ Putting some digit strings to a French number input. PASS document.execCommand("InsertText", false, "1234"); frenchInput.value is "1234" -PASS document.execCommand("InsertText", false, "1,234"); frenchInput.value is "1.234" +FAIL document.execCommand("InsertText", false, "1,234"); frenchInput.value should be 1.234. Was . PASS document.execCommand("InsertText", false, "1.234"); frenchInput.value is "1.234" Puttting some digit strings to an English number input. It is also failing on Qt and Gtk (Bug 109224).
Attachments
Patch (4.28 KB, patch)
2013-02-11 11:10 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2013-02-11 08:22:28 PST
Test skipped for EFL port in http://trac.webkit.org/changeset/142478.
Chris Dumez
Comment 2 2013-02-11 11:10:26 PST
Created attachment 187619 [details] Patch
Laszlo Gombos
Comment 3 2013-02-11 13:41:46 PST
Comment on attachment 187619 [details] Patch r=me.
Chris Dumez
Comment 4 2013-02-11 14:01:14 PST
Comment on attachment 187619 [details] Patch Clearing flags on attachment: 187619 Committed r142512: <http://trac.webkit.org/changeset/142512>
Chris Dumez
Comment 5 2013-02-11 14:01:29 PST
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.