binding: Makes ExceptionState STACK_ALLOCATED().

binding: Makes ExceptionState STACK_ALLOCATED().

Since we're going to make ExceptionStates scope-like objects at
http://crrev.com/2272613003 , ExceptionState should be
STACK_ALLOCATED().

BUG=

Committed: https://crrev.com/816254910aa039c21e728a7ff5d518174f9a2dfc
Cr-Commit-Position: refs/heads/master@{#415241}

[Yuki, 2016-08-25 13:28:09]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 13:28:29]

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

[Yuki, 2016-08-25 13:29:36]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 13:29:57]

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

[Yuki, 2016-08-25 13:40:32]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 13:40:55]

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

[Yuki, 2016-08-25 13:41:29]

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

[Yuki, 2016-08-25 13:41:30]

Could you guys review this CL?

[commit-bot: I haz the power, 2016-08-25 14:08:25]

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

[commit-bot: I haz the power, 2016-08-25 14:08:26]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[peria, 2016-08-25 15:31:18]

peria@chromium.org changed reviewers:
+ peria@chromium.org

[peria, 2016-08-25 15:31:18]

My comments are nits, so you don't need my LG.

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp (right):

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp:42: return
blink::isUndefinedOrNull(m_array);
We don't need 'blink::' here.

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ArrayValue.h (right):

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ArrayValue.h:42: explicit
ArrayValue(const v8::Local<v8::Array>& array, v8::Isolate* isolate)
This is not related to your change, but we don't need 'explicit' here.

[bashi, 2016-08-25 23:13:54]

lgtm once AnimationTest build failure is fixed.

[haraken, 2016-08-26 01:08:51]

LGTM

[Yuki, 2016-08-29 06:52:51]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-29 06:53:00]

Dry run: 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 07:57:00]

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

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

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yuki, 2016-08-29 09:30:01]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-29 09:30:23]

Dry run: 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 10:55:06]

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

[commit-bot: I haz the power, 2016-08-29 10:55:07]

Dry run: 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...)

[Yuki, 2016-08-30 04:44:21]

The CQ bit was checked by yukishiino@chromium.org to run a CQ dry run

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

Dry run: 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 05:45:49]

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

[commit-bot: I haz the power, 2016-08-30 05:45:50]

Dry run: This issue passed the CQ dry run.

[Yuki, 2016-08-30 06:09:07]

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp (right):

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp:42: return
blink::isUndefinedOrNull(m_array);
On 2016/08/25 15:31:18, peria wrote:
> We don't need 'blink::' here.

We do need it because ArrayValue::isUndefinedOrNull shadows
blink::isUndefinedOrNull.

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/ArrayValue.h (right):

https://codereview.chromium.org/2272313003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/ArrayValue.h:42: explicit
ArrayValue(const v8::Local<v8::Array>& array, v8::Isolate* isolate)
On 2016/08/25 15:31:18, peria wrote:
> This is not related to your change, but we don't need 'explicit' here.

Done.

[Yuki, 2016-08-30 06:09:15]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2016-08-30 06:09:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, bashi@chromium.org
Link to the patchset: https://codereview.chromium.org/2272313003/#ps120001
(title: "Synced.")

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

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 08:14:31]

Committed patchset #7 (id:120001)

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

Description was changed from

==========
binding: Makes ExceptionState STACK_ALLOCATED().

Since we're going to make ExceptionStates scope-like objects at
http://crrev.com/2272613003 , ExceptionState should be
STACK_ALLOCATED().

BUG=
==========

to

==========
binding: Makes ExceptionState STACK_ALLOCATED().

Since we're going to make ExceptionStates scope-like objects at
http://crrev.com/2272613003 , ExceptionState should be
STACK_ALLOCATED().

BUG=

Committed: https://crrev.com/816254910aa039c21e728a7ff5d518174f9a2dfc
Cr-Commit-Position: refs/heads/master@{#415241}
==========

[commit-bot: I haz the power, 2016-08-30 08:16:46]

Patchset 7 (id:??) landed as
https://crrev.com/816254910aa039c21e728a7ff5d518174f9a2dfc
Cr-Commit-Position: refs/heads/master@{#415241}