Bug 87631 - REGRESSION(r118618): fast/dom/beforeload/image-object-before-load* fail
Summary: REGRESSION(r118618): fast/dom/beforeload/image-object-before-load* fail
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
: 88092 (view as bug list)
Depends on:
Blocks: 35377 79666
  Show dependency tree / graph
 
Reported: 2012-05-28 01:08 PDT by Chris Dumez
Modified: 2012-06-10 05:44 PDT (History)
5 users (show)

See Also:

Attachments
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 Chris Dumez 2012-05-28 01:08:37 PDT 
The two following test are failing on EFL port after r118618 (Bug 35377):
  fast/dom/beforeload/image-object-before-load-innerHTML.html = TEXT
  fast/dom/beforeload/image-object-before-load.html = TEXT

The test fail because the style.width / style.height are '0px' instead of 'auto'.
The width/height of '0px' seems to indicate that the image load was actually prevented by the beforeload listener (as expected). Otherwise, the height/width would be '16px'.

The reason why the width/height is '0px' instead of 'auto' is because the object is recognized as being 'replaced' (See bug 61117 which introduces the 'auto' behavior for for non-replaced inline elements).
Comment 1 Csaba Osztrogonác 2012-06-01 07:32:35 PDT 
These tests fail on EFL and on Qt too.
Comment 2 Csaba Osztrogonác 2012-06-01 07:32:51 PDT 
*** Bug 88092 has been marked as a duplicate of this bug. ***
Comment 3 Csaba Osztrogonác 2012-06-01 07:38:18 PDT 
I skipped them on Qt-WK: http://trac.webkit.org/changeset/119230/trunk/LayoutTests/platform/qt-5.0-wk2/Skipped

Please unskip them with the proper fix.
Comment 4 Chris Dumez 2012-06-10 05:44:11 PDT 
r118618 has been rolled out and those test cases are now passing again. Closing the bug.