Make more [TreatNullAs=NullString] DOMString arguments nullable

Make more [TreatNullAs=NullString] DOMString arguments nullable

Fix handling of DOMString? combined with [TreatUndefinedAs=NullString] to
behave the same as [TreatNullAs=NulllString] combined with it, and then
change method arguments with TreatNullAs+TreatUndefinedAs to nullable with
just TreatUndefinedAs.

This does not alter behavior or change generated code.

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

[Jens Widell, 2014-07-02 12:42:20]

PTAL

https://codereview.chromium.org/367003002/diff/1/Source/core/dom/Document.idl
File Source/core/dom/Document.idl (right):

https://codereview.chromium.org/367003002/diff/1/Source/core/dom/Document.idl...
Source/core/dom/Document.idl:101: [TreatUndefinedAs=NullString,
Default=Undefined] optional DOMString? value);
If we fix http://crbug.com/335871, then in terms of behavior, this could just be
"optional DOMString? value = null". (CG would differ somewhat, though.)

The same probably goes for various arguments touched in this CL that don't have
[Default=Undefined], but in those cases it depends on how the C++ side is
implemented.

[haraken, 2014-07-02 12:50:27]

LGTM

https://codereview.chromium.org/367003002/diff/1/Source/core/dom/Document.idl
File Source/core/dom/Document.idl (right):

https://codereview.chromium.org/367003002/diff/1/Source/core/dom/Document.idl...
Source/core/dom/Document.idl:101: [TreatUndefinedAs=NullString,
Default=Undefined] optional DOMString? value);
On 2014/07/02 12:42:20, Jens Lindström wrote:
> If we fix http://crbug.com/335871, then in terms of behavior, this could just
be
> "optional DOMString? value = null". (CG would differ somewhat, though.)
> 
> The same probably goes for various arguments touched in this CL that don't
have
> [Default=Undefined], but in those cases it depends on how the C++ side is
> implemented.

Yeah, it's worth fixing.

[Jens Widell, 2014-07-02 14:22:52]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-07-02 14:23:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jl@opera.com/367003002/1

[commit-bot: I haz the power, 2014-07-02 15:34:34]

Change committed as 177396