[url] Remove comparisons for blob/filesystem schemes in scheme_host_port

Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=672877
Review-Url: https://codereview.chromium.org/2560923002
Cr-Commit-Position: refs/heads/master@{#442372}
Committed: https://chromium.googlesource.com/chromium/src/+/39a260f9aff94ecd6d518778e2ea900f58933636

[Charlie Harrison, 2016-12-07 19:24:21]

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

[commit-bot: I haz the power, 2016-12-07 19:24: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, 2016-12-07 20:57:11]

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

[commit-bot: I haz the power, 2016-12-07 20:57:12]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2016-12-07 21:00:21]

Description was changed from

==========
Move comparisons to DCHECKs for blob/filesystem schemes in scheme_host_port

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

to

==========
Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=348655
==========

[Charlie Harrison, 2016-12-07 21:07:13]

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

[Charlie Harrison, 2016-12-07 21:07:14]

Brett, WDYT? Is it okay to punt this policy to the existing implementation? To
me, this seems safe and makes the code more efficient and simpler (fewer special
cases). Additionally, we have unit test coverage to ensure this won't break.

For context: Origin construction is showing up a fair amount in profiles in the
renderer process. A big chunk of that time is spent in IsStandard, which is
mostly this operator== and GetStandardSchemeType (which has room for
improvement).

[Charlie Harrison, 2016-12-09 19:06:51]

Description was changed from

==========
Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=348655
==========

to

==========
Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=672877
==========

[Charlie Harrison, 2017-01-03 23:43:46]

brettw: friendly ping

[brettw, 2017-01-09 20:23:28]

lgtm

[Charlie Harrison, 2017-01-09 20:23:53]

Thanks!

[Charlie Harrison, 2017-01-09 20:23:56]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2017-01-09 20:24:24]

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-01-09 22:24:20]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483993436949410,
"parent_rev": "8ce922854237356ea07d4a5a67aa6d8289470f36", "commit_rev":
"39a260f9aff94ecd6d518778e2ea900f58933636"}

[commit-bot: I haz the power, 2017-01-09 22:24:54]

Description was changed from

==========
Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=672877
==========

to

==========
Remove comparisons for blob/filesystem schemes in scheme_host_port

These are checked implicitly via the fact that blob schemes are not
standard, and filesystem schemes have type SCHEME_WITHOUT_AUTHORITY.

IsStandard is in critical path of loading many resources, as it is
used for conversions to url::Origin.

BUG=672877

Review-Url: https://codereview.chromium.org/2560923002
Cr-Commit-Position: refs/heads/master@{#442372}
Committed:
https://chromium.googlesource.com/chromium/src/+/39a260f9aff94ecd6d518778e2ea...
==========

[commit-bot: I haz the power, 2017-01-09 22:24:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/39a260f9aff94ecd6d518778e2ea...