Bug 114311 - REGRESSION (142152): ensure we skip past out-of-flow objects when detecting whitespace to ignore after leading empty inlines
Summary: REGRESSION (142152): ensure we skip past out-of-flow objects when detecting w...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Robert Hogan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-09 15:54 PDT by Emil A Eklund
Modified: 2013-04-11 12:15 PDT (History)
6 users (show)

See Also:

Attachments
minimal test case (386 bytes, text/html)
2013-04-09 15:54 PDT, Emil A Eklund 		no flags Details
Patch (6.83 KB, patch)
2013-04-10 13:34 PDT, Robert Hogan 		no flags Details | Formatted Diff | Diff
Patch (6.92 KB, patch)
2013-04-10 14:21 PDT, Robert Hogan 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Emil A Eklund 2013-04-09 15:54:11 PDT 
Since r142152 we incorrectly layout out of flow content inside an inline-blocks with vertical-align.

What steps will reproduce the problem?
1. Go to http://join.iatn.net/
2. Notice in 26.0.1410.43 that the placeholder text shows up unusually high in our boxes. (Note that this isn't actual HTML5 placeholder attribute text, but a custom development to support all browsers.)

Prior to this version of Chrome, it rendered perfectly fine. The second screenshot shows how it renders in Safari 6.0.1 (8536.26.14), and that's also how it looked prior to this release of Chrome.

Originally reported for chromium in: https://code.google.com/p/chromium/issues/detail?id=226246
Comment 1 Emil A Eklund 2013-04-09 15:54:30 PDT 
Created attachment 197184 [details]
minimal test case
Comment 2 Robert Hogan 2013-04-10 13:34:20 PDT 
Created attachment 197397 [details]
Patch
Comment 3 Robert Hogan 2013-04-10 14:21:28 PDT 
Created attachment 197404 [details]
Patch
Comment 4 WebKit Commit Bot 2013-04-11 12:15:47 PDT 
Comment on attachment 197404 [details]
Patch

Clearing flags on attachment: 197404

Committed r148223: <http://trac.webkit.org/changeset/148223>
Comment 5 WebKit Commit Bot 2013-04-11 12:15:49 PDT 
All reviewed patches have been landed.  Closing bug.