Minor clean-up on WindowProxy::disposeContext()

Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253
Committed: https://crrev.com/488433ae435fbcd7ea4eac85f7042e7623f6bdef
Cr-Commit-Position: refs/heads/master@{#440839}

[haraken, 2016-12-28 08:33:21]

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

[haraken, 2016-12-28 08:33:21]

PTAL

[Yuki, 2016-12-28 08:45:07]

LGTM.

[haraken, 2016-12-28 08:46:19]

The CQ bit was checked by haraken@chromium.org

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

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

[haraken, 2016-12-28 08:52:00]

Description was changed from

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

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

to

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253
==========

[commit-bot: I haz the power, 2016-12-28 10:04:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482914779781080, "parent_rev":
"f996716a2198967d9c38be8150aeca7b7e94f9df", "commit_rev":
"3d281cdb2ed2d4e6825662e2e090f4542d85c940"}

[commit-bot: I haz the power, 2016-12-28 10:05:06]

Description was changed from

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253
==========

to

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253

Review-Url: https://codereview.chromium.org/2601033002
==========

[commit-bot: I haz the power, 2016-12-28 10:05:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:48:43]

Description was changed from

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253

Review-Url: https://codereview.chromium.org/2601033002
==========

to

==========
Minor clean-up on WindowProxy::disposeContext()

We don't need to check isContextInitialized() in clearForClose() and
clearForNavigation() because it is already checked in
WindowProxy::disposeContext().

Also this CL moves the ScriptState::Scope into disposeContext()
Though we don't need to enter ScriptState::Scope in case of clearForClose()
by accident, it looks safer to enter the scope always.

BUG=677253

Committed: https://crrev.com/488433ae435fbcd7ea4eac85f7042e7623f6bdef
Cr-Commit-Position: refs/heads/master@{#440839}
==========

[commit-bot: I haz the power, 2017-01-02 15:48:43]

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