59324 2011-04-25 08:42:46 -0700 [Windows and Leopard Tests] media/invalid-media-url-crash.html failing since r84762 2011-04-26 14:43:45 -0700 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) All Windows 7 RESOLVED FIXED LayoutTestFailure, PlatformOnly, Regression P2 Normal --- 1 jberlin mjs ap aroben eric.carlson inferno jberlin mjs staikos oldest_to_newest 391768 0 jberlin 2011-04-25 08:42:46 -0700 http://trac.webkit.org/changeset/84762 http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r84760%20(12065)/results.html http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r84762%20(12066)/results.html Interestingly, the WK1 bots are failing in a different way than the WK2 bots: WK1: http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r84775%20(12075)/media/invalid-media-url-crash-pretty-diff.html http://build.webkit.org/results/Windows%20XP%20Debug%20(Tests)/r84775%20(28015)/media/invalid-media-url-crash-pretty-diff.html WK2: http://build.webkit.org/results/Windows%207%20Release%20(WebKit2%20Tests)/r84776%20(5961)/media/invalid-media-url-crash-pretty-diff.html 392639 1 jberlin 2011-04-26 12:42:43 -0700 Added the test to the Windows skipped list in http://trac.webkit.org/changeset/84942 392654 2 jberlin 2011-04-26 13:19:33 -0700 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. 392658 3 ap 2011-04-26 13:27:04 -0700 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. 392660 4 ap 2011-04-26 13:28:19 -0700 I'm wondering how many other tests used this technique, and no longer test what they were supposed to. 392663 5 jberlin 2011-04-26 13:40:49 -0700 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. 392743 6 91166 mjs 2011-04-26 14:42:20 -0700 Created attachment 91166 Patch 392755 7 mjs 2011-04-26 14:43:45 -0700 Committed r84959: <http://trac.webkit.org/changeset/84959> 91166 2011-04-26 14:42:20 -0700 2011-04-26 14:43:06 -0700 Patch bug-59324-20110426144218.patch text/plain 1928 mjs SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDg0OTU4KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTEtMDQtMjYgIE1hY2llaiBTdGFjaG93aWFr ICA8bWpzQGFwcGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K KworICAgICAgICBbV2luZG93cyBhbmQgTGVvcGFyZCBUZXN0c10gbWVkaWEvaW52YWxpZC1tZWRp YS11cmwtY3Jhc2guaHRtbCBmYWlsaW5nIHNpbmNlIHI4NDc2MgorICAgICAgICBodHRwczovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTkzMjQKKyAgICAgICAgCisgICAgICAgIFVz ZSBhIHByb3Blcmx5IGludmFsaWQgVVJMLCBpbnN0ZWFkIG9mIGh0dHA6YWEuIFRoaXMgYWxzbyBh bGxvd3MgdGhlIHRlc3QgdG8KKyAgICAgICAgYmUgdW5za2lwcGVkIG9uIFNMLgorCisgICAgICAg ICogbWVkaWEvaW52YWxpZC1tZWRpYS11cmwtY3Jhc2guaHRtbDoKKyAgICAgICAgKiBwbGF0Zm9y bS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZDoKKwogMjAxMS0wNC0yNiAgU3RlcGhlbiBXaGl0ZSAg PHNlbm9yYmxhbmNvQGNocm9taXVtLm9yZz4KIAogICAgICAgICBVbnJldmlld2VkOyBjaHJvbWl1 bSB0ZXN0IGV4cGVjdGF0aW9ucyB1cGRhdGUuCkluZGV4OiBMYXlvdXRUZXN0cy9tZWRpYS9pbnZh bGlkLW1lZGlhLXVybC1jcmFzaC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL21lZGlh L2ludmFsaWQtbWVkaWEtdXJsLWNyYXNoLmh0bWwJKHJldmlzaW9uIDg0ODc4KQorKysgTGF5b3V0 VGVzdHMvbWVkaWEvaW52YWxpZC1tZWRpYS11cmwtY3Jhc2guaHRtbAkod29ya2luZyBjb3B5KQpA QCAtMyw3ICszLDcgQEAKICAgICAgICAgPHA+VGVzdHMgdGhhdCBpbnZhbGlkIG1lZGlhIHNyYyB1 cmwgZG9lcyBub3QgcmVzdWx0IGluIGNyYXNoLjwvcD4KICAgICAgICAgPHNjcmlwdCBzcmM9dmlk ZW8tdGVzdC5qcz48L3NjcmlwdD4KICAgICAgICAgPHNjcmlwdD4KLSAgICAgICAgICAgIHZhciBp bnZhbGlkX3VybCA9ICJodHRwOmFhIiArIFN0cmluZy5mcm9tQ2hhckNvZGUoMCkgKyAiJWFhI2Fh IjsKKyAgICAgICAgICAgIHZhciBpbnZhbGlkX3VybCA9ICIhOi8iICsgU3RyaW5nLmZyb21DaGFy Q29kZSgwKSArICIlYWEjYWEiOwogICAgICAgICAgICAgdmFyIGVycm9yX2NvdW50ID0gMDsKIAog ICAgICAgICAgICAgZnVuY3Rpb24gZXJyb3JFdmVudCgpCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0 Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9w bGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZAkocmV2aXNpb24gODQ4NzgpCisrKyBMYXlv dXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZAkod29ya2luZyBjb3B5KQpA QCAtMTc0LDYgKzE3NCw1IEBAIG1lZGlhL21lZGlhLWZ1bGxzY3JlZW4taW5saW5lLmh0bWwKIAog IyBLbm93biBTbm93IExlb3BhcmQtb25seSBjcmFzaGVyOiByZGFyOi8vOTA1NDcxNQogIyBodHRw czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTExMzgKLW1lZGlhL2ludmFsaWQt bWVkaWEtdXJsLWNyYXNoLmh0bWwKIGh0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1wcm9ncmVz cy5odG1sCiBodHRwL3Rlc3RzL21lZGlhL3ZpZGVvLWxvYWQtdHdpY2UuaHRtbAo=