Revert of Optimize KURL protocols

Revert of Optimize KURL protocols (patchset #9 id:160001 of https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
>
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
>
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
>
> BUG=348655
>
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655, 674388
Committed: https://crrev.com/8885747f52b034575336f97e354fa8043f6c1b9f
Cr-Commit-Position: refs/heads/master@{#439722}

[Charlie Harrison, 2016-12-20 04:16:24]

The CQ bit was checked by csharrison@chromium.org

[Charlie Harrison, 2016-12-20 04:16:25]

Created Revert of Optimize KURL protocols

[Charlie Harrison, 2016-12-20 04:16:38]

Description was changed from

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
> 
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
> 
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
> 
> BUG=348655
> 
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655
==========

to

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
> 
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
> 
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
> 
> BUG=348655
> 
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655,674388
==========

[commit-bot: I haz the power, 2016-12-20 04:16:51]

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

[Charlie Harrison, 2016-12-20 04:17:10]

TBR esprehn for revert

[commit-bot: I haz the power, 2016-12-20 06:03:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482207384996360, "parent_rev":
"0db5bde56bb8da6aa985e18a00c4baabc31fa3b1", "commit_rev":
"6d73cce4368d2795addfe5e7247ce49cc561fdf5"}

[commit-bot: I haz the power, 2016-12-20 06:03:33]

Description was changed from

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
> 
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
> 
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
> 
> BUG=348655
> 
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655,674388
==========

to

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
> 
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
> 
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
> 
> BUG=348655
> 
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655,674388

Review-Url: https://codereview.chromium.org/2586253004
==========

[commit-bot: I haz the power, 2016-12-20 06:03:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 06:08:03]

Description was changed from

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
> 
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
> 
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
> 
> BUG=348655
> 
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655,674388

Review-Url: https://codereview.chromium.org/2586253004
==========

to

==========
Revert of Optimize KURL protocols (patchset #9 id:160001 of
https://codereview.chromium.org/2463703002/ )

Reason for revert:
Causing DCHECK flakes. crbug.com/674388

Original issue's description:
> Optimize KURL protocols
>
> This patch optimizes KURL::protocol and KURL::protocolIs by keeping
> an AtomicString m_protocol on KURL. This reduces string allocations
> throughout the code using KURL::protocol().
>
> This also fixes an inconsistency with KURL::protocolIs that will return
> true for invalid URLs.
>
> BUG=348655
>
> Committed: https://crrev.com/775abc2d7c903f191f7b24f8b299ebabbea3f624
> Cr-Commit-Position: refs/heads/master@{#438197}

TBR=esprehn@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=348655,674388

Committed: https://crrev.com/8885747f52b034575336f97e354fa8043f6c1b9f
Cr-Commit-Position: refs/heads/master@{#439722}
==========

[commit-bot: I haz the power, 2016-12-20 06:08:04]

Patchset 1 (id:??) landed as
https://crrev.com/8885747f52b034575336f97e354fa8043f6c1b9f
Cr-Commit-Position: refs/heads/master@{#439722}