Make Dictionary::toV8Impl return v8::Undefined()

Make Dictionary::toV8Impl return v8::Undefined()

I'm now making toV8() never return an empty handle.
This CL makes Dictionary::toV8Impl return v8::Undefined() instead of an empty handle.
This shouldn't change any web-exposed behavior, since V8 interprets
a returned empty handle as v8::Undefined().

BUG=625512
Committed: https://crrev.com/5eb32ca62f5540448259a9df83a9752ac733257d
Cr-Commit-Position: refs/heads/master@{#415228}

[haraken, 2016-08-26 09:24:36]

haraken@chromium.org changed reviewers:
+ bashi@chromium.org

[haraken, 2016-08-26 09:24:36]

PTAL

[bashi, 2016-08-28 23:01:04]

The CQ bit was checked by bashi@chromium.org

[bashi, 2016-08-28 23:01:04]

lgtm

[commit-bot: I haz the power, 2016-08-28 23:01:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-29 00:13:42]

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

[commit-bot: I haz the power, 2016-08-29 00:13:43]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[haraken, 2016-08-29 16:32:32]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-29 16:45:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2016-08-29 21:10:41]

The CQ bit was unchecked by haraken@chromium.org

[haraken, 2016-08-29 21:10:45]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-29 21:57:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 04:51:52]

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

[commit-bot: I haz the power, 2016-08-30 04:51:53]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[haraken, 2016-08-30 04:55:33]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-30 04:57:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 06:52:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 06:54:24]

Description was changed from

==========
Make Dictionary::toV8Impl return v8::Undefined()

I'm now making toV8() never return an empty handle.
This CL makes Dictionary::toV8Impl return v8::Undefined() instead of an empty
handle.
This shouldn't change any web-exposed behavior, since V8 interprets
a returned empty handle as v8::Undefined().

BUG=625512
==========

to

==========
Make Dictionary::toV8Impl return v8::Undefined()

I'm now making toV8() never return an empty handle.
This CL makes Dictionary::toV8Impl return v8::Undefined() instead of an empty
handle.
This shouldn't change any web-exposed behavior, since V8 interprets
a returned empty handle as v8::Undefined().

BUG=625512

Committed: https://crrev.com/5eb32ca62f5540448259a9df83a9752ac733257d
Cr-Commit-Position: refs/heads/master@{#415228}
==========

[commit-bot: I haz the power, 2016-08-30 06:54:26]

Patchset 1 (id:??) landed as
https://crrev.com/5eb32ca62f5540448259a9df83a9752ac733257d
Cr-Commit-Position: refs/heads/master@{#415228}