Reland of name PendingScript::Url() to UrlForClassicScript()

Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1 of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior, because this is renaming only.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed: https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84ef56c08e28a2e

TBR=kouhei@chromium.org
BUG=686281, 711703
Review-Url: https://codereview.chromium.org/2819043002
Cr-Commit-Position: refs/heads/master@{#466906}
Committed: https://chromium.googlesource.com/chromium/src/+/8bb84356d2e79de6c4a88e323dc9c5d6a0df40c4

[hiroshige, 2017-04-14 22:41:53]

Created Reland of name PendingScript::Url() to UrlForClassicScript()

[hiroshige, 2017-04-14 22:44:08]

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

[commit-bot: I haz the power, 2017-04-14 22:44:29]

Dry run: 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-14 22:44:31]

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

[commit-bot: I haz the power, 2017-04-14 22:44:31]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-04-15 00:14:50]

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

[hiroshige, 2017-04-15 00:15:13]

Description was changed from

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=711703
==========

to

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=711703
==========

[commit-bot: I haz the power, 2017-04-15 00:15:23]

Dry run: 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-15 00:15:24]

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

[commit-bot: I haz the power, 2017-04-15 00:15:25]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-04-15 00:16:19]

Description was changed from

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=711703
==========

to

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

[hiroshige, 2017-04-15 00:33:10]

Description was changed from

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

to

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

[hiroshige, 2017-04-15 00:34:16]

Description was changed from

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

to

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior, because this is renaming only.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

[kouhei (in TOK), 2017-04-15 04:32:50]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2017-04-15 04:32:51]

lgtm

[commit-bot: I haz the power, 2017-04-15 04:33:00]

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

[commit-bot: I haz the power, 2017-04-15 04:33:02]

This CL has an open dependency (Issue 2653923008 Patch 760001). 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-22 00:46:32]

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

[commit-bot: I haz the power, 2017-04-22 00:46:48]

Dry run: 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-22 02:29:21]

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

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

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-04-25 00:49:49]

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

[commit-bot: I haz the power, 2017-04-25 00:50:19]

Dry run: 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-25 02:26:14]

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

[commit-bot: I haz the power, 2017-04-25 02:26:15]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-04-25 06:51:51]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2017-04-25 06:51:52]

The patchset sent to the CQ was uploaded after l-g-t-m from kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/2819043002/#ps200001
(title: "Rebase")

[commit-bot: I haz the power, 2017-04-25 06:52:15]

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-25 06:56:41]

CQ is committing da patch.

Bot data: {"patchset_id": 200001, "attempt_start_ts": 1493103111864030,
"parent_rev": "8cc56abb22687b014e5bd8102cb83f438baf2ff5", "commit_rev":
"8bb84356d2e79de6c4a88e323dc9c5d6a0df40c4"}

[commit-bot: I haz the power, 2017-04-25 06:56:51]

Description was changed from

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior, because this is renaming only.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703
==========

to

==========
Reland of name PendingScript::Url() to UrlForClassicScript() (patchset #1 id:1
of https://codereview.chromium.org/2822763002/ )

This CL doesn't change the behavior, because this is renaming only.

Reason for revert:
Issue 711703 is fixed on
https://codereview.chromium.org/2653923008/.

Original issue's description:
> Revert of Rename PendingScript::Url() to UrlForClassicScript() (patchset #1
id:1 of https://codereview.chromium.org/2815163005/ )
> 
> Reason for revert:
> https://codereview.chromium.org/2653923008/ is
> suspected to cause CHECK() failure (crbug.com/711703)
> inside PendingScript's prefinalizer.
> 
> BUG=711703
> 
> Original issue's description:
> > 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...
> 
> TBR=kouhei@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=686281
> 
> Review-Url: https://codereview.chromium.org/2822763002
> Cr-Commit-Position: refs/heads/master@{#464766}
> Committed:
https://chromium.googlesource.com/chromium/src/+/476b2ca298bce5eae94e680ce84e...

TBR=kouhei@chromium.org
BUG=686281, 711703

Review-Url: https://codereview.chromium.org/2819043002
Cr-Commit-Position: refs/heads/master@{#466906}
Committed:
https://chromium.googlesource.com/chromium/src/+/8bb84356d2e79de6c4a88e323dc9...
==========

[commit-bot: I haz the power, 2017-04-25 06:56:53]

Committed patchset #6 (id:200001) as
https://chromium.googlesource.com/chromium/src/+/8bb84356d2e79de6c4a88e323dc9...