[url] Early return for finding query/ref parts in path parsing

[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978
Committed: https://crrev.com/ad24ce2f18b3de5e33955675eb4710a6d722236f
Cr-Commit-Position: refs/heads/master@{#436593}

[Charlie Harrison, 2016-12-01 21:00:20]

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

[commit-bot: I haz the power, 2016-12-01 21:01:09]

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

[Charlie Harrison, 2016-12-01 21:09:46]

Description was changed from

==========
[url] Early return for finding query/ref parts in path parsing

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

to

==========
[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978
==========

[Charlie Harrison, 2016-12-01 21:52:08]

csharrison@chromium.org changed reviewers:
+ mkwst@chromium.org

[Charlie Harrison, 2016-12-01 21:52:08]

Hey Mike, here's another optimization I noticed. PTAL? I'm not sure how common
huge ref urls are but we might as well make sure they're fast!

[Mike West, 2016-12-06 13:45:06]

LGTM, sorry for the delayed response!

[Charlie Harrison, 2016-12-06 13:46:38]

The CQ bit was checked by csharrison@chromium.org

[Charlie Harrison, 2016-12-06 13:46:50]

Thanks, no worries!

[commit-bot: I haz the power, 2016-12-06 13:46:55]

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, 2016-12-06 14:37:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481031998538180,
"parent_rev": "685ff4c9efee039be8d2968ff772fa2987bd475b", "commit_rev":
"eab6b3fab249fe3d66ba6f91ed735ec1c563b96d"}

[commit-bot: I haz the power, 2016-12-06 14:37:51]

Description was changed from

==========
[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978
==========

to

==========
[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978
==========

[commit-bot: I haz the power, 2016-12-06 14:37:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-06 14:41:07]

Description was changed from

==========
[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978
==========

to

==========
[url] Early return for finding query/ref parts in path parsing

For URLs with huge ref parts (~.5mb), this saves ~18% off of
url::ResolveRelative and ~10% off of KURL::init.

BUG=657978

Committed: https://crrev.com/ad24ce2f18b3de5e33955675eb4710a6d722236f
Cr-Commit-Position: refs/heads/master@{#436593}
==========

[commit-bot: I haz the power, 2016-12-06 14:41:08]

Patchset 2 (id:??) landed as
https://crrev.com/ad24ce2f18b3de5e33955675eb4710a6d722236f
Cr-Commit-Position: refs/heads/master@{#436593}