Bug 54106 - frames/flattening/iframe-flattening-crash.html fails on GTK.
Summary: frames/flattening/iframe-flattening-crash.html fails on GTK.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-09 07:40 PST by Yael
Modified: 2011-02-09 11:06 PST (History)
2 users (show)

See Also:


Attachments
Patch. (3.61 KB, patch)
2011-02-09 07:49 PST, Yael
mrobinson: review+
Details | Formatted Diff | Diff
Patch. (4.19 KB, patch)
2011-02-09 09:09 PST, Yael
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yael 2011-02-09 07:40:07 PST
This test is constructed differently then other frame flattening tests.
Seems that the frame flattening flag is turned on too late, so layout is not triggered and the test fails.
I modified the test and would appreciate it if someone with WebKitGTK could test it and let me know if the new format works. 
A patch is coming.
Comment 1 Yael 2011-02-09 07:49:52 PST
Created attachment 81814 [details]
Patch.

Modified the test to be more consistent with the other frame flattening tests.
Martin, could you or someone with WebKitGTK please test if it fixes the WebKitGTK problem? thanks.
Comment 2 Martin Robinson 2011-02-09 08:48:22 PST
Comment on attachment 81814 [details]
Patch.

This test is passing now on GTK+. In your patch do you also think you could remove the file from the GTK+ skipped list. It's currently the last skipped test. Thanks for the fix!
Comment 3 Yael 2011-02-09 08:58:09 PST
(In reply to comment #2)
> (From update of attachment 81814 [details])
> This test is passing now on GTK+. In your patch do you also think you could remove the file from the GTK+ skipped list. It's currently the last skipped test. Thanks for the fix!

Thanks for testing. I will update the patch to remove the skipped test.
Comment 4 Yael 2011-02-09 09:09:07 PST
Created attachment 81820 [details]
Patch.
Comment 5 WebKit Commit Bot 2011-02-09 11:06:34 PST
Comment on attachment 81820 [details]
Patch.

Clearing flags on attachment: 81820

Committed r78085: <http://trac.webkit.org/changeset/78085>
Comment 6 WebKit Commit Bot 2011-02-09 11:06:39 PST
All reviewed patches have been landed.  Closing bug.