net::SimplifyUrlForRequest fast path for URLs without {user,pass,ref} parts.

net::SimplifyUrlForRequest fast path for URLs without {user,pass,ref} parts.

This function uses GURL::ReplaceComponents which triggers
canonicalization of the new URL. For the normal case where urls do not
have usernames, passwords, or ref parts, we can just return the original
URL.

ReplaceComponents takes around 13% of the time spent in
HttpCache::Transaction::Start with a SimpleCache backend on Linux, but
this method is used in a few different places.

BUG=664174
Committed: https://crrev.com/16060210ca945bcd601e380617d75f916a084932
Cr-Commit-Position: refs/heads/master@{#434756}

[Charlie Harrison, 2016-11-15 20:43:08]

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

[commit-bot: I haz the power, 2016-11-15 20:43:48]

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-11-15 20:51:57]

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

[Charlie Harrison, 2016-11-15 20:51:57]

gavinp@ ptal?

[commit-bot: I haz the power, 2016-11-15 20:58:58]

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

[commit-bot: I haz the power, 2016-11-15 20:58:59]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[gavinp, 2016-11-28 17:51:45]

lgtm. profiler for the win!

[Charlie Harrison, 2016-11-28 17:52:09]

Thanks Gavin!

[Charlie Harrison, 2016-11-28 17:52:16]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-11-28 17:52:37]

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-11-28 22:52:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480355536286570, "parent_rev":
"bd983934046837009bef85d14342d97c09b116d2", "commit_rev":
"ea6604f06291519daf73eb41119ecc4b6a112773"}

[commit-bot: I haz the power, 2016-11-28 22:53:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-28 22:55:25]

Description was changed from

==========
net::SimplifyUrlForRequest fast path for URLs without {user,pass,ref} parts.

This function uses GURL::ReplaceComponents which triggers
canonicalization of the new URL. For the normal case where urls do not
have usernames, passwords, or ref parts, we can just return the original
URL.

ReplaceComponents takes around 13% of the time spent in
HttpCache::Transaction::Start with a SimpleCache backend on Linux, but
this method is used in a few different places.

BUG=664174
==========

to

==========
net::SimplifyUrlForRequest fast path for URLs without {user,pass,ref} parts.

This function uses GURL::ReplaceComponents which triggers
canonicalization of the new URL. For the normal case where urls do not
have usernames, passwords, or ref parts, we can just return the original
URL.

ReplaceComponents takes around 13% of the time spent in
HttpCache::Transaction::Start with a SimpleCache backend on Linux, but
this method is used in a few different places.

BUG=664174

Committed: https://crrev.com/16060210ca945bcd601e380617d75f916a084932
Cr-Commit-Position: refs/heads/master@{#434756}
==========

[commit-bot: I haz the power, 2016-11-28 22:55:26]

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