Revert of Check whether ScriptResource is still loading due to revalidation

Revert of Check whether ScriptResource is still loading due to revalidation (patchset #2 id:20001 of https://codereview.chromium.org/2706303002/ )

Reason for revert:
Sufficient data are collected and thus finishing the
investigation.

BUG=696305
Original issue's description:
> Check whether ScriptResource is still loading due to revalidation
> 
> ScriptResource is assumed to be already loaded when it is used by
> ScriptResource::script() and PendingScript.
> However, there might slight chances that isLoaded() is false due to
> revalidation, possibly causing Issue 692856.
> 
> In order to check whether and how often revalidation can cause
> isLoaded() to be false in existing DCHECK()s, this CL replaces
> DCHECK(isLoaded()) with three CHECK()s, each of which fails when:
> 1. isLoaded() is false not because of revalidation,
> 2. isLoaded() is false because revalidation is ongoing and response is
>    already received, and
> 3. isLoaded() is false because revalidation is ongoing and response is
>    not yet received.
> 
> BUG=692856
> 
> Review-Url: https://codereview.chromium.org/2706303002
> Cr-Commit-Position: refs/heads/master@{#453057}
> Committed: https://chromium.googlesource.com/chromium/src/+/6dc58b5746363ddbb99addb4c06c69a385212b7d

TBR=japhet@chromium.org,kouhei@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=692856
Review-Url: https://codereview.chromium.org/2714403002
Cr-Commit-Position: refs/heads/master@{#453150}
Committed: https://chromium.googlesource.com/chromium/src/+/4c7dd9f84d0f9b306bea368e4015fae01fcd6541

[hiroshige, 2017-02-27 03:18:26]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2017-02-27 03:18:27]

Created Revert of Check whether ScriptResource is still loading due to
revalidation

[commit-bot: I haz the power, 2017-02-27 03:18:40]

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

[kouhei (in TOK), 2017-02-27 03:22:43]

lgtm

[commit-bot: I haz the power, 2017-02-27 04:52:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488165506405330, "parent_rev":
"264049d2cf95e68dfff844792ef78894c78e7749", "commit_rev":
"4c7dd9f84d0f9b306bea368e4015fae01fcd6541"}

[commit-bot: I haz the power, 2017-02-27 04:53:27]

Description was changed from

==========
Revert of Check whether ScriptResource is still loading due to revalidation
(patchset #2 id:20001 of https://codereview.chromium.org/2706303002/ )

Reason for revert:
Sufficient data are collected and thus finishing the
investigation.

BUG=696305


Original issue's description:
> Check whether ScriptResource is still loading due to revalidation
> 
> ScriptResource is assumed to be already loaded when it is used by
> ScriptResource::script() and PendingScript.
> However, there might slight chances that isLoaded() is false due to
> revalidation, possibly causing Issue 692856.
> 
> In order to check whether and how often revalidation can cause
> isLoaded() to be false in existing DCHECK()s, this CL replaces
> DCHECK(isLoaded()) with three CHECK()s, each of which fails when:
> 1. isLoaded() is false not because of revalidation,
> 2. isLoaded() is false because revalidation is ongoing and response is
>    already received, and
> 3. isLoaded() is false because revalidation is ongoing and response is
>    not yet received.
> 
> BUG=692856
> 
> Review-Url: https://codereview.chromium.org/2706303002
> Cr-Commit-Position: refs/heads/master@{#453057}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6dc58b5746363ddbb99addb4c06c...

TBR=japhet@chromium.org,kouhei@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=692856
==========

to

==========
Revert of Check whether ScriptResource is still loading due to revalidation
(patchset #2 id:20001 of https://codereview.chromium.org/2706303002/ )

Reason for revert:
Sufficient data are collected and thus finishing the
investigation.

BUG=696305


Original issue's description:
> Check whether ScriptResource is still loading due to revalidation
> 
> ScriptResource is assumed to be already loaded when it is used by
> ScriptResource::script() and PendingScript.
> However, there might slight chances that isLoaded() is false due to
> revalidation, possibly causing Issue 692856.
> 
> In order to check whether and how often revalidation can cause
> isLoaded() to be false in existing DCHECK()s, this CL replaces
> DCHECK(isLoaded()) with three CHECK()s, each of which fails when:
> 1. isLoaded() is false not because of revalidation,
> 2. isLoaded() is false because revalidation is ongoing and response is
>    already received, and
> 3. isLoaded() is false because revalidation is ongoing and response is
>    not yet received.
> 
> BUG=692856
> 
> Review-Url: https://codereview.chromium.org/2706303002
> Cr-Commit-Position: refs/heads/master@{#453057}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6dc58b5746363ddbb99addb4c06c...

TBR=japhet@chromium.org,kouhei@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=692856

Review-Url: https://codereview.chromium.org/2714403002
Cr-Commit-Position: refs/heads/master@{#453150}
Committed:
https://chromium.googlesource.com/chromium/src/+/4c7dd9f84d0f9b306bea368e4015...
==========

[commit-bot: I haz the power, 2017-02-27 04:53:29]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4c7dd9f84d0f9b306bea368e4015...