Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of fil@adobe.com designates
 64.18.1.208 as permitted sender)
From: Filip Maj <fil@adobe.com>
To: "callback-dev@incubator.apache.org" <callback-dev@incubator.apache.org>
Date: Mon, 16 Apr 2012 16:46:26 -0700
Subject: Re: JS: tagged 1.6.1
Thread-Topic: JS: tagged 1.6.1
Thread-Index: Ac0cKtnRCxZHzde5R3yXSIMkJikVEw==
Message-ID: <CBB1FCE3.4738%fil@adobe.com>
In-Reply-To: <CBB1FAEE.471B%fil@adobe.com>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.14.0.111121
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

With respect to 1.6.1, I would change the native side on iOS, Shaz, to
call:

cordova.require('cordova/channel').onResume.fire(); (or onPause)

Depending on what approach everyone thinks we should use (the first or
second of my suggestions below, or something else entirely) we can slate
that change for all platforms for 1.7.

On 4/16/12 4:42 PM, "Filip Maj" <fil@adobe.com> wrote:

>I see the issue.
>
>fireDocumentEvent looks at the documentEventHandlers and checks if
>something exists in that object with the name of the event. We never
>actually register resume or pause channels with the
>"documentEventHandlers" object, so calling firedocumentevent will never
>work.
>
>Android just requires the channel module directly and fires the proper
>onResume / onPause channels [1].
>
>This is implemented inconsistently. We should do something one way and
>stick to it. Either have:
>
>- cordova.js create the resume + pause channels and register them using
>the addDocumentEventHandler [2] method, and have native side call
>cordova.fireDocumentEvent, OR
>- keep the manual creation of the pause/resume channels [3], and have
>native call cordova.require('corodva/channel').(onPause ||
>onResume).fire()
>
>[1]=20
>https://github.com/apache/incubator-cordova-android/blob/master/framework/
>s
>rc/org/apache/cordova/DroidGap.java#L889
>[2]=20
>https://github.com/apache/incubator-cordova-js/blob/master/lib/cordova.js#
>L
>94
>[3]=20
>https://github.com/apache/incubator-cordova-js/blob/master/lib/common/chan
>n
>el.js#L225-L229
>
>On 4/16/12 4:30 PM, "Shazron" <shazron@gmail.com> wrote:
>
>>I'm failing two manual tests. Step 1 of
>>http://wiki.apache.org/cordova/iOSManualTests - I'm not getting pause
>>and resume events for some reason, but all other events are fine.
>>
>>I've verified I'm getting the lifecycle events in native, but when
>>calling to javascript "cordova.fireDocumentEvent('pause') " and
>>"cordova.fireDocumentEvent('resume') " - none of the my callbacks are
>>receiving the event.
>>
>>Any clue? I see there are overrides for document.addEventListener that
>>special-case these two events though, not sure what's going on in
>>there yet. Debugging.
>>
>>On Mon, Apr 16, 2012 at 3:20 PM, Joe Bowser <bowserj@gmail.com> wrote:
>>> Re-tagged Android at 1.6.1, somehow the JS was already changed.
>>>Creepy!
>>>
>>> On Mon, Apr 16, 2012 at 3:09 PM, Filip Maj <fil@adobe.com> wrote:
>>>
>>>> Do what you need to do, team.
>>>>
>>>>
>