Bug 15169 - Freeze on Google maps using the "Dig a hole" tool
Summary: Freeze on Google maps using the "Dig a hole" tool
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P1 Major
Assignee: mitz
URL: http://maps.google.com
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2007-09-10 03:24 PDT by Alex Stapleton
Modified: 2007-09-18 11:42 PDT (History)
2 users (show)

See Also:


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+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Stapleton 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.
Comment 1 Alex Stapleton 2007-09-10 03:24:55 PDT
Created attachment 16242 [details]
Sample of Safari during a freeze
Comment 2 David Kilzer (:ddkilzer) 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.
Comment 3 David Kilzer (:ddkilzer) 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.
Comment 4 mitz 2007-09-12 12:23:33 PDT
I have a reduction and a possible fix.
Comment 5 Mark Rowe (bdash) 2007-09-12 12:29:24 PDT
<rdar://problem/5477414>
Comment 6 mitz 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.
Comment 7 Darin Adler 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
Comment 8 Sam Weinig 2007-09-18 11:42:04 PDT
Landed in r25619.