Summary: | [Windows and Leopard Tests] media/invalid-media-url-crash.html failing since r84762 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jessie Berlin <jberlin> | ||||
Component: | Page Loading | Assignee: | Maciej Stachowiak <mjs> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, aroben, eric.carlson, inferno, jberlin, mjs, staikos | ||||
Priority: | P2 | Keywords: | LayoutTestFailure, PlatformOnly, Regression | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | Windows 7 | ||||||
Attachments: |
|
Description
Jessie Berlin
2011-04-25 08:42:46 PDT
Added the test to the Windows skipped list in http://trac.webkit.org/changeset/84942 This is also timing out on Leopard as of r84762 and is already on the mac-snowleopard skipped list. I am starting to wonder if the right thing to do is disable this test. This test needs to be updated now that we treat http:aa\0%aa#aa as valid URL. But I'm not sure what URLs will stay reliably invalid. I'm wondering how many other tests used this technique, and no longer test what they were supposed to. Maciej is going to try to first fix the test replacing the invalid URL with !:, which should really be invalid, and then going to try to find out why the check for invalid URLs was added in the code. Created attachment 91166 [details]
Patch
Committed r84959: <http://trac.webkit.org/changeset/84959> |