RESOLVED FIXED 118965
segfault in RenderLayerCompositor when the iframe's position attribute is changed and it embeds <object>. 
https://bugs.webkit.org/show_bug.cgi?id=118965
Summary segfault in RenderLayerCompositor when the iframe's position attribute is cha...
zalan
Reported 2013-07-22 05:47:49 PDT
1. visit http://highwaymedianew.squarespace.com 2. resize window to a small size 3. crash when windows is about 400px wide
Attachments
test reduction (502 bytes, text/html)
2013-07-22 05:48 PDT, zalan 		no flags
Details
Patch (5.04 KB, patch)
2013-07-22 12:39 PDT, zalan 		no flags
DetailsFormatted DiffDiff
Patch (5.03 KB, patch)
2013-07-22 12:41 PDT, zalan 		no flags
DetailsFormatted DiffDiff
Patch (8.62 KB, patch)
2013-07-22 14:13 PDT, zalan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
zalan
Comment 1 2013-07-22 05:48:11 PDT
<rdar://problem/14403809>
zalan
Comment 2 2013-07-22 05:48:56 PDT
Created attachment 207241 [details] test reduction
zalan
Comment 3 2013-07-22 12:39:02 PDT
Created attachment 207263 [details] Patch
zalan
Comment 4 2013-07-22 12:41:39 PDT
Created attachment 207264 [details] Patch
Simon Fraser (smfr)
Comment 5 2013-07-22 13:03:44 PDT
Comment on attachment 207264 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=207264&action=review > LayoutTests/fast/frames/resources/plugin.html:1 > +<object type='application/x-shockwave-flash'></object> You should use the test Netscape plugin, with its drawing mode set to make it composited. We should not require Flash to be installed for layout tests.
zalan
Comment 6 2013-07-22 13:13:10 PDT
(In reply to comment #5) > (From update of attachment 207264 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=207264&action=review > > > LayoutTests/fast/frames/resources/plugin.html:1 > > +<object type='application/x-shockwave-flash'></object> > > You should use the test Netscape plugin, with its drawing mode set to make it composited. We should not require Flash to be installed for layout tests. Indeed. I was under the impression that it does not actually need the plugin to be installed, but going with the Netscape plugin is anyway a better way of doing it. Thanks!
zalan
Comment 7 2013-07-22 14:13:37 PDT
Created attachment 207278 [details] Patch
WebKit Commit Bot
Comment 8 2013-07-22 15:08:53 PDT
Comment on attachment 207278 [details] Patch Clearing flags on attachment: 207278 Committed r153003: <http://trac.webkit.org/changeset/153003>
WebKit Commit Bot
Comment 9 2013-07-22 15:08:57 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.