Closed Bug 568249 Opened 10 years ago Closed 10 years ago

Support geolocation on Android

Categories

(Core :: Widget: Android, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: mwu, Assigned: mwu)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Attachment #447561 - Flags: review?(dougt)
Comment on attachment 447561 [details] [diff] [review]
Add Geolocation provider for Android

This is great!

>+#include "AndroidLocationProvider.h"
>+
>+using namespace mozilla;
Not sure why we need this.  If we do, fine.  If not, lets remove it


>diff --git a/widget/src/android/nsAppShell.cpp b/widget/src/android/nsAppShell.cpp

Is there a way to get notifications outside of the appshell?  This is the first example of having to use the app shell to get geo notifications.  If not, thats fine.
Attachment #447561 - Flags: review?(dougt) → review+
(In reply to comment #2)
> >+#include "AndroidLocationProvider.h"
> >+
> >+using namespace mozilla;
> Not sure why we need this.  If we do, fine.  If not, lets remove it
> 
It's used so we don't have to do mozilla:: when using AndroidBridge.

> >diff --git a/widget/src/android/nsAppShell.cpp b/widget/src/android/nsAppShell.cpp
> 
> Is there a way to get notifications outside of the appshell?  This is the first
> example of having to use the app shell to get geo notifications.  If not, thats
> fine.
I think we could come up with other ways but this is the simplest way right now AFAICT.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.