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? > > > > > > > > > > > > > > >
