Bug 59324 - [Windows and Leopard Tests] media/invalid-media-url-crash.html failing since r84762
Summary: [Windows and Leopard Tests] media/invalid-media-url-crash.html failing since ...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 528+ (Nightly build)
Hardware: All Windows 7
: P2 Normal
Assignee: Maciej Stachowiak
Keywords: LayoutTestFailure, PlatformOnly, Regression
Depends on:
Reported: 2011-04-25 08:42 PDT by Jessie Berlin
Modified: 2011-04-26 14:43 PDT (History)
7 users (show)

See Also:

Patch (1.88 KB, patch)
2011-04-26 14:42 PDT, Maciej Stachowiak
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Jessie Berlin 2011-04-26 12:42:43 PDT
Added the test to the Windows skipped list in http://trac.webkit.org/changeset/84942
Comment 2 Jessie Berlin 2011-04-26 13:19:33 PDT
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.
Comment 3 Alexey Proskuryakov 2011-04-26 13:27:04 PDT
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.
Comment 4 Alexey Proskuryakov 2011-04-26 13:28:19 PDT
I'm wondering how many other tests used this technique, and no longer test what they were supposed to.
Comment 5 Jessie Berlin 2011-04-26 13:40:49 PDT
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.
Comment 6 Maciej Stachowiak 2011-04-26 14:42:20 PDT
Created attachment 91166 [details]
Comment 7 Maciej Stachowiak 2011-04-26 14:43:45 PDT
Committed r84959: <http://trac.webkit.org/changeset/84959>