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

[Kenneth Russell]

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
>>
>>
>