[webkit-dev] Blob will support ArrayBufferView instead of ArrayBuffer for Constructor Parameters

[Yin, Li]

Agree.
I will remove it.

-----Original Message-----
From: Kenneth Russell [mailto:kbr at google.com] 
Sent: Thursday, March 21, 2013 7:37 AM
To: Takashi Toyoshima
Cc: Kinuko Yasuda; Yin, Li; webkit-dev at lists.webkit.org
Subject: Re: [webkit-dev] Blob will support ArrayBufferView instead of ArrayBuffer for Constructor Parameters

Agree; remove the warning.

-Ken


On Wed, Mar 20, 2013 at 10:33 AM, Takashi Toyoshima <toyoshim at chromium.org> wrote:
> +1 on removing the warning.
>
>
> On Wed, Mar 20, 2013 at 10:24 AM, Kinuko Yasuda <kinuko at chromium.org> wrote:
>>
>> Hi,
>>
>> I haven't been closely following the thread on public-webapps, but 
>> looks like ArrayBuffer is back again:
>> http://dev.w3.org/2006/webapi/FileAPI/#dfn-Blob
>>
>> It was certainly not there in spec 20120712:
>> http://www.w3.org/TR/2012/WD-FileAPI-20120712/#dfn-Blob
>>
>> But added back when it's updated to take sequence<> 
>> http://www.w3.org/TR/2012/WD-FileAPI-20121025/#dfn-Blob
>>
>> Currently it still works on WebKit but we show a deprecation warning 
>> on console. Should we remove the warning?
>>
>>
>>
>> On Tue, Jun 5, 2012 at 9:17 PM, Yin, Li <li.yin at intel.com> wrote:
>>>
>>> Okay, I updated the patch, currently, it supports both of 
>>> ArrayBuffer and ArrayBufferView.
>>>
>>> And I have filed a new bug about removing support of ArrayBuffer.
>>> https://bugs.webkit.org/show_bug.cgi?id=88389
>>>
>>> Thanks for your good comments.
>>>
>>> ----------
>>> Li Yin
>>>
>>> -----Original Message-----
>>> From: Kenneth Russell [mailto:kbr at google.com]
>>> Sent: Wednesday, June 06, 2012 5:21 AM
>>> To: Takashi Toyoshima
>>> Cc: Yin, Li; webkit-dev at lists.webkit.org
>>> Subject: Re: [webkit-dev] Blob will support ArrayBufferView instead 
>>> of ArrayBuffer for Constructor Parameters
>>>
>>> I think that WebKit should continue to support the old constructor 
>>> for a brief period of time, but warn about its use. I've commented 
>>> on the patch indicating this.
>>>
>>> Li Yin, if you implement this, please file a follow-on bug about 
>>> removing support for the old constructor. The new bug should depend 
>>> on the old, and it would be nice if you would own the follow-on bug.
>>>
>>> -Ken
>>>
>>>
>>> On Tue, Jun 5, 2012 at 7:33 AM, Takashi Toyoshima 
>>> <toyoshim at google.com>
>>> wrote:
>>> > Hi,
>>> >
>>> > The same change will happen in XHR and WebSocket.
>>> >
>>> > I suggest to keep on holding ArrayBuffer interface in the 
>>> > WebSocket API spec at the thread
>>> > https://www.w3.org/Bugs/Public/show_bug.cgi?id=17263
>>> >
>>> > From the discussion, IE seems to continue supporting ArrayBuffer 
>>> > at least for WebSocket.
>>> > So, I think WebKit also should continue to support it in addition 
>>> > to ArrayBufferView.
>>> >
>>> > Thanks,
>>> >
>>> > On Tue, Jun 5, 2012 at 9:32 PM, Yin, Li <li.yin at intel.com> wrote:
>>> >> Hi,
>>> >>
>>> >>      From FileAPI Spec:
>>> >> http://dev.w3.org/2006/webapi/FileAPI/#dfn-Blob
>>> >>
>>> >>      Blob should support ArrayBufferView instead of ArrayBuffer 
>>> >> for Constructor Parameters.
>>> >>
>>> >>      There is a bug in WebKit to track this issue.
>>> >> https://bugs.webkit.org/show_bug.cgi?id=88294
>>> >>
>>> >>      I have submitted patch to do that already.
>>> >>
>>> >>
>>> >>
>>> >> Any comments are appreciated.
>>> >>
>>> >>
>>> >>
>>> >> Thanks,
>>> >>
>>> >> Li Yin
>>> >>
>>> >>
>>> >> _______________________________________________
>>> >> webkit-dev mailing list
>>> >> webkit-dev at lists.webkit.org
>>> >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>>> >>
>>> >
>>> >
>>> >
>>> > --
>>> > Takashi Toyoshima
>>> > Software Engineer, Google
>>> > _______________________________________________
>>> > webkit-dev mailing list
>>> > webkit-dev at lists.webkit.org
>>> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>>> _______________________________________________
>>> webkit-dev mailing list
>>> webkit-dev at lists.webkit.org
>>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>>
>>
>