[Webkit-unassigned] [Bug 22022] [GTK] add a GeoLocation implementation

[bugzilla-daemon at webkit.org]

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


aurelianmaga at yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #26052|                            |review?
               Flag|                            |




------- Comment #20 from aurelianmaga at yahoo.com  2008-12-16 07:50 PDT -------
Created an attachment (id=26052)
 --> (https://bugs.webkit.org/attachment.cgi?id=26052&action=view)
updated patch

I’m sending this new patch which hopefully addresses Holger’s and Greg
comments.

>This is plain wrong. The enum is matching the PositionError of the w3c
>geolocation API. You may not just add values to it...

agreed

>wrong, for the same reason. If you think this should be there then please
>participate in the W3C geolocation discussion?!

agreed. Is there any reason to have PositionError::LOCATION_PROVIDER_ERROR and
PositionError::POSITION_NOT_FOUND_ERROR in the first place?

>hmm, could you explain that part? It feels odd to treat a non existing option
>like a request for a high accuracy...

agreed. I was trying to get around bug 22847

>oh well, could you point me to the gurantee that we will get a signal? I could
>not find anything in the GeoClue API doing that. And in general we do not
>comment out code...we remove it

in geoclue/example/master-pos-example.c position_changed_cb is triggered
periodically


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