Introduce ClassicPendingScript::is_external_ explicitly

Introduce ClassicPendingScript::is_external_ explicitly

To clarify whether it is external, and to reduce dependency to
GetResource() and possibly allow us to clear the Resource earlier.

BUG=686281

[hiroshige, 2017-04-29 01:36:18]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-29 01:36:56]

Dry run: CQ is trying da patch.

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

[hiroshige, 2017-04-29 01:41:02]

Description was changed from

==========
Introduce ClassicPendingScript::is_external_ explicitly

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

to

==========
Introduce ClassicPendingScript::is_external_ explicitly

To clarify whether it is external, and to reduce dependency to
GetResource() and possibly allow us to clear the Resource earlier.

BUG=686281
==========

[hiroshige, 2017-04-29 01:41:02]

hiroshige@chromium.org changed reviewers:
+ japhet@chromium.org, kouhei@chromium.org, sigbjornf@opera.com

[hiroshige, 2017-04-29 01:41:18]

PTAL.

[hiroshige, 2017-04-29 01:41:34]

hiroshige@chromium.org changed reviewers:
+ jbroman@chromium.org

[commit-bot: I haz the power, 2017-04-29 03:11:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-29 03:11:22]

Dry run: This issue passed the CQ dry run.

[sof, 2017-04-29 15:01:10]

https://codereview.chromium.org/2846363003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/ClassicPendingScript.cpp (right):

https://codereview.chromium.org/2846363003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/ClassicPendingScript.cpp:177: if
(is_external_) {
Why is this (and all the other is_external_ checks added here) 'better' than the
GetResource() checks previously there?

[hiroshige, 2017-08-04 22:09:57]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-04 22:10:06]

Dry run: CQ is trying da patch.

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

[hiroshige, 2017-08-04 23:07:43]

Moving to Gerrit:
https://chromium-review.googlesource.com/c/602256