Agree; remove the warning. -Ken
On Wed, Mar 20, 2013 at 10:33 AM, Takashi Toyoshima <toyos...@chromium.org> wrote: > +1 on removing the warning. > > > On Wed, Mar 20, 2013 at 10:24 AM, Kinuko Yasuda <kin...@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....@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:k...@google.com] >>> Sent: Wednesday, June 06, 2012 5:21 AM >>> To: Takashi Toyoshima >>> Cc: Yin, Li; webkit-dev@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 <toyos...@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....@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@lists.webkit.org >>> >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >>> >> >>> > >>> > >>> > >>> > -- >>> > Takashi Toyoshima >>> > Software Engineer, Google >>> > _______________________________________________ >>> > webkit-dev mailing list >>> > webkit-dev@lists.webkit.org >>> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >>> _______________________________________________ >>> webkit-dev mailing list >>> webkit-dev@lists.webkit.org >>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> >> > _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev