25921 2009-05-21 02:12:30 -0700 REGRESSION: Using setTimeout.call or setTimeout.apply only works the first time is used 2009-05-21 17:29:10 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) All All RESOLVED FIXED HasReduction, InRadar, Regression P1 Normal --- 1 rhudea webkit-unassigned ap mrowe oliver steve zwarich oldest_to_newest 122300 0 rhudea 2009-05-21 02:12:30 -0700 I'll add a test case 122301 1 30533 rhudea 2009-05-21 02:13:58 -0700 Created attachment 30533 Simple test case 122314 2 mrowe 2009-05-21 03:47:09 -0700 Seems like one of Oliver's recent optimisations may have introduced this. 122315 3 mrowe 2009-05-21 03:47:26 -0700 <rdar://problem/6910264> 122360 4 mrowe 2009-05-21 09:14:26 -0700 *** Bug 23670 has been marked as a duplicate of this bug. *** 122417 5 zwarich 2009-05-21 14:02:45 -0700 This occurs in the 4.0 beta, so it's not Oliver's recent changes. 122421 6 oliver 2009-05-21 14:10:30 -0700 It also impacts the interpreter -- i think it may be a bug in op_resolve_global, but that doesn't explain why it only seems to effect .call and .apply 122424 7 oliver 2009-05-21 14:26:21 -0700 op_resolve_global is apparently broked. 122433 8 oliver 2009-05-21 15:03:58 -0700 Got it, we're caching a property on the global objects prototype chain as if it were directly on the global object 122479 9 oliver 2009-05-21 17:29:10 -0700 Committing to http://svn.webkit.org/repository/webkit/trunk ... M JavaScriptCore/ChangeLog M JavaScriptCore/interpreter/Interpreter.cpp M JavaScriptCore/jit/JITStubs.cpp M LayoutTests/ChangeLog A LayoutTests/fast/js/global-function-resolve-expected.txt A LayoutTests/fast/js/global-function-resolve.html A LayoutTests/fast/js/resources/global-function-resolve.js Committed r44016 30533 2009-05-21 02:13:58 -0700 2009-05-21 02:13:58 -0700 Simple test case test2.html text/html 577 rhudea DQo8aHRtbD4NCiAgPGhlYWQ+DQogICAgPHRpdGxlPlNldFRpbWVvdXQgVGVzdCBQYWdlPC90aXRs ZT4NCiANCiAgICA8c2NyaXB0Pg0KICAgIGZ1bmN0aW9uIF9zZXRUaW1lb3V0KHRoaXNPYmplY3Qs IGZ1bmMsIHQpIHsNCiAgICAgICAgc2V0VGltZW91dC5jYWxsKHRoaXNPYmplY3QsIGZ1bmMsIHQp Ow0KICAgIH0NCg0KICAgIGZ1bmN0aW9uIHN0YXJ0KCkgew0KICAgICAgICBfc2V0VGltZW91dCh3 aW5kb3csIHRlc3QxLCAwKTsNCiAgICB9DQoNCiAgICBmdW5jdGlvbiB0ZXN0MSgpIHsNCiAgICAg ICAgX3NldFRpbWVvdXQod2luZG93LCB0ZXN0MiwgMCk7DQogICAgfQ0KDQogICAgZnVuY3Rpb24g dGVzdDIoKSB7DQogICAgICAgIHZhciBkID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImxvZyIp Ow0KICAgICAgICBkLmlubmVySFRNTCA9ICJTVUNDRVNTIjsNCiAgICB9DQogDQogICAgPC9zY3Jp cHQ+DQogDQogIDwvaGVhZD4NCiANCiAgPGJvZHkgb25sb2FkPSJzdGFydCgpIj4NCiANCiAgICA8 ZGl2IGlkPSJsb2ciPg0KICAgICAgICBGQUlMDQogICAgPC9kaXY+DQogIDwvYm9keT4NCiANCjwv aHRtbD4NCg==