Remove ActiveDOMCallback::isScriptControllerTerminating

Remove ActiveDOMCallback::isScriptControllerTerminating

We stopped returning an empty handle from toV8(). Now we can remove the IsEmpty() check
for the handle returned by toV8().

Also this CL removes ActiveDOMCallback::isScriptControllerTerminating.
The check shouldn't have been necessary because if the ScriptController is terminating,
ScriptState::contextIsValid() returns false and we won't reach the check in the first place.

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

[haraken, 2016-08-03 09:18:07]

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

[haraken, 2016-08-03 09:18:07]

PTAL

This CL depends on https://codereview.chromium.org/2209593002/.

[peria, 2016-08-03 09:23:48]

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

[peria, 2016-08-03 09:23:49]

lgtm

[Yuki, 2016-08-03 09:25:07]

lgtm

[haraken, 2016-08-17 06:39:22]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-17 06:39:40]

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-17 10:06:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-17 10:08:17]

Description was changed from

==========
Remove ActiveDOMCallback::isScriptControllerTerminating

We stopped returning an empty handle from toV8(). Now we can remove the
IsEmpty() check
for the handle returned by toV8().

Also this CL removes ActiveDOMCallback::isScriptControllerTerminating.
The check shouldn't have been necessary because if the ScriptController is
terminating,
ScriptState::contextIsValid() returns false and we won't reach the check in the
first place.

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

to

==========
Remove ActiveDOMCallback::isScriptControllerTerminating

We stopped returning an empty handle from toV8(). Now we can remove the
IsEmpty() check
for the handle returned by toV8().

Also this CL removes ActiveDOMCallback::isScriptControllerTerminating.
The check shouldn't have been necessary because if the ScriptController is
terminating,
ScriptState::contextIsValid() returns false and we won't reach the check in the
first place.

BUG=625512

Committed: https://crrev.com/dffcde2825d2d8bbf7d0e19e978ba95a169c147a
Cr-Commit-Position: refs/heads/master@{#412486}
==========

[commit-bot: I haz the power, 2016-08-17 10:08:18]

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