Please remove BlobBuilder (currently prefixed WebKitBlobBuilder). It has been replaced with the Blob() constructor in the File API spec. Also see https://bugzilla.mozilla.org/show_bug.cgi?id=744907
<rdar://problem/11261686>
Given that Chrome has shipped WebKitBlobBuilder for while, we will have to tred carefully here. As a first step, we should probably just implement the Blob constructor.
I filed https://bugs.webkit.org/show_bug.cgi?id=84555 to track adding the Blob constructor.
Created attachment 139412 [details] Patch
This patch adds a flag, ENABLE_LEGACY_WEBKIT_BLOB_BUILDER, to control exposing the WebKitBlobBuilder. (It doesn't control whether the C++ class is compiled, as that is used internally). It also disables WebKitBlobBuilder on the Mac port, since we have never shipped it, it doesn't make sense to keep it enabled.
Comment on attachment 139412 [details] Patch Attachment 139412 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/12582102
Created attachment 139499 [details] Patch
{ option => "legacy-notifications", desc => "Toggle Legacy WebKitBlobBuilder Support", Is that first string correct?
Comment on attachment 139499 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=139499&action=review > Tools/Scripts/build-webkit:271 > + { option => "legacy-notifications", desc => "Toggle Legacy WebKitBlobBuilder Support", > + define => "ENABLE_LEGACY_NOTIFICATIONS", default => (isGtk() || isChromium() || isBlackBerry()), value => \$legacyWebKitBlobBuilderSupport }, I don't think this is the right option name...
Created attachment 139507 [details] Patch
Committed r115666: <http://trac.webkit.org/changeset/115666>