Bug 46513

Summary:

REGRESSION (r68054): Buttons no longer render at correct sizes

Product:

WebKit

Reporter:

Mark Rowe (bdash) <mrowe>

Component:

Forms

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, ap, darin, eric, mitz, mrowe, webkit.review.bot

Priority:

Keywords:

InRadar, Regression

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

URL:

data:text/html,%3Cinput%20type=%22submit%22%3E%20%3Cinput%20type=%22submit%22%20style=%22font-size:%2010px%22%3E

Attachments:

Description	Flags
Patch	mitz: review+

Description Mark Rowe (bdash) 2010-09-24 14:23:22 PDT

After <http://trac.webkit.org/changeset/68054> it looks as though some amount of state is not being reset between the handling of independent buttons.  Typically buttons expressed as <input type="submit" style="font-size: 10px”> render at a shorter height.  If it’s on a page following a regular sized button you’ll see that its rendering at the wrong height.

Comment 1 Mark Rowe (bdash) 2010-09-24 14:24:16 PDT

<rdar://problem/8476536>

Comment 2 Darin Adler 2010-09-24 18:33:48 PDT

One thing I noticed in http://trac.webkit.org/changeset/68054 is that I got the logic backwards in this expression: "((states & DefaultState) ? nonDefaultButton : defaultButton)". But I don’t see anything else yet.

Comment 3 Darin Adler 2010-09-27 17:11:00 PDT

Created attachment 68994 [details]
Patch

Comment 4 Darin Adler 2010-09-27 17:40:27 PDT

Committed r68459: <http://trac.webkit.org/changeset/68459>

Comment 5 WebKit Review Bot 2010-09-27 17:48:21 PDT

http://trac.webkit.org/changeset/68459 might have broken Qt Linux Release minimal, Qt Linux ARMv5 Release, and Qt Linux ARMv7 Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/68459
http://trac.webkit.org/changeset/68460