Bug 57347 - [Snow Leopard WebKit2 Release Tests] fast/images/move-image-to-new-document.html timing out since r82087
Summary: [Snow Leopard WebKit2 Release Tests] fast/images/move-image-to-new-document.h...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.6
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, LayoutTestFailure, PlatformOnly, Regression
Depends on:
Blocks:
 
Reported: 2011-03-29 09:07 PDT by Jessie Berlin
Modified: 2011-04-14 16:02 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.34 KB, patch)
2011-03-29 23:13 PDT, Andy Estes
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jessie Berlin 2011-03-29 09:07:55 PDT
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(WebKit2%20Tests)/r82085%20(10076)/results.html
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(WebKit2%20Tests)/r82087%20(10077)/results.html

This, surprisingly, points to http://trac.webkit.org/changeset/82087 as the culprit.

Since it is just timing out and not flakey, we should probably land expected failing results.
Comment 1 Jessie Berlin 2011-03-29 09:19:14 PDT
<rdar://problem/9202498>
Comment 2 Adam Roben (:aroben) 2011-03-29 10:49:34 PDT
(In reply to comment #0)
> Since it is just timing out and not flakey, we should probably land expected failing results.

If it's timing out, we should skip it. Tests take a long time to time out, and it's better for everyone not to slow down the tests because of them.
Comment 3 Jessie Berlin 2011-03-29 11:29:44 PDT
Added to the mac-wk2 Skipped list in http://trac.webkit.org/changeset/82292.
Comment 4 Andy Estes 2011-03-29 23:03:30 PDT
The timeout is actually expected now after <http://trac.webkit.org/changeset/82001> and the test needs to be changed to behave correctly. It just so happened that <http://trac.webkit.org/changeset/82087> exposed it by correcting a MIME type registration issue between QuickTime and WebKit2.

I'll change the test to call setPluginsDisabled(), which will give the embed tag in this test its old image loading behavior. Unfortunately setPluginsDisabled isn't implemented in WebKit2's LayoutTestController, so it'll still have to be skipped until that bug is fixed.
Comment 5 Andy Estes 2011-03-29 23:10:57 PDT
Sorry, I meant to say setPluginsEnabled(false). There is no such thing as setPluginsDisabled().
Comment 6 Andy Estes 2011-03-29 23:13:20 PDT
Created attachment 87467 [details]
Patch
Comment 7 Andy Estes 2011-03-29 23:52:11 PDT
Committed r82399: <http://trac.webkit.org/changeset/82399>
Comment 8 Andy Estes 2011-04-14 16:02:12 PDT
https://bugs.webkit.org/show_bug.cgi?id=58593 tracks implementing layoutTestController.setPluginsEnabled() in WebKitTestRunner, which will allow us to un-skip this test and two others.