Bug 59324

Summary: [Windows and Leopard Tests] media/invalid-media-url-crash.html failing since r84762
Product: WebKit Reporter: Jessie Berlin <jberlin>
Component: Page LoadingAssignee: 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 Flags
Patch abarth: review+

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]
Patch
Comment 7 Maciej Stachowiak 2011-04-26 14:43:45 PDT
Committed r84959: <http://trac.webkit.org/changeset/84959>