Bug 100992 - [BlackBerry] Add a prompt to enable global location services
Summary: [BlackBerry] Add a prompt to enable global location services
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-01 14:53 PDT by otcheung
Modified: 2012-11-01 18:17 PDT (History)
7 users (show)

See Also:


Attachments
Patch (7.38 KB, patch)
2012-11-01 14:59 PDT, otcheung
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description otcheung 2012-11-01 14:53:02 PDT
Currently, there is no check for the global location services setting. We need to add a prompt to ask the user to enable location services once per session if location services is turned off and the web page is requesting for it.
Comment 1 otcheung 2012-11-01 14:59:07 PDT
Created attachment 171939 [details]
Patch
Comment 2 Rob Buis 2012-11-01 15:07:21 PDT
Comment on attachment 171939 [details]
Patch

Looks good.
Comment 3 kov's GTK+ EWS bot 2012-11-01 15:27:35 PDT
Comment on attachment 171939 [details]
Patch

Attachment 171939 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/14665167
Comment 4 WebKit Review Bot 2012-11-01 15:45:55 PDT
Comment on attachment 171939 [details]
Patch

Rejecting attachment 171939 [details] from commit-queue.

otcheung@rim.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 5 WebKit Review Bot 2012-11-01 18:17:48 PDT
Comment on attachment 171939 [details]
Patch

Clearing flags on attachment: 171939

Committed r133247: <http://trac.webkit.org/changeset/133247>
Comment 6 WebKit Review Bot 2012-11-01 18:17:52 PDT
All reviewed patches have been landed.  Closing bug.