Bug 79445

Summary: REGRESSION(90089): Input type='search' text shakes up and down when animating
Product: WebKit Reporter: Jon Lee <jonlee>
Component: Layout and RenderingAssignee: Kent Tamura <tkent>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, morrita, tkent, webkit-bug-importer, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.7   
Attachments:
Description Flags
Test case
none
Better test case
none
Patch
none
Patch 2 dglazkov: review+, webkit.review.bot: commit-queue-

Description Jon Lee 2012-02-23 22:41:55 PST
Created attachment 128653 [details]
Test case

Open the attached test page and wait a couple seconds. Soon the search field will animate. The input type='search' placeholder text shakes up and down during animation. The jittering does not occur if you remove the "type=search".

It looks like this is a regression caused by r90089.

<rdar://problem/10840441>
Comment 1 Jon Lee 2012-02-23 22:52:57 PST
Created attachment 128656 [details]
Better test case

Made a better test case, with both search and text fields. It has something to do with specifying both left:0 and right:0.
Comment 2 Kent Tamura 2012-02-24 00:14:28 PST
Thank you for the report.  I'll take a look at this next week.
Comment 3 Kent Tamura 2012-02-27 01:33:54 PST
The value also shakes.
Comment 4 Kent Tamura 2012-02-29 01:53:40 PST
Created attachment 129417 [details]
Patch
Comment 5 Kent Tamura 2012-02-29 01:58:26 PST
Created attachment 129418 [details]
Patch 2

Removed a TAB in the test
Comment 6 Kent Tamura 2012-02-29 21:06:34 PST
Morita-san, can you look at the patch?
Comment 7 WebKit Review Bot 2012-03-01 19:35:59 PST
Comment on attachment 129418 [details]
Patch 2

Rejecting attachment 129418 [details] from commit-queue.

New failing tests:
fast/repaint/search-field-cancel.html
Full output: http://queues.webkit.org/results/11751702
Comment 8 Kent Tamura 2012-03-01 21:34:47 PST
(In reply to comment #7)
> Rejecting attachment 129418 [details] from commit-queue.
> 
> New failing tests:
> fast/repaint/search-field-cancel.html

It's possible that this patch will change the behavior of the test though I had no difference on Mac.
I'm landing the patch with a test_expectation.txt adjustment, and will see the bot results.
Comment 9 Kent Tamura 2012-03-01 21:43:16 PST
Committed r109503: <http://trac.webkit.org/changeset/109503>