Sync the XHR-related interfaces with the spec

Sync the XHR-related interfaces with the spec

No observable changes are intended.

The change to open() results in big changes to the generated code, but
it should not be possible to observe the difference; the 2-argument form
still treats async as true, and passing undefined for either username or
password will still be converted to nullptr and then a null string.

For both open() and send(), handing the default argument values in
bindings allows the C++ code to be slightly simpler.

Dropping [TreatReturnedNullStringAs=Undefined] for
getAllResponseHeaders() makes no difference as
XMLHttpRequest::getAllResponseHeaders() will never return a null string.
Even when there are no headers, the untouched StringBuilder object will
return an empty string from toString(), not a null string.

The generated code now handles missing ProgressEventInit members, so the
ProgressEvent constructor doesn't need to.

BUG=460722
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196221

[philipj_slow, 2015-05-29 21:50:43]

philipj@opera.com changed reviewers:
+ yhirano@chromium.org

[philipj_slow, 2015-05-29 21:50:44]

yhirano@, can you PTAL?

[yhirano, 2015-06-01 01:25:12]

LGTM for core/xmlhttprequest

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
File Source/core/xmlhttprequest/XMLHttpRequest.idl (right):

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
Source/core/xmlhttprequest/XMLHttpRequest.idl:71: // TODO(philipj): The data
argument should be of type (Document or BodyInit)?
Please wrap the comment in 80 columns.

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
Source/core/xmlhttprequest/XMLHttpRequest.idl:72: [RaisesException] void
send(optional (ArrayBuffer or ArrayBufferView or Blob or Document or DOMString
or FormData)? data = null);
The parameter name should be |body| instead of |data|?

[philipj_slow, 2015-06-01 11:59:54]

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
File Source/core/xmlhttprequest/XMLHttpRequest.idl (right):

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
Source/core/xmlhttprequest/XMLHttpRequest.idl:71: // TODO(philipj): The data
argument should be of type (Document or BodyInit)?
On 2015/06/01 01:25:12, yhirano wrote:
> Please wrap the comment in 80 columns.

Done.

https://codereview.chromium.org/1167563002/diff/20001/Source/core/xmlhttprequ...
Source/core/xmlhttprequest/XMLHttpRequest.idl:72: [RaisesException] void
send(optional (ArrayBuffer or ArrayBufferView or Blob or Document or DOMString
or FormData)? data = null);
On 2015/06/01 01:25:12, yhirano wrote:
> The parameter name should be |body| instead of |data|?

Done.

[philipj_slow, 2015-06-01 12:00:17]

philipj@opera.com changed reviewers:
+ jl@opera.com

[philipj_slow, 2015-06-01 12:00:18]

jl@, can you please review the rest of core/?

[Jens Widell, 2015-06-01 12:16:59]

LGTM

[philipj_slow, 2015-06-01 12:27:34]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-06-01 12:27:34]

The patchset sent to the CQ was uploaded after l-g-t-m from yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/1167563002/#ps40001
(title: "data->body")

[commit-bot: I haz the power, 2015-06-01 12:27:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1167563002/40001

[commit-bot: I haz the power, 2015-06-01 13:18:37]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196221