Yes! It's fixed :) Great work Joe on finding this! On Tue, Aug 18, 2015 at 10:51 AM, Karen Tran <ktop...@gmail.com> wrote:
> I just tried the manual geolocation tests on mobilespec with the 3rd > preview and I am able to get location data now. > Thanks for the help Andrew! > > On Mon, Aug 17, 2015 at 11:29 AM, Carlos Santana <csantan...@gmail.com> > wrote: > > > Hey Andrew any update on this, Do you have a link we can track progress? > Or > > do you know if latest preview dropped contains this fix? > > > > By the way when is Android M suppose to be General Available? > > > > > > On Fri, Jul 24, 2015 at 1:56 PM Andrew Grieve <agri...@chromium.org> > > wrote: > > > > > Was out last week, but did manage to escalate the geolocation bug. Will > > > hopefully be fixed for official M release :) > > > > > > On Thu, Jul 16, 2015 at 1:31 PM, Joe Bowser <bows...@gmail.com> wrote: > > > > > > > Sent you the test app off-list. > > > > > > > > On Wed, Jul 15, 2015 at 11:12 AM Andrew Grieve <agri...@chromium.org > > > > > > wrote: > > > > > > > > > Thanks for looking into this Joe! The runtime permissions is quite > a > > > big > > > > > change! > > > > > > > > > > M is still in preview, so if you find any webview bugs, please feel > > > free > > > > to > > > > > send me a repro app and I'll do my best to get it fixed. > > > > > > > > > > In terms of Cordova API changes, here's some thoughts on your > branch: > > > > > - Plugins may want to request different perms at different times, > so > > > I'd > > > > > remove new functions from CordovaPlugin except > > > onRequestPermissionResult > > > > > - Might be better to copy how CordovaInterfaceImpl does > > > > > startActivityForResult/onActivityResponse rather than have > > > PluginManager > > > > do > > > > > it. > > > > > > > > > > > > > > > On Tue, Jul 14, 2015 at 6:07 PM, Joe Bowser <bows...@gmail.com> > > wrote: > > > > > > > > > > > So, since Cordova-Android wasn't completely killed off by Google > at > > > the > > > > > > last Google IO like I thought it would be, we're going to have to > > > make > > > > > sure > > > > > > it's compatible with Android M (Marshmellow? Marzipan?). The > good > > > news > > > > > is > > > > > > that this only affects the following plugins: > > > > > > > > > > > > MediaRecorder > > > > > > Contacts > > > > > > File > > > > > > FileTransfer > > > > > > Geolocation > > > > > > > > > > > > Now, for the really bad news. We might have to write a > Geolocation > > > > > plugin > > > > > > for Android again, because Google's Android WebView doesn't play > > nice > > > > > with > > > > > > Android's new permission system, and even when you do grant the > > > > > application > > > > > > containing the process permission to use geolocation, you still > > get a > > > > > > location error. I still have to test this further, but it also > may > > > be > > > > > > possible that file URIs no longer have the ability to get the > > > > geolocation > > > > > > either, which would be weird, since this would be a Chrome thing > > and > > > > not > > > > > an > > > > > > Android thing. > > > > > > > > > > > > Given our poor track record of maintaining plugins in general, > and > > > the > > > > > > weird errors with Geolocation, I'm not really wanting to bring > back > > > the > > > > > > code, so I'm hoping that this gets resolved in M3 with the next > dev > > > > > version > > > > > > of M. > > > > > > > > > > > > But so far, I have the changes that I made to Cordova on a > private > > > > branch > > > > > > on Github that people can see here: > > > > > > > > > > > > https://github.com/infil00p/cordova-android/tree/m-compat > > > > > > > > > > > > Let me know if you have any questions. I'm not sure whether this > > > makes > > > > > > perfect sense yet, but I think these API changes indicate that we > > > > should > > > > > > probably bump the version to 5.0 once M comes out. > > > > > > > > > > > > Thoughts? Questions? > > > > > > > > > > > > > > > > > > > > >