bindings: Use IDL dictionary to implement FontFaceSetLoadEventInit (issue 1159403005 by bashi@chromium.org)

[ba...@chromium.org]

Reviewers: haraken, Kunihiko Sakamoto,

Message:
PTAL?

Since FontFaceSetLoadEvent is [NoInterfaceObject] we don't have any layout
tests
for the constructor. Per the spec, it should be exposed, but I guess that we
need to send an intent-to-ship before exposing FontFaceSetLoadEvent.
Exposing
the event isn't the purpose of this CL so I didn't remove
[NoInterfaceObject].

Description:
bindings: Use IDL dictionary to implement FontFaceSetLoadEventInit

This CL also adds an empty array default value ("[]") in IDL dictionary
as FontFaceSetLoadEventInit requires it.

BUG=433179

Please review this at https://codereview.chromium.org/1159403005/

Base URL: svn://svn.chromium.org/blink/trunk

Affected files (+117, -8 lines):
M Source/bindings/scripts/v8_types.py
M Source/bindings/tests/idls/core/TestDictionary.idl
M Source/bindings/tests/results/core/TestDictionary.h
M Source/bindings/tests/results/core/TestDictionary.cpp
M Source/bindings/tests/results/core/V8TestDictionary.cpp
M Source/core/core.gypi
M Source/core/css/FontFaceSetLoadEvent.h
M Source/core/css/FontFaceSetLoadEvent.cpp
A + Source/core/css/FontFaceSetLoadEventInit.idl

[har...@chromium.org]

LGTM



https://codereview.chromium.org/1159403005/

[ksak...@chromium.org]

core/css lgtm

https://codereview.chromium.org/1159403005/

['commit-bot: I haz the power' via codereview.chromium.org]

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


https://codereview.chromium.org/1159403005/

['commit-bot: I haz the power' via codereview.chromium.org]

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

https://codereview.chromium.org/1159403005/