Removed some URL Utility functions & improved unit tests

Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the
functions that remain.

Committed: https://crrev.com/693f71d7a044b4f2464bf09f4d6e44b77ca1aecc
Cr-Commit-Position: refs/heads/master@{#426511}

[pkl (ping after 24h if needed), 2016-09-26 15:23:00]

pkl@chromium.org changed reviewers:
+ rohitrao@chromium.org

[pkl (ping after 24h if needed), 2016-09-27 19:45:09]

ping

[rohitrao (ping after 24h), 2016-09-27 19:46:41]

This CL looks like it's doing more than just deleting unused methods.  Can you
provide some more detail and update the CL description if necessary?

[pkl (ping after 24h if needed), 2016-09-27 20:53:25]

Description was changed from

==========
Removed some URL Utility functions that are no longer used.
==========

to

==========
Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the 
functions that remain.
==========

[pkl (ping after 24h if needed), 2016-09-27 20:53:37]

On 2016/09/27 19:46:41, rohitrao wrote:
> This CL looks like it's doing more than just deleting unused methods.  Can you
> provide some more detail and update the CL description if necessary?

Updated description.

[pkl (ping after 24h if needed), 2016-10-07 22:05:11]

ping

[pkl (ping after 24h if needed), 2016-10-18 14:13:32]

pinging again... please review.

[rohitrao (ping after 24h), 2016-10-20 16:32:50]

lgtm

[pkl (ping after 24h if needed), 2016-10-20 16:38:20]

The CQ bit was checked by pkl@chromium.org

[commit-bot: I haz the power, 2016-10-20 16:38:54]

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-10-20 17:12:50]

Description was changed from

==========
Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the 
functions that remain.
==========

to

==========
Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the 
functions that remain.
==========

[commit-bot: I haz the power, 2016-10-20 17:12:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 13:19:23]

Description was changed from

==========
Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the 
functions that remain.
==========

to

==========
Removed some URL Utility functions & improved unit tests

The removed utility functions are no longer used.
The improved unit tests give better coverage to the
functions that remain.

Committed: https://crrev.com/693f71d7a044b4f2464bf09f4d6e44b77ca1aecc
Cr-Commit-Position: refs/heads/master@{#426511}
==========

[commit-bot: I haz the power, 2016-10-21 13:19:24]

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