RESOLVED FIXED Bug 15169
Freeze on Google maps using the "Dig a hole" tool 
https://bugs.webkit.org/show_bug.cgi?id=15169
Summary Freeze on Google maps using the "Dig a hole" tool
Alex Stapleton
Reported 2007-09-10 03:24:21 PDT
Under the My Maps tab on google maps there is a tool called "Dig a hole through the earth", checking the box to enable this function causes WebKit to freeze for about 30 seconds.
Attachments
Sample of Safari during a freeze (61.23 KB, text/plain)
2007-09-10 03:24 PDT, Alex Stapleton 		no flags
Details
Shark profile of Safari during hang (835.72 KB, application/octet-stream)
2007-09-10 04:38 PDT, David Kilzer (:ddkilzer) 		no flags
Details
Fix hang on accessing the empty string property of a CSSStyleDeclaration (3.71 KB, patch)
2007-09-12 12:58 PDT, mitz 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alex Stapleton
Comment 1 2007-09-10 03:24:55 PDT
Created attachment 16242 [details] Sample of Safari during a freeze
David Kilzer (:ddkilzer)
Comment 2 2007-09-10 04:35:21 PDT
Confirmed with a local debug build of WebKit r25458 with Safari 3 Public Beta v. 3.0.3 (522.12.1) on Mac OS X 10.4.10 (8R218). (It hangs a LOT longer on my poor PB G4!) Note that the "Dig a hole through the Earth" feature must be added to the "My Maps" section. It's a third-party item "Created by others". Click the "Browser the directory" link to find and add it.
David Kilzer (:ddkilzer)
Comment 3 2007-09-10 04:38:53 PDT
Created attachment 16243 [details] Shark profile of Safari during hang This is a shark profile of Safari while it was hanging. Looks like most of the time was spent in WebCore::DeprecatedString::at(unsigned) const, which is suggested by the sample in Attachment #16242 [details] as well.
mitz
Comment 4 2007-09-12 12:23:33 PDT
I have a reduction and a possible fix.
Mark Rowe (bdash)
Comment 5 2007-09-12 12:29:24 PDT
<rdar://problem/5477414>
mitz
Comment 6 2007-09-12 12:58:27 PDT
Created attachment 16268 [details] Fix hang on accessing the empty string property of a CSSStyleDeclaration Includes regression test.
Darin Adler
Comment 7 2007-09-12 13:01:28 PDT
Comment on attachment 16268 [details] Fix hang on accessing the empty string property of a CSSStyleDeclaration r=me
Sam Weinig
Comment 8 2007-09-18 11:42:04 PDT
Landed in r25619.
Note You need to log in before you can comment on or make changes to this bug.