Bug 78111 – Add unprefixed Blob.webkitSlice (slice)

• Component: New Bugs
• Status: RESOLVED FIXED

[Simon Pieters (:zcorpan)]

Please rename Blob.webkitSlice to Blob.slice. Opera is working on changing the semantics of Blob.slice, but we don't want to prefix it. Since Gecko and WebKit have been shipping without Blob.slice support for several months, the existing content, if any, that relied on the old semantics of Blob.slice have had the chance to break and get updated. With Opera soon shipping Blob.slice with the new semantics, it's time to drop the prefix.

[Kinuko Yasuda]

Created attachment 146823 [details]
Patch

[Gustavo Noronha (kov)]

Comment on attachment 146823 [details]
Patch

Attachment 146823 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/12943214

[Kinuko Yasuda]

Created attachment 146833 [details]
Patch

[Kinuko Yasuda]

Created attachment 146839 [details]
Patch

[Kinuko Yasuda]

Created attachment 146843 [details]
Patch

[Gustavo Noronha (kov)]

Comment on attachment 146843 [details]
Patch

Attachment 146843 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/12941324

[Kinuko Yasuda]

Created attachment 146848 [details]
Patch

[WebKit Review Bot]

Comment on attachment 146848 [details]
Patch

Attachment 146848 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/12941361

New failing tests:
fast/filesystem/workers/file-writer-truncate-extend.html
fast/filesystem/workers/file-writer-write-overlapped.html
fast/filesystem/file-writer-truncate-extend.html
inspector/profiler/heap-snapshot-loader.html

[WebKit Review Bot]

Created attachment 146867 [details]
Archive of layout-test-results from ec2-cr-linux-04

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-04  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick

[Kinuko Yasuda]

Created attachment 147024 [details]
Patch

[Kinuko Yasuda]

Could someone review this?

[Adam Barth]

Comment on attachment 147024 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=147024&action=review

> Source/WebCore/bindings/gobject/WebKitDOMCustom.h:32
> -WEBKIT_API WebKitDOMBlob* webkit_dom_blob_slice(WebKitDOMBlob* self, gint64 start, gint64 end, const gchar* content_type);
> +WEBKIT_API WebKitDOMBlob* webkit_dom_blob_webkit_slice(WebKitDOMBlob* self, gint64 start, gint64 end, const gchar* content_type);

Why the addition of "webkit" here?  I would have thought we'd be happy with fewer webkits, not more.

> Source/WebCore/fileapi/Blob.cpp:97
> +    String message("Blob.webkitSlice() is deprecated. Use Blob.slice() instead .");

Looks like you've got an extra space before the .

> Source/WebCore/fileapi/Blob.h:75
> +    // Prefixed version is going to be deprecated. This internally calls sliceInternal() (as slice() does) after showing a deprecation message.

"is going to be deprecated" => "is deprecated" :)

[Kinuko Yasuda]

(In reply to comment #12)
> (From update of attachment 147024 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=147024&action=review
> 
> > Source/WebCore/bindings/gobject/WebKitDOMCustom.h:32
> > -WEBKIT_API WebKitDOMBlob* webkit_dom_blob_slice(WebKitDOMBlob* self, gint64 start, gint64 end, const gchar* content_type);
> > +WEBKIT_API WebKitDOMBlob* webkit_dom_blob_webkit_slice(WebKitDOMBlob* self, gint64 start, gint64 end, const gchar* content_type);
> 
> Why the addition of "webkit" here?  I would have thought we'd be happy with fewer webkits, not more.

It's for keeping backward compatibility, because I added !LANGUAGE_GOBJECT not to generate webkitSlice binding for gobject as its binding doesn't support CallWith=ScriptExecutionContext. (Hope this works for gtk)

> > Source/WebCore/fileapi/Blob.cpp:97
> > +    String message("Blob.webkitSlice() is deprecated. Use Blob.slice() instead .");
> 
> Looks like you've got an extra space before the .
> 
> > Source/WebCore/fileapi/Blob.h:75
> > +    // Prefixed version is going to be deprecated. This internally calls sliceInternal() (as slice() does) after showing a deprecation message.
> 
> "is going to be deprecated" => "is deprecated" :)

Will fix these nits.  Thanks for your review!

[Adam Barth]

Ah!  I understand.  Thanks.  :)

[Kinuko Yasuda]

Committed r120165: <http://trac.webkit.org/changeset/120165>