Bug 72436 - Layout Test http/tests/multipart/invalid-image-data.html is crashing
Summary: Layout Test http/tests/multipart/invalid-image-data.html is crashing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nate Chapin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-15 16:14 PST by Peter Kasting
Modified: 2011-11-16 14:11 PST (History)
3 users (show)

See Also:


Attachments
patch (4.10 KB, patch)
2011-11-16 11:18 PST, Nate Chapin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Kasting 2011-11-15 16:14:47 PST
The following layout test is failing an assertion on the Win dbg 2 canary (see http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fmultipart%2Finvalid-image-data.html ):

http/tests/multipart/invalid-image-data.html

ASSERTION FAILED: size == 0 || size >= m_encodedSize
(this is in CachedResource::setEncodedSize())

The most obvious change in the blame list is http://trac.webkit.org/changeset/100311/ .
Comment 1 Nate Chapin 2011-11-16 11:18:21 PST
Created attachment 115413 [details]
patch
Comment 2 Adam Barth 2011-11-16 11:20:36 PST
Comment on attachment 115413 [details]
patch

I'm not sure sendDataToResource is the perfect name, but I understand why you chose it.
Comment 3 WebKit Review Bot 2011-11-16 14:11:10 PST
Comment on attachment 115413 [details]
patch

Clearing flags on attachment: 115413

Committed r100503: <http://trac.webkit.org/changeset/100503>
Comment 4 WebKit Review Bot 2011-11-16 14:11:14 PST
All reviewed patches have been landed.  Closing bug.