Bug 79445 - REGRESSION(90089): Input type='search' text shakes up and down when animating
Summary: REGRESSION(90089): Input type='search' text shakes up and down when animating
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.7
: P2 Normal
Assignee: Kent Tamura
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-02-23 22:41 PST by Jon Lee
Modified: 2012-03-01 21:43 PST (History)
5 users (show)

See Also:

Attachments
Test case (840 bytes, text/html)
2012-02-23 22:41 PST, Jon Lee 		no flags Details
Better test case (916 bytes, text/html)
2012-02-23 22:52 PST, Jon Lee 		no flags Details
Patch (4.26 KB, patch)
2012-02-29 01:53 PST, Kent Tamura 		no flags Details | Formatted Diff | Diff
Patch 2 (4.27 KB, patch)
2012-02-29 01:58 PST, Kent Tamura 		dglazkov: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>