Need to add V8 bindings for DOMFormData.
Created attachment 50526 [details]
This patch depends on the patch for 36024. The chromium builder might show broken due to that patch 36024 is not landed.
I will check in this patch immediately after landing patch for 36024.
Attachment 50526 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/bindings/v8/custom/V8XMLHttpRequestCustom.cpp:38: Alphabetical sorting problem. [build/include_order] 
Total errors found: 1 in 7 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 50528 [details]
Fix style error
Attachment 50526 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/642025
Attachment 50528 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/623019
Comment on attachment 50528 [details]
Fix style error
Looks good in general. However, I don't see DOMFormData.idl anywhere. I assume that's why the EWS failed to build?
A couple additional minor things:
> diff --git a/WebCore/bindings/v8/DOMObjectsInclude.h b/WebCore/bindings/v8/DOMObjectsInclude.h
> index cd8443a..fd1a875 100644
> --- a/WebCore/bindings/v8/DOMObjectsInclude.h
> +++ b/WebCore/bindings/v8/DOMObjectsInclude.h
> @@ -82,6 +82,7 @@
> #include "DocumentType.h"
> #include "DocumentFragment.h"
> #include "DOMCoreException.h"
> +#include "DOMFormData.h"
> #include "DOMImplementation.h"
> #include "DOMParser.h"
> #include "DOMSelection.h"
I deleted this file today, since it wasn't getting much use. Feel free to revert this part of the change.
> diff --git a/WebCore/bindings/v8/DerivedSourcesAllInOne.cpp b/WebCore/bindings/v8/DerivedSourcesAllInOne.cpp
> index b626499..76ad172 100644
> --- a/WebCore/bindings/v8/DerivedSourcesAllInOne.cpp
> +++ b/WebCore/bindings/v8/DerivedSourcesAllInOne.cpp
> @@ -94,6 +94,7 @@
> #include "bindings/V8DocumentFragment.cpp"
> #include "bindings/V8DocumentType.cpp"
> #include "bindings/V8DOMCoreException.cpp"
> +#include "bindings/V8DOMFormData.cpp"
> #include "bindings/V8DOMImplementation.cpp"
> #include "bindings/V8DOMParser.cpp"
> #include "bindings/V8DOMSelection.cpp"
This file is current unused and about to be deleted as well.
The idl is in the patch for bug 36024.
Created attachment 50562 [details]
Revert changes done to the removed file and the file to be removed.
Attachment 50562 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/588032
Comment on attachment 50562 [details]
Excellent, thanks for the clarification.
Committed as http://trac.webkit.org/changeset/56004.