Use union type for NavigatorBeacon.sendBeacon()'s data argument

Use union type for NavigatorBeacon.sendBeacon()'s data argument

This lets us lose some overloading and code duplication.

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

[Jens Widell, 2014-11-11 13:43:14]

jl@opera.com changed reviewers:
+ bashi@chromium.org, haraken@chromium.org, sigbjornf@opera.se

[Jens Widell, 2014-11-11 13:43:14]

PTAL

Again, this is not ready to land. A code generator bug causes
UnionTypesModules.h not to be included in the generated V8NavigatorPartial.cpp,
so this patch breaks compilation.

bashi@: Is this sort of problem something you're currently working on?

[bashi, 2014-11-11 23:27:56]

On 2014/11/11 13:43:14, Jens Widell wrote:
> PTAL
> 
> Again, this is not ready to land. A code generator bug causes
> UnionTypesModules.h not to be included in the generated
V8NavigatorPartial.cpp,
> so this patch breaks compilation.
> 
> bashi@: Is this sort of problem something you're currently working on?

Replacing overloadings are on my TODO list, but I don't have concrete CLs nor
plans for them now so I would appreciate if you could do that:) Thank you so
much for working on this!

[Jens Widell, 2014-11-13 11:38:55]

On 2014/11/11 13:43:14, Jens Widell wrote:
> PTAL
> 
> Again, this is not ready to land. A code generator bug causes
> UnionTypesModules.h not to be included in the generated
V8NavigatorPartial.cpp,
> so this patch breaks compilation.

This *is* ready to land now, in case anyone was waiting for that. ;-)

[haraken, 2014-11-13 11:48:25]

LGTM

[Jens Widell, 2014-11-13 14:42:02]

The CQ bit was checked by jl@opera.com

[Jens Widell, 2014-11-13 14:42:11]

The CQ bit was unchecked by jl@opera.com

[sof, 2014-11-13 14:51:08]

sigbjornf@opera.com changed reviewers:
+ sigbjornf@opera.com

[sof, 2014-11-13 14:51:09]

lgtm

[Jens Widell, 2014-11-13 15:00:04]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-11-13 15:00:35]

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

[commit-bot: I haz the power, 2014-11-13 15:02:36]

Committed patchset #2 (id:20001) as 185293

[Jens Widell, 2014-11-13 16:59:00]

jl@opera.com changed reviewers:
- sigbjornf@opera.se