net: move GetIdentifyFromURL function into url_util.h

net: move GetIdentifyFromURL function into url_util.h

This seems to be a more appropriate place for this function,
than the generic net_util.h header file, which will soon be gone.

BUG=488531
TEST=net_unittests
R=eroman@chromium.org

Committed: https://crrev.com/c38cb957636e2cdada1a4d2ac509fc0a3f54590d
Cr-Commit-Position: refs/heads/master@{#369403}

[eroman, 2016-01-13 20:21:02]

lgtm!

https://codereview.chromium.org/1582083002/diff/1/net/base/net_util.h
File net/base/net_util.h (right):

https://codereview.chromium.org/1582083002/diff/1/net/base/net_util.h#newcode63
net/base/net_util.h:63: NET_EXPORT std::string GetHostOrSpecFromURL(const GURL&
url);
This is a good candidate for url_util too

https://codereview.chromium.org/1582083002/diff/1/net/base/net_util.h#newcode85
net/base/net_util.h:85: NET_EXPORT GURL SimplifyUrlForRequest(const GURL& url);
And this

https://codereview.chromium.org/1582083002/diff/1/net/base/net_util.h#newcode106
net/base/net_util.h:106: NET_EXPORT_PRIVATE bool HasGoogleHost(const GURL& url);
And this. Basically anything which takes a GURL input I think.

[tfarina, 2016-01-13 22:58:49]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2016-01-13 23:01:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1582083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1582083002/1

[commit-bot: I haz the power, 2016-01-13 23:39:18]

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

[commit-bot: I haz the power, 2016-01-13 23:39:18]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[tfarina, 2016-01-13 23:42:29]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2016-01-13 23:43:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1582083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1582083002/1

[commit-bot: I haz the power, 2016-01-14 01:10:01]

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

[commit-bot: I haz the power, 2016-01-14 01:10:02]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)

[tfarina, 2016-01-14 01:11:10]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2016-01-14 01:16:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1582083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1582083002/1

[commit-bot: I haz the power, 2016-01-14 01:53:09]

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

[commit-bot: I haz the power, 2016-01-14 01:53:10]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[tfarina, 2016-01-14 12:40:49]

The CQ bit was checked by tfarina@chromium.org

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

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1582083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1582083002/1

[commit-bot: I haz the power, 2016-01-14 12:45:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-14 12:46:03]

Description was changed from

==========
net: move GetIdentifyFromURL function into url_util.h

This seems to be a more appropriate place for this function,
than the generic net_util.h header file, which will soon be gone.

BUG=488531
TEST=net_unittests
R=eroman@chromium.org
==========

to

==========
net: move GetIdentifyFromURL function into url_util.h

This seems to be a more appropriate place for this function,
than the generic net_util.h header file, which will soon be gone.

BUG=488531
TEST=net_unittests
R=eroman@chromium.org

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

[commit-bot: I haz the power, 2016-01-14 12:46:04]

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