IDL: allow optional values to be undefined in overload resolution

IDL: allow optional values to be undefined in overload resolution

Recent changes to overload resolution resulted in
　　IDBObjectStore.createIndex
failing (throwing an exception) if the last (optional) argument,
'options', was undefined; concretely:
　　objectStore.createIndex('i', 'p', undefined)

Per Web IDL spec, this should work: the last argument
is optional, and should be treated as missing.

This worked in M33, but failed in M34, though there was
no test case, hence not caught earlier.

This is a conservative fix, patching it so that this case works: 
overloads with an optional non-wrapper type
(primarily Dictionary)
generate an overload resolution check that allows undefined.
This is presumably a rare case; I haven't changed behavior
for optional arguments otherwise.

The full issue is:
Issue 335871: Treat undefined as missing optional arguments
...which is pending Python rewrite and resolution rewrite.

Caused by this CL (r165620):
Add overload support for non-wrapper type vs. primitive type
https://codereview.chromium.org/131973012
...because it made us stricter about undefined
(not allowing undefined on resolution).

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

[Nils Barth (inactive), 2014-02-04 01:54:51]

haraken, could you PTAL at bindings changes?
Joshua, could you PTAL at test cases?

Thanks!

[haraken, 2014-02-04 02:19:51]

Probably looks OK, so LGTM.

It's a shame that we have to add a lot of ad-hoc hacks every time the
overloading algorithm causes issues. We need to rewrite the overloading
algorithm at some point.

[Nils Barth (inactive), 2014-02-04 02:53:12]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-02-04 02:53:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/152413005/1

[commit-bot: I haz the power, 2014-02-04 06:32:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 06:32:42]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[Nils Barth (inactive), 2014-02-04 07:34:01]

The CQ bit was checked by nbarth@chromium.org

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/152413005/1

[commit-bot: I haz the power, 2014-02-04 09:11:38]

Change committed as 166373

[Nils Barth (inactive), 2014-02-04 11:24:25]

Python side at:
IDL (Python): allow optional values to be undefined in overload resolution
https://codereview.chromium.org/153743002/

(Separate CLs b/c Perl one urgent.)