Use IDL argument default values for Text()/Comment() constructors

Use IDL argument default values for Text()/Comment() constructors

The 'data' argument should default to "", not null.

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

[Jens Widell, 2014-06-26 15:14:57]

PTAL

There's an isNull() check in CharacterData's constructor (in CharacterData.h)
that might not be needed now, but both Text::create() and Comment::create() are
called from various C++ code, so it's hard to tell if that check can safely be
removed.

[haraken, 2014-06-26 15:36:16]

OK, this doesn't change web-exposed behavior. LGTM.

[Jens Widell, 2014-06-26 15:40:35]

On 2014/06/26 15:36:16, haraken wrote:
> OK, this doesn't change web-exposed behavior.

No, the actual default was always an empty string. This change just makes the
IDL match the spec, and gets rid of invalid null default values on non-nullable
arguments.

[Jens Widell, 2014-06-26 16:08:58]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-06-26 16:10:31]

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

[commit-bot: I haz the power, 2014-06-26 19:14:58]

Change committed as 177016