Bug 54150

Summary: Repro crash with Sony Google TV ad at Gizmodo
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, pnormand, zmo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on: 54187    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch mitz: review+

Description Anders Carlsson 2011-02-09 15:32:15 PST
Repro crash with Sony Google TV ad at Gizmodo
Comment 1 Anders Carlsson 2011-02-09 15:37:24 PST
Created attachment 81886 [details]
Patch
Comment 2 Anders Carlsson 2011-02-09 15:47:15 PST
Committed r78157: <http://trac.webkit.org/changeset/78157>
Comment 3 Zhenyao Mo 2011-02-09 17:27:23 PST
The test is TIMEOUT on all chromium bots.
Comment 4 Philippe Normand 2011-02-10 01:39:38 PST
also times out on all GTK+ bots
Comment 5 Philippe Normand 2011-02-10 02:01:09 PST
(In reply to comment #3)
> The test is TIMEOUT on all chromium bots.

Adding Tools/DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSDestroyingPluginFromDestroyStream.cpp

to your build should fix the test, this is what I did on GTK and the test passes now.
Comment 6 Eric Seidel (no email) 2011-02-10 02:37:04 PST
This is failing on the Snow Leopard bot.
Comment 7 Eric Seidel (no email) 2011-02-10 02:49:21 PST
Reverted r78157 for reason:

Fails on a bunch of bots

Committed r78185: <http://trac.webkit.org/changeset/78185>
Comment 8 Anders Carlsson 2011-02-10 09:48:13 PST
(In reply to comment #7)
> Reverted r78157 for reason:
> 
> Fails on a bunch of bots
> 
> Committed r78185: <http://trac.webkit.org/changeset/78185>

This change was WebKit2-specific, and if the test times out for other ports it's better to add the test to the Skipped list and file new bugs.
Comment 9 Eric Seidel (no email) 2011-02-10 15:22:34 PST
I reverted it based on all the bots being red.  But with further investigation it could have probably been skipped for other ports instead. But seems like something better done when landing.

Also if this is re-landed, there was a Gtk build fix which was landed right after this which should be included in any re-landing.
Comment 10 Anders Carlsson 2011-02-10 18:47:55 PST
Created attachment 82087 [details]
Patch
Comment 11 Anders Carlsson 2011-02-10 18:57:49 PST
Committed r78299: <http://trac.webkit.org/changeset/78299>