Permissions: use newly supported required attribute on Dictionary properties.

Permissions: use newly supported required attribute on Dictionary properties.

This is removing the custom checks and rely on the bindings instead.
Also adding a couple of tests regarding bindings.

This CL doesn't change any behaviour.

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

[mlamouri (slow - plz ping), 2015-04-03 13:02:22]

mlamouri@chromium.org changed reviewers:
+ avayvod@chromium.org

[whywhat, 2015-04-03 13:09:21]

Shouldn't you have a then() with assert_notreached() in your second test case
(with empty name)?

I'd comment on the exact line but it's impossible from Nexus 9 apparently :(

lgtm

[mlamouri (slow - plz ping), 2015-04-03 13:11:48]

On 2015/04/03 at 13:09:21, avayvod wrote:
> Shouldn't you have a then() with assert_notreached() in your second test case
(with empty name)?
> 
> I'd comment on the exact line but it's impossible from Nexus 9 apparently :(
> 
> lgtm

Hmm, I have one. Each then() here has two methods, one for success and one for
failure. WebIDL failures will reject the promise.

[mlamouri (slow - plz ping), 2015-04-03 13:15:55]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-04-03 13:16:04]

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

[commit-bot: I haz the power, 2015-04-03 14:25:28]

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