[webkit-reviews] review requested: [Bug 21475] Provide support for the Geolocation API : [Attachment 24463] updated
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Oct 17 14:43:26 PDT 2008
Greg Bolsinga <bolsinga at apple.com> has asked for review:
Bug 21475: Provide support for the Geolocation API
https://bugs.webkit.org/show_bug.cgi?id=21475
Attachment 24463: updated
https://bugs.webkit.org/attachment.cgi?id=24463&action=edit
------- Additional Comments from Greg Bolsinga <bolsinga at apple.com>
This addresses the items in Sam's review. Objects go through ::create() for
RefPtrs. Header order cleaned up. Doesn't add page to the back/forward cache if
it is currently using Geolocation (from IRC). stopsUpdating when
Geolocation::disconnectFrame is called. Copies data structures before iterating
(nice catch).
The GeolocationServiceFake (and its tests) were removed from the patch. Having
geolocation be a property on Navigator only when in DRT has proven to be
complex. The test to verify that geolocation hasn't leaked into navigator is
some fashion is retained and more robust than the last patch.
I will file a followup bug for the GeolocationServiceFake implementation and
test.
More information about the webkit-reviews
mailing list