Bug 28619

Summary: [GTK] Geolocation needs permission API before being enabled by default
Product: WebKit Reporter: Gustavo Noronha (kov) <gustavo>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: jmalonzo
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
enable geolocation on the buildbot build
none
Patch v1 none

Gustavo Noronha (kov)
Reported 2009-08-21 12:23:44 PDT
We need to have an API which allows the browser to decide what to expose to the world before we enable geoclue by default.
Attachments
enable geolocation on the buildbot build (128.81 KB, patch)
2009-08-21 17:13 PDT, Jan Alonzo 		no flags
DetailsFormatted DiffDiff
Patch v1 (128.71 KB, patch)
2009-08-28 06:57 PDT, Jan Alonzo 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Gustavo Noronha (kov)
Comment 1 2009-08-21 12:26:57 PDT
Test failing because of this (reenable when we turn it on): fast/dom/Geolocation/argument-types.html
Jan Alonzo
Comment 2 2009-08-21 17:13:16 PDT
Created attachment 38417 [details] enable geolocation on the buildbot build
Gustavo Noronha (kov)
Comment 3 2009-08-23 15:13:40 PDT
Comment on attachment 38417 [details] enable geolocation on the buildbot build r=me Can you check that the bot has all the geoclue* packages that you have installed, when landing? =)
Jan Alonzo
Comment 4 2009-08-28 06:57:37 PDT
Created attachment 38732 [details] Patch v1
Jan Alonzo
Comment 5 2009-08-28 06:59:02 PDT
(In reply to comment #4) > Created an attachment (id=38732) [details] > Patch v1 This is a simplified approach than the previous one.
Jan Alonzo
Comment 6 2009-08-28 07:29:14 PDT
Comment on attachment 38732 [details] Patch v1 Landed as http://trac.webkit.org/changeset/47864. Clearing flag as the API still needs to be written.
Gustavo Noronha (kov)
Comment 7 2010-08-19 08:24:21 PDT
This API already exists now.
Note You need to log in before you can comment on or make changes to this bug.