Bug 87137 - Layout Test fast/block/margin-collapse/103.html is failing (layout assertion)
Summary: Layout Test fast/block/margin-collapse/103.html is failing (layout assertion)
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Emil A Eklund
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-22 09:09 PDT by Joshua Bell
Modified: 2012-12-06 13:42 PST (History)
2 users (show)

See Also:


Attachments
Reduced test case (537 bytes, text/html)
2012-05-22 17:55 PDT, Emil A Eklund
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Bell 2012-05-22 09:09:41 PDT
The following layout test is failing on [insert platform]

fast/block/margin-collapse/103.html

Probable cause:

WebKit blame range: http://trac.webkit.org/log/?verbose=on&rev=117821&stop_rev=117815

Abbreviated stack is:

STDERR: ASSERTION FAILED: newLogicalTop >= logicalTop
STDERR: Backtrace:
STDERR: 	WebKit::WebURLRequest::hasUserGesture [0x0247F0EE+11353182]
...
STDERR: 	WebKit::WebURLRequest::hasUserGesture [0x02EBCBB6+22091558]
STDERR: 	webkit_glue::WebURLResponseExtraDataImpl::~WebURLResponseExtraDataImpl [0x099788AE+949584]
STDERR: 	(No symbol) [0x005071B9]
...
STDERR: 	(No symbol) [0x0050DFFC]
STDERR: 	base::SharedMemory::CreateAnonymous [0x00B67B7F+364615]
...
STDERR: 	base::SharedMemory::CreateAnonymous [0x00B6EC70+393528]
STDERR: 	(No symbol) [0x004ABB8D]
...
STDERR: 	(No symbol) [0x00500A0F]
STDERR: 	RegisterWaitForInputIdle [0x7C817077+73]
Test results:

http://build.chromium.org/f/chromium/layout_test_results/Webkit_Win__dbg__1_/138145/layout-test-results.zip
Comment 1 Joshua Bell 2012-05-22 09:10:18 PDT
Subpixel layout is the most likely culprit given the range and assert. Assigning to eae@ for investigation.
Comment 2 Joshua Bell 2012-05-22 09:10:50 PDT
Also, only failing on webkit win debug.
Comment 4 Emil A Eklund 2012-05-22 17:55:59 PDT
Created attachment 143420 [details]
Reduced test case

The crash seems to be window size specific. With the reduced test case (attached) it doesn't crash when loaded but as soon as the window is made small enough to toggle scrolling
Comment 5 Joshua Bell 2012-12-06 13:42:02 PST
No longer showing on flakiness dashboard. Presumably fixed, re-open at will.