Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 2DC4E112E8 for ; Wed, 10 Sep 2014 20:57:15 +0000 (UTC) Received: (qmail 9333 invoked by uid 500); 10 Sep 2014 20:57:14 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 9292 invoked by uid 500); 10 Sep 2014 20:57:14 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 9280 invoked by uid 99); 10 Sep 2014 20:57:14 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Sep 2014 20:57:14 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of brian.leroux@gmail.com designates 209.85.223.177 as permitted sender) Received: from [209.85.223.177] (HELO mail-ie0-f177.google.com) (209.85.223.177) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Sep 2014 20:56:48 +0000 Received: by mail-ie0-f177.google.com with SMTP id rd18so3653962iec.8 for ; Wed, 10 Sep 2014 13:56:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:content-type; bh=QiH+izoYJjU3ryuGAZymrDapKxLdaGenNCnkGfmHIn8=; b=AYwWY6ucJ9744rLjKpRCCVXFFjHyeeNwM+2351r7d8Zfa1RVIq/hYVz1JFQASW5eVT VtsECIvk4dc3M7MZaFwPBfp7vDOBo2ldCk3KcMfQVYKAiCNMkdD5f79hJV8ldoP1MgN5 Q/3b4uGXEMoKSXeuZf35Y2+9Tj9/H+hmp9stv4I01hb2S+eeGOgKPwptLGnAHa0cPOdt EW4fYj7OX56SKC+OBpzZDd3dhBwlJziaSY8r3Z3tK/xHF2nkHRZECD5cuelQgGLQUnGg sfxHefqZOSGpcTLMMPMMQHIDKkIGiGof3rver+Q5kMjhGZC7fBXvmzVUow/ouCCzv7Bv tBvg== MIME-Version: 1.0 X-Received: by 10.42.161.198 with SMTP id u6mr48754144icx.31.1410382607410; Wed, 10 Sep 2014 13:56:47 -0700 (PDT) Sender: brian.leroux@gmail.com Received: by 10.50.221.140 with HTTP; Wed, 10 Sep 2014 13:56:47 -0700 (PDT) Received: by 10.50.221.140 with HTTP; Wed, 10 Sep 2014 13:56:47 -0700 (PDT) In-Reply-To: References: Date: Wed, 10 Sep 2014 13:56:47 -0700 X-Google-Sender-Auth: CqaJGEHOpdL2fy3ZjmWTLvmOX3I Message-ID: Subject: Re: Android: activityResultKeepRunning From: Brian LeRoux To: dev@cordova.apache.org Content-Type: multipart/alternative; boundary=90e6ba6e8fb84feee60502bc4a4c X-Virus-Checked: Checked by ClamAV on apache.org --90e6ba6e8fb84feee60502bc4a4c Content-Type: text/plain; charset=UTF-8 I consider 4 a release branch. Merge in tested green lit code to your hearts desire but 4.0 is definitely not a feature. It should be always in a releasable state. On Sep 10, 2014 1:53 PM, "Michal Mocny" wrote: > Question is, do you consider the fact that bugs are introduced & discovered > (possibly with pain) a sign that the system is broken, or a sign that the > system is working? > > I sense that Andrew worries that if work has to land on a feature branch of > this feature branch, it won't get eyeballs. > > I sense that Joe worries that if we land everything/anything in Android-4.0 > it will become unstable, as mistakes are prone to happen (see i.e. recent > issue with black background). > > Personally, I prefer eyeballs and instability to delayed discovery and a > sense of stability, especially for a feature branch like Android-4.0. > There are workarounds for demos (i.e. create your own branch off of a > known working version), but its not as easy to solve the eyeball problem. > > -Michal > > On Wed, Sep 10, 2014 at 3:50 PM, Joe Bowser wrote: > > > I think this needs to be thought through more, and I'm extremely wary > when > > you say this is a single commit, especially based on the last couple of > > months and how long it took 3.6 to go through. Given that we have people > > travelling halfway across the planet who intend to show people their work > > in less than two weeks, I would definitely like it if you were to put > this > > in your own branch for testing. > > > > On Wed, Sep 10, 2014 at 12:41 PM, Andrew Grieve > > wrote: > > > > > I don't think there'd be much value in that. It'll be a single commit > > > that almost entirely just deletes lines. > > > > > > What do you think about the never auto-pausing on backgrounding? or > > > about auto-pausing when intent sending? > > > > > > On Wed, Sep 10, 2014 at 12:32 PM, Joe Bowser > wrote: > > > > Can you put this on its own branch before it lands in 4.0.x? That'd > be > > > > awesome! > > > > > > > > On Tue, Sep 9, 2014 at 5:32 PM, Andrew Grieve > > > wrote: > > > >> > > > >> For cordova-android 4.0, I'd like to go as far as just deleting the > > > >> "KeepRunning" . > > > >> > > > >> Apps get a "pause" event when they are backgrounded, and they can do > > > >> any pause-type logic there (e.g. unlisten to accelerometer events or > > > >> pausing audio). > > > >> > > > >> Any strong objections? > > > >> > > > >> On Tue, Sep 9, 2014 at 4:27 PM, Andrew Grieve > > > > >> wrote: > > > >> > Commit description: If multitasking is turned on > (keepRunning=true), > > > >> > then temporarily disable it when starting a new activity that > > returns > > > >> > a result - such as camera. > > > >> > > > > >> > > > > >> > > > > > > > https://github.com/apache/cordova-android/commit/26adfb634651196106fb5b66f15eecb535a06d82 > > > >> > > > > >> > Bryce / anyone - clues as to *why* we'd want to disable JS timers > > when > > > >> > firing off an intent? > > > > > > > > > > > > > > --90e6ba6e8fb84feee60502bc4a4c--