[Webkit-unassigned] [Bug 52216] GeolocationController should call stopUpdating on destruction

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 12 08:45:10 PST 2011


https://bugs.webkit.org/show_bug.cgi?id=52216





--- Comment #5 from John Knottenbelt <jknotten at chromium.org>  2011-01-12 08:45:10 PST ---
(In reply to comment #4)
> Created an attachment (id=78695)
 --> (https://bugs.webkit.org/attachment.cgi?id=78695&action=review) [details]
> Patch

The patch adds a layout test to ensure that the corresponding assertion in the WebCore::GeolocationClientMock is executed. The test creates a geolocation watch in a separate window, and then closes that window and waits for the close to complete. If the watch is still updating at the time GeolocationClientMock::geolocationDestroyed() is invoked, the assertion will fail. 

The shutdown code in DumpRenderTree WebViewHost::~WebViewHost includes a navigation to "about:blank" before invoking WebWidget::close. This ensures that the watches are detached (and consequently the GeolocationClient will stop updating) before the GeolocationController is destroyed. 

Without the navigation to about:blank, the test crashes on the assertion, analogously to http://code.google.com/p/chromium/issues/detail?id=69069. 

The test is therefore adding coverage to ensure that future changes to WebKit maintains the assertion that the GeolocationClient is not updating when the GeolocationController is destroyed.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list