cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Grieve <agri...@chromium.org>
Subject Re: [Android] Working with M - An update
Date Wed, 19 Aug 2015 01:06:07 GMT
Yes! It's fixed :) Great work Joe on finding this!

On Tue, Aug 18, 2015 at 10:51 AM, Karen Tran <ktop500@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 <csantana23@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 <agrieve@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 <bowserj@gmail.com> wrote:
> > >
> > > > Sent you the test app off-list.
> > > >
> > > > On Wed, Jul 15, 2015 at 11:12 AM Andrew Grieve <agrieve@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 <bowserj@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?
> > > > > >
> > > > >
> > > >
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message