Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
MIME-Version: 1.0
Sender: agrieve@google.com
In-Reply-To: 
 <CAMtcESYr8ibr6XbhaV0LjYsb__YtLq0m_LpgOuZr6RZ-5Pufyg@mail.gmail.com>
References: 
 <CAOBL_k6oJup5k8NmpvniRi1=rzqXAgVOig3dFewjs8s8WGm4QQ@mail.gmail.com>
 <CABiQX1UKKQ9s_8on811GBrHTc8NtesNEQ2eS9QUFph79BhXa7w@mail.gmail.com>
 <CAOBL_k4pogxq_GO6HvpeJx7S1D9W2qpSeDFgoy9o4vcnKR4M=g@mail.gmail.com>
 <CABiQX1VjzphKoomUG3D0iN=QQCz+NzAqR-mU8HFHqnMD=9STAQ@mail.gmail.com>
 <CAHYtTqmQHSp4qSFu+4YSnX7Uay8k_72n71kxPhzt9oOFBMn=AA@mail.gmail.com>
 <CAMtcESYr8ibr6XbhaV0LjYsb__YtLq0m_LpgOuZr6RZ-5Pufyg@mail.gmail.com>
From: Andrew Grieve <agrieve@chromium.org>
Date: Tue, 18 Aug 2015 21:06:07 -0400
Message-ID: 
 <CABiQX1UYExAkGXN2Od8BmfDmEasGSa_DQ5iD6XsNh5=DFak0OA@mail.gmail.com>
Subject: Re: [Android] Working with M - An update
To: dev <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a1134fa7cdc702b051d9fa432

--001a1134fa7cdc702b051d9fa432
Content-Type: text/plain; charset=UTF-8

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

--001a1134fa7cdc702b051d9fa432--