Bug 79445

Summary:

REGRESSION(90089): Input type='search' text shakes up and down when animating

Product:

WebKit

Reporter:

Jon Lee <jonlee>

Component:

Layout and Rendering

Assignee:

Kent Tamura <tkent>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov, morrita, tkent, webkit-bug-importer, webkit.review.bot

Priority:

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-

Jon Lee

Reported 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>

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 attachment proposed patch, testcase, etc.

Jon Lee

Comment 1 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.

Kent Tamura

Comment 2 2012-02-24 00:14:28 PST

Thank you for the report. I'll take a look at this next week.

Kent Tamura

Comment 3 2012-02-27 01:33:54 PST

The value also shakes.

Kent Tamura

Comment 4 2012-02-29 01:53:40 PST

Created attachment 129417 [details] Patch

Kent Tamura

Comment 5 2012-02-29 01:58:26 PST

Created attachment 129418 [details] Patch 2 Removed a TAB in the test

Kent Tamura

Comment 6 2012-02-29 21:06:34 PST

Morita-san, can you look at the patch?

WebKit Review Bot

Comment 7 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

Kent Tamura

Comment 8 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.

Kent Tamura

Comment 9 2012-03-01 21:43:16 PST

Committed r109503: <http://trac.webkit.org/changeset/109503>

Note You need to log in before you can comment on or make changes to this bug.