cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Grieve <agri...@chromium.org>
Subject Re: [DISCUSS] Cordova-Android 4.0.0 Release
Date Fri, 30 Jan 2015 02:10:04 GMT
On Wed, Jan 28, 2015 at 3:32 PM, Andrew Grieve <agrieve@chromium.org> wrote:

>
>
> On Wed, Jan 28, 2015 at 1:44 PM, Joe Bowser <bowserj@gmail.com> wrote:
>
>> On Wed Jan 28 2015 at 10:38:07 AM Andrew Grieve <agrieve@chromium.org>
>> wrote:
>>
>> >
>> > - Make CordovaActivity not implement CordovaInterface, but instead
>> provide
>> > CordovaInterface via an inner class (to solidify that you can't cast the
>> > activity to CordovaInterface and expect that to work - some used to do
>> this
>> > but I think we've cleaned it all up now)
>> >
>> > This literally came out of nowhere.  Why are you trying so hard to
>> remove
>> the embedded view use case? What if someone is implementing an activity
>> that inherits from another activity like MapActivity?  This API change
>> came
>> without any discussion.
>>
> I meant for this to be discussion. Certainly this is non-critical, but I
> think it makes the embedded use-case easier not harder. Will do it in a PR
> for review.
>

Here's the issue: https://issues.apache.org/jira/browse/CB-8382
and PR: https://github.com/apache/cordova-android/pull/152


>
>
>>
>> All of this can be done in a few days, but I'd also like to see the dust
>> > settle a bit before going forward with 4.0.0 release. E.g. At least wait
>> > until we do a blog post for 3.7.0 (are you doing this?), and have done a
>> > tools release that updates the pinned version to 3.7.0
>> >
>> >
>> If someone else wants to do the blog post on that, that's fine.  And I
>> agree that there should be a tools release with 3.7.0 pinned, even though
>> 3.7.0 is really just a technicality so we can get 4.0.0 out IMO.
>>
>
> 3.7.0 adds Lollipop support. That's pretty big! I won't have time to get
> to it this week if there are any other takers?
>
>
>>
>>
>> >
>> > On Wed, Jan 28, 2015 at 12:52 PM, Joe Bowser <bowserj@gmail.com> wrote:
>> >
>> > > Reminder: failures with plugins are not blockers.  I've run into that
>> > > contact issue numerous times when testing with my personal device.  I
>> > > recommend making sure that your contacts are completely clean so that
>> you
>> > > don't get these weird results.
>> > >
>> > > The file failures have been happening for quite a while, and those are
>> > not
>> > > blockers for the platform release either.  Do these failures happen
>> on a
>> > > platform other than ICS?
>> > >
>> > > On Wed, Jan 28, 2015, 9:06 AM Murat Sutunc <muratsu@microsoft.com>
>> > wrote:
>> > >
>> > > > I’ve ran the mobile-spec tests on android 4.0.3 with 4.0.x and there
>> > are
>> > > > some failures. I’ve searched the jira for issues but wasn’t able
to
>> > find
>> > > > any. Has anyone else ran into these issues before?
>> > > >
>> > > > org.apache.cordova.contacts.tests.tests >> Contacts
>> > (navigator.contacts)
>> > > > Round trip Contact tests (creating + save + delete + find).
>> > > > Contacts.spec.24 Creating, saving, finding a contact should work,
>> after
>> > > > which we should not be able to find it, and we should not be able
to
>> > > delete
>> > > > it again.
>> > > > •       Expected 2 to be 1
>> > > > •       Expected 1 to be 0
>> > > >          it("contacts.spec.24 Creating, saving, finding a contact
>> > should
>> > > > work, removing it should work, after which we should not be able to
>> > find
>> > > > it, and we should not be able to delete it again.", function (done)
>> {
>> > > >               // Save method is not supported on Windows platform
>> > > >               if (isWindows) {
>> > > >                   pending();
>> > > >                   return;
>> > > >               }
>> > > >               if (isWindowsPhone8) {
>> > > >                   done();
>> > > >                   return;
>> > > >               }
>> > > >               gContactObj = new Contact();
>> > > >               gContactObj.name = new ContactName();
>> > > >               gContactObj.name.familyName = "DeleteMe";
>> > > >               gContactObj.save(function(c_obj) {
>> > > >                   var findWin = function(cs) {
>> > > >                       expect(cs.length).toBe(1);
>> > > >                       // update to have proper saved id
>> > > >                       gContactObj = cs[0];
>> > > >                       gContactObj.remove(function() {
>> > > >                           var findWinAgain = function(seas) {
>> > > >                               expect(seas.length).toBe(0);
>> > > >                               gContactObj.remove(function() {
>> > > >                                   throw("success callback called
>> after
>> > > > non-existent Contact object called remove(). Test failed.");
>> > > >                               }, function(e) {
>> > > >                                   expect(e.code).toBe(ContactErr
>> > > > or.UNKNOWN_ERROR);
>> > > >                                   done();
>> > > >                               });
>> > > >                           };
>> > > >                           var findFailAgain = function(e) {
>> > > >                               throw("find error callback invoked
>> after
>> > > > delete, test failed.");
>> > > >                           };
>> > > >                           var obj = new ContactFindOptions();
>> > > >                           obj.filter="DeleteMe";
>> > > >                           obj.multiple=true;
>> > > >                           navigator.contacts.find(["displayName",
>> > "name",
>> > > > "phoneNumbers", "emails"], findWinAgain, findFailAgain, obj);
>> > > >                       }, function(e) {
>> > > >                           throw("Newly created contact's remove
>> > function
>> > > > invoked error callback. Test failed.");
>> > > >                       });
>> > > >                   };
>> > > >                   var findFail = fail;
>> > > >                   var obj = new ContactFindOptions();
>> > > >                   obj.filter="DeleteMe";
>> > > >                   obj.multiple=true;
>> > > >                   navigator.contacts.find(["displayName", "name",
>> > > > "phoneNumbers", "emails"], findWin, findFail, obj);
>> > > >               }, fail);
>> > > >           });
>> > > >
>> > > > org.apache.cordova.file.tests.test >> file api filereader
>> file.spec.81
>> > > > (couldn’t find a JIRA issue)
>> > > > •       Expected `` to be null
>> > > >         describe('FileReader', function () {
>> > > >             it("file.spec.81 should have correct methods", function
>> ()
>> > {
>> > > >                 var reader = new FileReader();
>> > > >                 expect(reader).toBeDefined();
>> > > >                 expect(typeof
>> > > reader.readAsBinaryString).toBe('function');
>> > > >                 expect(typeof
>> reader.readAsDataURL).toBe('function');
>> > > >                 expect(typeof reader.readAsText).toBe('function');
>> > > >                 expect(typeof reader.readAsArrayBuffer).
>> > toBe('function');
>> > > >                 expect(typeof reader.abort).toBe('function');
>> > > >                 //////////// test below fails ////////////
>> > > >                 ////////////   '' !== null    ////////////
>> > > >                 expect(reader.result).toBe(null);
>> > > >             });
>> > > >         });
>> > > >
>> > > > org.apache.cordova.file.tests.tests >> file api parent references
>> > > > file.spec.111 (couldn’t find a fire issue):
>> > > > •       root.getFile succeeds, it is expected to fail.
>> > > > var fileName = "traverse.file.uri";
>> > > >                 // create a new file entry
>> > > >                 createFile(fileName, function (entry) {
>> > > >                     // lookup file system entry
>> > > >                     root.getFile('../' + fileName, {
>> > > >                         create : false
>> > > >                     }, succeed.bind(null, done,
>> > > > "root.getFile('../"+fileName+ "')- Unexpected success callback, it
>> > should
>> > > > not traverse abvoe the root directory"),
>> > > >                     function (error) { //.....
>> > > >
>> > > > org.apache.cordova.file-transfer.tests.tests >> FileTransfer
methods
>> > > > download filetransfer.spec.6 should get 401 status on http basic
>> auth
>> > > > failure
>> > > > •       Expected null to be 401
>> > > >                 it('filetransfer.spec.6 should get 401 status on
>> http
>> > > > basic auth failure', function (done) {
>> > > >
>> > > >                     // NOTE:
>> > > >                     //      using server without credentials
>> > > >                     var fileURL = SERVER + '/download_basic_auth';
>> > > >
>> > > >                     var downloadFail = function (error) {
>> > > >                         expect(error.http_status).toBe(401);
>> > > >                         expect(error.http_status).not.toBe(404,
>> > "Ensure "
>> > > > + fileURL + " is in the white list");
>> > > >                         done();
>> > > >                     };
>> > > >
>> > > >                     transfer.download(fileURL, localFilePath,
>> > > > unexpectedCallbacks.httpWin, downloadFail);
>> > > >                 });
>> > > >
>> > > > org.apache.cordova.geolocation.tests.tests >> getCurrentPosition
>> > method
>> > > > success callback geolocation.spec.6 should be called with a position
>> > > object
>> > > > •       Expected true to be false
>> > > >             it("geolocation.spec.6 should be called with a Position
>> > > > object", function (done) {
>> > > >                 // this test asks for using geolocation and
>> interrupts
>> > > > autotests running.
>> > > >                 // That's why we have to pending that for Windows
>> Store
>> > > > 8.0/8.1 apps
>> > > >                 if (isWindowsStore) {
>> > > >                     pending();
>> > > >                 }
>> > > >                 navigator.geolocation.getCurrentPosition(function
>> (p)
>> > {
>> > > >                     expect(p.coords).toBeDefined();
>> > > >                     expect(p.timestamp).toBeDefined();
>> > > >                     done();
>> > > >                 },
>> > > >                 fail.bind(null, done),
>> > > >                 {
>> > > >                     maximumAge: 300000 // 5 minutes maximum age of
>> > cached
>> > > > position
>> > > >                 });
>> > > >             });
>> > > >
>> > > > org.apache.cordova.geolocation.tests.tests >> watchPosition
method
>> > > > success callback geolocation.spec.8 should be called with a
>> position
>> > > object
>> > > > •       Expected true to be false
>> > > >             it("geolocation.spec.8 should be called with a Position
>> > > > object", function (done) {
>> > > >                 // this test asks for using geolocation and
>> interrupts
>> > > > autotests running.
>> > > >                 // That's why we have to pending that for Windows
>> Store
>> > > > 8.0/8.1 apps
>> > > >                 if (isWindowsStore) {
>> > > >                     pending();
>> > > >                 }
>> > > >                 successWatch = navigator.geolocation.watchPosition(
>> > > >                     function (p) {
>> > > >                         expect(p.coords).toBeDefined();
>> > > >                         expect(p.timestamp).toBeDefined();
>> > > >                         done();
>> > > >                     },
>> > > >                     fail.bind(null, done),
>> > > >                     {
>> > > >                         maximumAge: (5 * 60 * 1000) // 5 minutes
>> > maximum
>> > > > age of cached position
>> > > >                     });
>> > > >             });
>> > > >
>> > > >
>> > > > -----Original Message-----
>> > > > From: Josh Bavari [mailto:jbavari@gmail.com]
>> > > > Sent: Wednesday, January 28, 2015 8:30 AM
>> > > > To: dev@cordova.apache.org
>> > > > Subject: Re: [DISCUSS] Cordova-Android 4.0.0 Release
>> > > >
>> > > > Joe and team,
>> > > >
>> > > > I work for Ionic and I've had some involvement with the Cordova
>> project
>> > > > since last year. At Ionic, we've released a Crosswalk build using
>> > Cordova
>> > > > Android 4.0 so we can use the cordova crosswalk engine for the ionic
>> > > > platform.
>> > > >
>> > > > I've been working with Ian and Andrew on this to gather more
>> > > understanding
>> > > > and to get some help along the way. I must say, excellent work,
>> > everyone.
>> > > >
>> > > > As such, we've accumulated quite a bit of users who are actively
>> using
>> > > > Cordova Android 4.0. Currently, we've had over 10k test trials with
>> it,
>> > > and
>> > > > I'm happy to say, mostly it's been smooth.
>> > > >
>> > > > What I've done is made a fork to adjust a few small things, but for
>> the
>> > > > most part, we're using 4.0.
>> > > >
>> > > > I'd love to provide any more feedback that you'd wish.
>> > > >
>> > > > Thanks again for the awesome work.
>> > > >
>> > > > On Wed, Jan 28, 2015 at 9:21 AM, Joe Bowser <bowserj@gmail.com>
>> wrote:
>> > > >
>> > > > > Hey
>> > > > >
>> > > > > So, it's finally here.  I want to see us work more on Pluggable
>> > > > > Webviews, and adding the API, but I think it's time that we
>> released
>> > > > > what we've been working on for almost a year to our users.  I
know
>> > > > > that the API isn't exactly the most awesome we can make it, but
it
>> > > > > works, and I'd rather have it out at 80% than it sitting for
a few
>> > more
>> > > > months in limbo.
>> > > > >
>> > > > > Are there any major blocking tasks that would prevent a vote
>> thread
>> > > > > that anyone knows about, or should we start firing up a release?
>> I
>> > > > > don't think we're going to make our January date, but the first
>> week
>> > > > > of February isn't that terrible.
>> > > > >
>> > > > > Thoughts?
>> > > > >
>> > > > > Joe
>> > > > >
>> > > >
>> > > >
>> > > >
>> > > > --
>> > > > "Clear thoughts produce clear results."
>> > > > Josh Bavari
>> > > > Application Developer
>> > > > Phone: 405-509-9448
>> > > > Cell: 405-812-0496
>> > > > Email: jbavari@gmail.com
>> > > >
>> > > >
>> ---------------------------------------------------------------------
>> > > > To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
>> > > > For additional commands, e-mail: dev-help@cordova.apache.org
>> > > >
>> > >
>> >
>>
>
>

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