Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (athena.apache.org: domain of v-segreb@microsoft.com
 designates 207.46.163.242 as permitted sender)
From: "Sergey Grebnov (Akvelon)" <v-segreb@microsoft.com>
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Subject: RE: Windows Phone binary bridge
Thread-Topic: Windows Phone binary bridge
Thread-Index: 
 AQHO8pkl45YznI7k+0+t2CeIVc1g0JpHhngQgAAFC4CAACPpgIAABhQAgARGBDCAAAcKAIAAEwkA
Date: Mon, 9 Dec 2013 16:44:38 +0000
Message-ID: 
 <30d772ddd85f426ab5082ad67f749246@BL2PR03MB417.namprd03.prod.outlook.com>
References: 
 <CAK_TSXJejh6jC=KiSrYaJd8DtOn3frg3Ah2kYYCt5yDhqEyhbQ@mail.gmail.com>
 <9bc28c5131fb4eb7a8eb4595902e8a06@BL2PR03MB338.namprd03.prod.outlook.com>
 <CAK_TSXJ5QiAhLmsNo07dzzYt=1M1DbLf2NdPW8iVhQtjouMg1A@mail.gmail.com>
 <CAKP6k2ujhMpRvULTPh+0qjNYU=V92Q4TMo9Mu3VypQda3mX0YA@mail.gmail.com>
 <311af38e5ccb4e4b9e29237622d531f9@DM2PR03MB366.namprd03.prod.outlook.com>
 <5a192c7c5bcb4355a713f07bb2706211@BL2PR03MB417.namprd03.prod.outlook.com>
 <CAEeF2Tdze+qOOc5rbzimQRnBAje4E3EMYObp=4xoS-xPMWvV-Q@mail.gmail.com>
In-Reply-To: 
 <CAEeF2Tdze+qOOc5rbzimQRnBAje4E3EMYObp=4xoS-xPMWvV-Q@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

This was temporary added for File api (WP8 only)
https://github.com/sgrebnov/cordova-plugin-file/commit/3b39ec350bf5987dcf46=
76b3e0597d87bbac199e

Michal, right now I see there is base64 string used .

Thx!
Sergey
-----Original Message-----
From: mmocny@google.com [mailto:mmocny@google.com] On Behalf Of Michal Mocn=
y
Sent: Monday, December 9, 2013 7:33 PM
To: dev
Subject: Re: Windows Phone binary bridge

Where do you see an implementation using Array.apply(null, new
Uint8Array(...)) ?

We used to have this bug in ios/android binary bridge but that was patched =
many releases ago (unless there is a stray one somewhere that still needs f=
ixing).


On Mon, Dec 9, 2013 at 10:15 AM, Sergey Grebnov (Akvelon) < v-segreb@micros=
oft.com> wrote:

> Working on this...I just found out that current implementation via=20
> Array.apply(null, new Uint8Array(...)) doesn't work for big buffers=20
> (bigger than 150Kb) and could raise Maximum call stack size exceeded=20
> error. So I'm looking on better/right way to do typed arrays=20
> conversion... will keep you updated.
>
> Thx!
> Sergey
> -----Original Message-----
> From: Parashuram Narasimhan (MS OPEN TECH)=20
> [mailto:panarasi@microsoft.com]
> Sent: Saturday, December 7, 2013 1:52 AM
> To: dev@cordova.apache.org
> Subject: RE: Windows Phone binary bridge
>
> I think Sergey should be able to squeeze this into his schedule. We=20
> were looking at fixing plugin support and this could be a good way to sta=
rt it.
>
> -----Original Message-----
> From: Jesse [mailto:purplecabbage@gmail.com]
> Sent: Friday, December 6, 2013 1:30 PM
> To: dev@cordova.apache.org
> Subject: Re: Windows Phone binary bridge
>
> FYI, you can buy a no contract Nokia 520 WP8 device for $120, and=20
> developing in the emulator is more than sufficient for most tasks.
>
> Sergey, do you got this?
>
>
> @purplecabbage
> risingj.com
>
>
> On Fri, Dec 6, 2013 at 11:21 AM, Ian Clelland <iclelland@chromium.org
> >wrote:
>
> > On Fri, Dec 6, 2013 at 2:15 PM, Sergey Grebnov (Akvelon) <=20
> > v-segreb@microsoft.com> wrote:
> >
> > > Agree and  like this idea. I don't think there are many other=20
> > > components (if any) which use ArrayBuffer but must be tested well=20
> > > anyway since this changes core. Ian do you want to complete this=20
> > > yourself or you are ok if
> > I
> > > handle this?
> > >
> >
> > I'm totally okay with you doing it -- I don't have a real windows=20
> > phone device to test it on; I'd be committing with my eyes closed=20
> > and hoping for the best :)
> >
> > I think you're right that there aren't any other core plugins using=20
> > the binary bridge, but that could change, and there are probably=20
> > already third-party plugins using it.
> >
> > If we do this right, then other developers won't have to roll their=20
> > own solution in each case, and it will be easier to port the=20
> > third-party plugins to WP.
> >
> > Let me know if you need any support; I'll help if I can.
> >
> > Ian
> >
> >
> > >
> > > -Sergey
> > > -----Original Message-----
> > > From: iclelland@google.com [mailto:iclelland@google.com] On Behalf=20
> > > Of
> > Ian
> > > Clelland
> > > Sent: Friday, December 6, 2013 7:37 PM
> > > To: dev@cordova.apache.org
> > > Subject: Windows Phone binary bridge
> > >
> > > As far as I can tell from reading its exec.js, WP8 does not have a=20
> > > binary bridge of any kind. Non-string data get JSON-serialized for
> transfer.
> > >
> > > The recent patch for CB-5532 broke the File plugin by explicitly=20
> > > casting ArrayBuffer data to Array type. Sergey's fixed it now, but=20
> > > the FileWriter JS code still has special cases for Windows and WP8
> platforms.
> > >
> > > Would it make sense to have this code moved into the WP8 exec bridge?
> > Then
> > > it could apply to any plugins which transfer binary data, and any
> > improved
> > > transport methods would automatically be used by all plugins,=20
> > > rather than being a special case in each plugin.
> > >
> > > I don't know much about the Windows phone platform, so I could be=20
> > > completely off-base about this, but if it works, I think it'll=20
> > > make the implementation cleaner, and generally make Cordova better=20
> > > on the
> > platform.
> > >
> > > Ian
> > >
> >
>