Rename PendingScript::Url() to UrlForClassicScript()

Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=686281
Review-Url: https://codereview.chromium.org/2815163005
Cr-Commit-Position: refs/heads/master@{#464613}
Committed: https://chromium.googlesource.com/chromium/src/+/06fcf4b92be26a28fde65c743e6a0d40bbc53dc0

[hiroshige, 2017-04-13 17:57:28]

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

[commit-bot: I haz the power, 2017-04-13 17:58:34]

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-13 18:00:14]

Description was changed from

==========
Rename PendingScript::Url() to UrlForClassicScript()

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

to

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=594639, 686281
==========

[hiroshige, 2017-04-13 18:00:14]

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

[hiroshige, 2017-04-13 18:10:11]

PTAL

[commit-bot: I haz the power, 2017-04-13 19:29:36]

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

[commit-bot: I haz the power, 2017-04-13 19:29:38]

Dry run: This issue passed the CQ dry run.

[kouhei (in TOK), 2017-04-13 23:30:06]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2017-04-13 23:30:07]

lgtm

[commit-bot: I haz the power, 2017-04-13 23:30:31]

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

[commit-bot: I haz the power, 2017-04-13 23:30:31]

This CL has an open dependency (Issue 2820533003 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[hiroshige, 2017-04-13 23:43:56]

Description was changed from

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=594639, 686281
==========

to

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

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

[hiroshige, 2017-04-13 23:44:22]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2017-04-13 23:44: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, 2017-04-13 23:49:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492127062851160, "parent_rev":
"603b9c6d7630552558b308527da3142df6c9ffa8", "commit_rev":
"06fcf4b92be26a28fde65c743e6a0d40bbc53dc0"}

[commit-bot: I haz the power, 2017-04-13 23:50:17]

Description was changed from

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

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

to

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=686281

Review-Url: https://codereview.chromium.org/2815163005
Cr-Commit-Position: refs/heads/master@{#464613}
Committed:
https://chromium.googlesource.com/chromium/src/+/06fcf4b92be26a28fde65c743e6a...
==========

[commit-bot: I haz the power, 2017-04-13 23:50:18]

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

[hiroshige, 2017-04-14 18:58:22]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2822763002/ by hiroshige@chromium.org.

The reason for reverting is: https://codereview.chromium.org/2653923008/ is
suspected to cause CHECK() failure (crbug.com/711703)
inside PendingScript's prefinalizer.

BUG=711703.

[hiroshige, 2017-04-14 22:35:51]

Description was changed from

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=686281

Review-Url: https://codereview.chromium.org/2815163005
Cr-Commit-Position: refs/heads/master@{#464613}
Committed:
https://chromium.googlesource.com/chromium/src/+/06fcf4b92be26a28fde65c743e6a...
==========

to

==========
Rename PendingScript::Url() to UrlForClassicScript()

Because it is/can be used only for classic scripts.

BUG=686281

Review-Url: https://codereview.chromium.org/2815163005
Cr-Commit-Position: refs/heads/master@{#464613}
Committed:
https://chromium.googlesource.com/chromium/src/+/06fcf4b92be26a28fde65c743e6a...
==========