Remove FetchManager::m_isStopped

Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176
Committed: https://crrev.com/1d61819fbeda1b74a725e77185766aba0a0cf82f
Cr-Commit-Position: refs/heads/master@{#437173}

[haraken, 2016-12-07 06:02:16]

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

[haraken, 2016-12-07 06:02:16]

PTAL

[haraken, 2016-12-07 06:04:49]

Description was changed from

==========
Remove FetchManager::m_isStopped

It's redundant. We should just use ExecutionContext::isContextDestroyed().

BUG=610176
==========

to

==========
Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176
==========

[haraken, 2016-12-07 06:05:29]

https://codereview.chromium.org/2557023002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/fetch/GlobalFetch.cpp (left):

https://codereview.chromium.org/2557023002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/fetch/GlobalFetch.cpp:48: if
(m_fetchManager->isStopped()) {

If this returns true, scriptState->contextIsValid() should have returned false
already at line 43.

[yhirano, 2016-12-08 02:11:51]

lgtm

[haraken, 2016-12-08 02:22:07]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-08 02:22:57]

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-12-08 04:06:43]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481163727130690,
"parent_rev": "d73c542c5ed7cd31ba02bd74189fe931c587ef8f", "commit_rev":
"994bd3efb28ab81f9e16475f86b606f827fef885"}

[commit-bot: I haz the power, 2016-12-08 04:07:10]

Description was changed from

==========
Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176
==========

to

==========
Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176
==========

[commit-bot: I haz the power, 2016-12-08 04:07:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-08 04:09:44]

Description was changed from

==========
Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176
==========

to

==========
Remove FetchManager::m_isStopped

It's redundant. GlobalFetch is using FetchManager::m_isStopped to check if
the context has already been destroyed. However, the check is not needed
because scriptState->contextIsValid() is already checking the fact.

BUG=610176

Committed: https://crrev.com/1d61819fbeda1b74a725e77185766aba0a0cf82f
Cr-Commit-Position: refs/heads/master@{#437173}
==========

[commit-bot: I haz the power, 2016-12-08 04:09:45]

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