57347 2011-03-29 09:07:55 -0700 [Snow Leopard WebKit2 Release Tests] fast/images/move-image-to-new-document.html timing out since r82087 2011-04-14 16:02:12 -0700 1 1 1 Unclassified WebKit Plug-ins 528+ (Nightly build) Mac OS X 10.6 RESOLVED FIXED InRadar, LayoutTestFailure, PlatformOnly, Regression P2 Normal --- 1 jberlin webkit-unassigned aestes aroben jberlin mjs oldest_to_newest 375403 0 jberlin 2011-03-29 09:07:55 -0700 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. 375415 1 jberlin 2011-03-29 09:19:14 -0700 <rdar://problem/9202498> 375523 2 aroben 2011-03-29 10:49:34 -0700 (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. 375580 3 jberlin 2011-03-29 11:29:44 -0700 Added to the mac-wk2 Skipped list in http://trac.webkit.org/changeset/82292. 376049 4 aestes 2011-03-29 23:03:30 -0700 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. 376051 5 aestes 2011-03-29 23:10:57 -0700 Sorry, I meant to say setPluginsEnabled(false). There is no such thing as setPluginsDisabled(). 376052 6 87467 aestes 2011-03-29 23:13:20 -0700 Created attachment 87467 Patch 376076 7 aestes 2011-03-29 23:52:11 -0700 Committed r82399: <http://trac.webkit.org/changeset/82399> 386145 8 aestes 2011-04-14 16:02:12 -0700 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. 87467 2011-03-29 23:13:20 -0700 2011-03-29 23:23:03 -0700 Patch bug-57347-20110329231318.patch text/plain 2394 aestes SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDgyMzkyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTEtMDMtMjkgIEFuZHkgRXN0ZXMgIDxhZXN0 ZXNAYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg ICAgICAgIFtTbm93IExlb3BhcmQgV2ViS2l0MiBSZWxlYXNlIFRlc3RzXSBmYXN0L2ltYWdlcy9t b3ZlLWltYWdlLXRvLW5ldy1kb2N1bWVudC5odG1sIHRpbWluZyBvdXQgc2luY2UgcjgyMDg3Cisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01NzM0NworCisg ICAgICAgICogZmFzdC9pbWFnZXMvc2NyaXB0LXRlc3RzL21vdmUtaW1hZ2UtdG8tbmV3LWRvY3Vt ZW50LmpzOiBEaXNhYmxlCisgICAgICAgIHBsdWctaW5zIHNvIHRoYXQgdGhlIGVtYmVkIHBvcnRp b24gb2YgdGhlIHRlc3QgZ2V0cyB0aGUgZXhwZWN0ZWQgaW1hZ2UKKyAgICAgICAgbG9hZGluZyBi ZWhhdmlvciBvbiBib3RzIHdpdGggUXVpY2tUaW1lIGluc3RhbGxlZC4KKyAgICAgICAgKiBwbGF0 Zm9ybS9tYWMtd2syL1NraXBwZWQ6IFNraXAgdGhlIHRlc3QgaW4gV2ViS2l0MiBzaW5jZSBpdCBk b2Vzbid0CisgICAgICAgIGltcGxlbWVudCBzZXRQbHVnaW5zRW5hYmxlZCgpLgorCiAyMDExLTAz LTI5ICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAgVXBkYXRlIFNu b3cgTGVvcGFyZCBiYXNlbGluZXMuCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9zY3Jp cHQtdGVzdHMvbW92ZS1pbWFnZS10by1uZXctZG9jdW1lbnQuanMKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5 b3V0VGVzdHMvZmFzdC9pbWFnZXMvc2NyaXB0LXRlc3RzL21vdmUtaW1hZ2UtdG8tbmV3LWRvY3Vt ZW50LmpzCShyZXZpc2lvbiA4MjM4NSkKKysrIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL3Njcmlw dC10ZXN0cy9tb3ZlLWltYWdlLXRvLW5ldy1kb2N1bWVudC5qcwkod29ya2luZyBjb3B5KQpAQCAt MSw1ICsxLDggQEAKIGRlc2NyaXB0aW9uKCJUZXN0IHRoYXQgd2hlbiBpbWFnZXMgYXJlIG1vdmVk IHRvIGEgbmV3IGRvY3VtZW50LCBhIG5ldyBsb2FkIGZpcmVzLiIpOwogCitpZiAod2luZG93Lmxh eW91dFRlc3RDb250cm9sbGVyKQorICAgIGxheW91dFRlc3RDb250cm9sbGVyLnNldFBsdWdpbnNF bmFibGVkKGZhbHNlKTsKKwoganNUZXN0SXNBc3luYyA9IHRydWU7CiAKIHZhciBzdWJmcmFtZSA9 IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoImlmcmFtZSIpOwpJbmRleDogTGF5b3V0VGVzdHMvcGxh dGZvcm0vbWFjLXdrMi9Ta2lwcGVkCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsYXRmb3Jt L21hYy13azIvU2tpcHBlZAkocmV2aXNpb24gODIzODUpCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9y bS9tYWMtd2syL1NraXBwZWQJKHdvcmtpbmcgY29weSkKQEAgLTE3OTEsOSArMTc5MSw2IEBAIGh0 dHAvdGVzdHMvaW5zcGVjdG9yL3Jlc291cmNlLWhhci1jb252ZXIKICMgaHR0cDovL3dlYmtpdC5v cmcvYi81NjU4NAogaW5zcGVjdG9yL2RlYnVnZ2VyL2RlYnVnLWlubGluZWQtc2NyaXB0cy5odG1s CiAKLSMgVGltZXMgb3V0IGh0dHA6Ly93ZWJraXQub3JnL2IvNTczNDcKLWZhc3QvaW1hZ2VzL21v dmUtaW1hZ2UtdG8tbmV3LWRvY3VtZW50Lmh0bWwKLQogIyMjIEVORCBPRiAoMSkgQ2xhc3NpZmll ZCBmYWlsdXJlcyB3aXRoIGJ1ZyByZXBvcnRzCiAjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMj IyMjIyMjIyMjIyMjCiAKQEAgLTE4NDAsNiArMTgzNyw3IEBAIGZhc3QvZXZlbnRzL29uYmVmb3Jl dW5sb2FkLWZvY3VzZWQtaWZyYW0KICMgV2ViS2l0VGVzdFJ1bm5lciBuZWVkcyBsYXlvdXRUZXN0 Q29udHJvbGxlci5zZXRQbHVnaW5zRW5hYmxlZAogcGxhdGZvcm0vbWFjL3BsdWdpbnMvZGlzYWJs ZS1wbHVnaW5zLmh0bWwKIGZhc3QvaW1hZ2VzL2VtYmVkLWltYWdlLmh0bWwKK2Zhc3QvaW1hZ2Vz L21vdmUtaW1hZ2UtdG8tbmV3LWRvY3VtZW50Lmh0bWwKIAogIyBXZWJLaXRUZXN0UnVubmVyIG5l ZWRzIGxheW91dFRlc3RDb250cm9sbGVyLnNldEhhbmRsZXNBdXRoZW50aWNhdGlvbkNoYWxsZW5n ZXMKIGh0dHAvdGVzdHMvbWlzYy9hdXRoZW50aWNhdGlvbi1zZW50LXRvLXJlZGlyZWN0Lmh0bWwK