[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