3552 2005-06-15 14:44:44 -0700 Anomaly (infinite loop) with window.location.href='#foo' 2006-02-08 11:09:01 -0800 1 1 1 Unclassified WebKit JavaScriptCore 412 Mac OS X 10.4 VERIFIED DUPLICATE 7058 http://www.s-seven.net/safari_bug.html HasReduction P2 Normal --- 7080 1 ben mjs joost sullivan oldest_to_newest 12152 0 ben 2005-06-15 14:44:44 -0700 In Safari 1.3/2.0 and Safari 2.0 using the latest CVS WebKit as of this report, executing the javascipt window.location.href='#foo' causes the activity indicator to spin constantly, even though the page appears to have loaded successfully and works as designed. This is especially visible when tabbed browsing is enabled as there is an activity indicator on the tab. It also appears that the "Stop" toolbar button never turns back into the "Reload Page" button. While this is happening, if you select "Reload Page" from the "View" menu, it appears as though the page attempts to load over and over again. Steps to Reproduce: Create an html document with the following: <html><body> <h3 id="h_value">h_value</h3><p>This is under h_value</p> <script type="text/javascript">window.location.href='#h_value';</script> </body></html> Expected Results: The page should load and the activity indicator should stop spinning. Actual Results: The page loads and the activity indicator does not stop spinning and if reload page is selected from the view menu, the page loads in an infinite loop. Regression: The problem did not exist in Safari 1.2.4. Notes: An online example can be seen at http://www.s-seven.net/safari_bug.html 12155 1 2375 ben 2005-06-15 14:58:18 -0700 Created attachment 2375 Code to demonstrate Bug 3552. 12205 2 joost 2005-06-16 00:09:03 -0700 Confirmed, and an ugly one it is! 15747 3 3210 dacarson 2005-08-02 15:45:16 -0700 Created attachment 3210 Another example of defect. Similarly, you can't access the DOM after a call to location.hash. See attachement. If a style is added, such as: <style> DIV.t:first-letter {text-transform:uppercase;} </style> Safari will crash as it tries to update a DOM element that has disappeared. 28700 4 ben 2006-01-17 09:33:04 -0800 UPDATE: In WebKit SVN r12148, though there is nothing to indicate a change specific to this bug, hitting the "Stop" button or selecting the "Stop" menu item does stop the activity indicator and selecting the "Reload" page menu item doesn't seem to create the infinite loop, though the progress indicator doesn't stop on page load as it should indicating that there is still a problem here. 31420 5 joost 2006-02-05 01:18:44 -0800 The infinite loop part of this was filed as bug 7080, to be able to fix both the root cause of this bug and the fact that it goes into an infinite loop. 31806 6 ddkilzer 2006-02-07 21:21:25 -0800 This bug was fixed with the commit for Bug 7058 (confirmed locally by applying the patch in reverse and rebuilding r12652), so marking it a duplicate of that bug. Bug 6309 was also fixed at the same time. Reporter, please verify that this was fixed with nightly r12598 or newer. Should a test case be prepared and checked in for WebKit to make sure this doesn't regress? *** This bug has been marked as a duplicate of 7058 *** 31828 7 ben 2006-02-08 09:38:47 -0800 My test case is confirmed fixed with nightly r12598 or newer. 31834 8 sullivan 2006-02-08 10:40:01 -0800 Wow, what a pleasant surprise that this older bug was fixed by the change for 7080. The intention of 7080 was to reverse a regression caused by a very recent code change, so it's a surprise that it fixed any older case. I guess the new Timer reorganization that led to 7080 improved something about the flow of control here such that, when 7080 was fixed, some old problems were also fixed. 31836 9 ddkilzer 2006-02-08 11:09:01 -0800 Verified by reporter. 2375 2005-06-15 14:58:18 -0700 2005-06-15 14:58:18 -0700 Code to demonstrate Bug 3552. safari_bug_3552.html text/html 1694 ben PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+Cgo8aHRtbD4KPGhlYWQ+Cgo8c2NyaXB0 IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CmZ1bmN0aW9uIGZvbygpCnsKCXdpbmRvdy5sb2NhdGlv bi5ocmVmPScjaF92YWx1ZTYnOwp9Cjwvc2NyaXB0PgoKPC9oZWFkPgoKCjxib2R5IG9ubG9hZD0i amF2YXNjcmlwdDpmb28oKTsiPgo8aHI+CjxoMiBpZD0iaF92YWx1ZTEiPmhfdmFsdWUxPC9oMj4K PHA+VGhpcyBpcyB1bmRlciBoX3ZhbHVlMTwvcD4KPGJyPgo8aDIgaWQ9ImhfdmFsdWUyIj5oX3Zh bHVlMjwvaDI+CjxwPlRoaXMgaXMgdW5kZXIgaF92YWx1ZTI8L3A+Cjxicj4KPGgyIGlkPSJoX3Zh bHVlMyI+aF92YWx1ZTM8L2gyPgo8cD5UaGlzIGlzIHVuZGVyIGhfdmFsdWUzPC9wPgo8YnI+Cjxo MiBpZD0iaF92YWx1ZTQiPmhfdmFsdWU0PC9oMj4KPHA+VGhpcyBpcyB1bmRlciBoX3ZhbHVlNDwv cD4KPGJyPgo8aDIgaWQ9ImhfdmFsdWU1Ij5oX3ZhbHVlNTwvaDI+CjxwPlRoaXMgaXMgdW5kZXIg aF92YWx1ZTU8L3A+Cjxicj4KPGgyIGlkPSJoX3ZhbHVlNiI+aF92YWx1ZTY8L2gyPgo8cD5UaGlz IGlzIHVuZGVyIGhfdmFsdWU2PC9wPgo8aHI+ClRoaXMgaXMgYSBidWcgaW4gU2FmYXJpIDEuMyBh bmQgMi4wPGJyPgpXb3JrcyBpbiBTYWZhcmkgMS4yLjQ8YnI+Cjxicj4KPGNvZGU+PHByZT4KZnVu Y3Rpb24gZm9vKCkKewoJd2luZG93LmxvY2F0aW9uLmhyZWY9JyNoX3ZhbHVlNic7Cn0KPC9wcmU+ PC9jb2RlPgo8YnI+Cjx1bD4KPGxpPgpJZiB5b3UgaGF2ZSB0YWJiZWQgYnJvd3NpbmcgZW5hYmxl ZCwgbm90aWNlIGhvdyB0aGUgYWN0aXZpdHkgaW5kaWNhdG9yIGlzIHN0aWxsIHNwaW5uaW5nLgo8 L2xpPgo8bGk+Ck5vdGljZSBob3cgdGhlICJSZWxvYWQgUGFnZSIgdG9vbGJhciBpY29uIGlzIHN0 aWxsIGEgIlN0b3AiIGljb24gYW5kIGl0IGRvZXMgbm90aGluZy4KPC9saT4KPGxpPgpOb3RpY2Ug aG93IHRoZSAiU3RvcCIgbWVudSBpdGVtIG9uIHRoZSAiVmlldyIgbWVudSBkb2VzIG5vdGhpbmcu CjwvbGk+CjxsaT4KTm90aWNlIGhvdyB0aGUgIlJlbG9hZCBQYWdlIiBtZW51IGl0ZW0gb24gdGhl ICJWaWV3IiBtZW51IHdpbGwgY2F1c2UgdGhlIHBhZ2UgdG8gbG9hZCBpbmZpbml0ZWx5LiAgCjwv bGk+CjxsaT4KTW92aW5nIHRoZSBmb2N1cyB0byB0aGUgYWRkcmVzcyBlZGl0IGFuZCBwcmVzc2lu ZyAiUmV0dXJuIiB3aWxsIGFsc28gY2F1c2UgdGhlIHBhZ2UgdG8gbG9hZCBpbmZpbml0ZWx5LiAg CjwvbGk+CjwvdWw+Cjxocj4KPGgyIGlkPSJoX3ZhbHVlNyI+aF92YWx1ZTc8L2gyPgo8cD5UaGlz IGlzIHVuZGVyIGhfdmFsdWU3PC9wPgo8YnI+CjxoMiBpZD0iaF92YWx1ZTgiPmhfdmFsdWU4PC9o Mj4KPHA+VGhpcyBpcyB1bmRlciBoX3ZhbHVlODwvcD4KPGJyPgo8aDIgaWQ9ImhfdmFsdWU5Ij5o X3ZhbHVlOTwvaDI+CjxwPlRoaXMgaXMgdW5kZXIgaF92YWx1ZTk8L3A+Cjxicj4KPGgyIGlkPSJo X3ZhbHVlMTAiPmhfdmFsdWUxMDwvaDI+CjxwPlRoaXMgaXMgdW5kZXIgaF92YWx1ZTEwPC9wPgo8 YnI+CjxoMiBpZD0iaF92YWx1ZTExIj5oX3ZhbHVlMTE8L2gyPgo8cD5UaGlzIGlzIHVuZGVyIGhf dmFsdWUxMTwvcD4KPGJyPgo8aHI+CjwvYm9keT4KCjwvaHRtbD4KIAo= 3210 2005-08-02 15:45:16 -0700 2005-08-02 15:45:16 -0700 Another example of defect. location_test.html text/html 382 dacarson PGh0bWw+DQo8aGVhZD4NCjxzY3JpcHQ+DQpmdW5jdGlvbiB1cGRhdGVIZWFkaW5nKCkgew0KICAg IHZhciBlbCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdIZWFkaW5nJyk7DQogICAgZWwuaW5u ZXJIVE1MID0gImJlZm9yZSBjYWxsIHRvIGxvY2F0aW9uIjsNCiAgICBsb2NhdGlvbi5oYXNoPSIz IjsNCiAgICBlbC5pbm5lckhUTUwgPSAiYWZ0ZXIgY2FsbCI7DQp9DQoNCjwvc2NyaXB0Pg0KPC9o ZWFkPg0KPGJvZHk+DQoNCjxkaXYgY2xhc3M9dCBpZD0nSGVhZGluZyc+SGVhZGluZzwvZGl2Pjxi cj4NCjxhIGhyZWY9ImphdmFzY3JpcHQ6dXBkYXRlSGVhZGluZygpIj5qcyB1cGRhdGUgaGVhZGlu ZyBhbmQgbG9jYXRpb248L2E+PGJyPg0KPC9ib2R5Pg0KPC9odG1sPg==