Revert of Check whether ScriptResource is still loading due to revalidation (patchset #2 id:20001 o…

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
BUG=692856
Review-Url: https://codereview.chromium.org/2714403002
Cr-Commit-Position: refs/heads/master@{#453150}
(cherry picked from commit 4c7dd9f84d0f9b306bea368e4015fae01fcd6541)

Review-Url: https://codereview.chromium.org/2723583002 .
Cr-Commit-Position: refs/branch-heads/3025@{#4}
Cr-Branched-From: 636f441be350ec76e73b4e4f407bf8a7e86984fd-refs/heads/master@{#453134}
Committed: https://chromium.googlesource.com/chromium/src/+/7f6cd2d8dede99b5d71072e115195de55ae2cacb

[hiroshige, 2017-02-27 19:11:05]

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
BUG=692856

Review-Url: https://codereview.chromium.org/2714403002
Cr-Commit-Position: refs/heads/master@{#453150}
(cherry picked from commit 4c7dd9f84d0f9b306bea368e4015fae01fcd6541)
==========

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
BUG=692856

Review-Url: https://codereview.chromium.org/2714403002
Cr-Commit-Position: refs/heads/master@{#453150}
(cherry picked from commit 4c7dd9f84d0f9b306bea368e4015fae01fcd6541)

Review-Url: https://codereview.chromium.org/2723583002 .
Cr-Commit-Position: refs/branch-heads/3025@{#4}
Cr-Branched-From:
636f441be350ec76e73b4e4f407bf8a7e86984fd-refs/heads/master@{#453134}
Committed:
https://chromium.googlesource.com/chromium/src/+/7f6cd2d8dede99b5d71072e11519...
==========

[hiroshige, 2017-02-27 19:11:06]

Committed patchset #1 (id:1) manually as
7f6cd2d8dede99b5d71072e115195de55ae2cacb.

[kouhei (in TOK), 2017-02-28 00:44:09]

lgtm