Revert of Convert //net and //chromecast to std::unordered_*

Revert of Convert //net and //chromecast to std::unordered_* (patchset #5 id:80001 of https://codereview.chromium.org/1869503003/ )

Reason for revert:
Speculatively reverting this since it was in the blamelist https://chromium.googlesource.com/chromium/src/+log/39399be2c6a0063520d8d9b289e8f7b31e1eb094%5E..4241ad1dee5396c329a5e9379c3db3027f1b6456?pretty=fuller that I came up with after analyzing the content_browsertests failures (DownloadContentTest) on
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/27572 and https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/11573

Original issue's description:
> Convert //net and //chromecast to std::unordered_*
>
> Do //chromecast at the same time as //net since there aren't many
> uses in //chromecast and one needs to be changed to mirror a //net
> API change.
>
> BUG=576864, 554287
>
> Committed: https://crrev.com/4241ad1dee5396c329a5e9379c3db3027f1b6456
> Cr-Commit-Position: refs/heads/master@{#386485}

TBR=mmenke@chromium.org,lcwu@chromium.org,davidben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864, 554287
Committed: https://crrev.com/31ddcd5dea3d3dcdf9e55ba3213d84d1e904aa27
Cr-Commit-Position: refs/heads/master@{#386624}

[kjellander_chromium, 2016-04-12 08:14:16]

Created Revert of Convert //net and //chromecast to std::unordered_*

[commit-bot: I haz the power, 2016-04-12 08:14:27]

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

[commit-bot: I haz the power, 2016-04-12 08:19:32]

Description was changed from

==========
Revert of Convert //net and //chromecast to std::unordered_* (patchset #5
id:80001 of https://codereview.chromium.org/1869503003/ )

Reason for revert:
Speculatively reverting this since it was in the blamelist
https://chromium.googlesource.com/chromium/src/+log/39399be2c6a0063520d8d9b28...
that I came up with after analyzing the content_browsertests failures
(DownloadContentTest) on 
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...
and
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> Convert //net and //chromecast to std::unordered_*
> 
> Do //chromecast at the same time as //net since there aren't many
> uses in //chromecast and one needs to be changed to mirror a //net
> API change.
> 
> BUG=576864,554287
> 
> Committed: https://crrev.com/4241ad1dee5396c329a5e9379c3db3027f1b6456
> Cr-Commit-Position: refs/heads/master@{#386485}

TBR=mmenke@chromium.org,lcwu@chromium.org,davidben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864,554287
==========

to

==========
Revert of Convert //net and //chromecast to std::unordered_* (patchset #5
id:80001 of https://codereview.chromium.org/1869503003/ )

Reason for revert:
Speculatively reverting this since it was in the blamelist
https://chromium.googlesource.com/chromium/src/+log/39399be2c6a0063520d8d9b28...
that I came up with after analyzing the content_browsertests failures
(DownloadContentTest) on 
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...
and
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> Convert //net and //chromecast to std::unordered_*
> 
> Do //chromecast at the same time as //net since there aren't many
> uses in //chromecast and one needs to be changed to mirror a //net
> API change.
> 
> BUG=576864,554287
> 
> Committed: https://crrev.com/4241ad1dee5396c329a5e9379c3db3027f1b6456
> Cr-Commit-Position: refs/heads/master@{#386485}

TBR=mmenke@chromium.org,lcwu@chromium.org,davidben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864,554287
==========

[commit-bot: I haz the power, 2016-04-12 08:19:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-12 08:20:36]

Description was changed from

==========
Revert of Convert //net and //chromecast to std::unordered_* (patchset #5
id:80001 of https://codereview.chromium.org/1869503003/ )

Reason for revert:
Speculatively reverting this since it was in the blamelist
https://chromium.googlesource.com/chromium/src/+log/39399be2c6a0063520d8d9b28...
that I came up with after analyzing the content_browsertests failures
(DownloadContentTest) on 
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...
and
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> Convert //net and //chromecast to std::unordered_*
> 
> Do //chromecast at the same time as //net since there aren't many
> uses in //chromecast and one needs to be changed to mirror a //net
> API change.
> 
> BUG=576864,554287
> 
> Committed: https://crrev.com/4241ad1dee5396c329a5e9379c3db3027f1b6456
> Cr-Commit-Position: refs/heads/master@{#386485}

TBR=mmenke@chromium.org,lcwu@chromium.org,davidben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864,554287
==========

to

==========
Revert of Convert //net and //chromecast to std::unordered_* (patchset #5
id:80001 of https://codereview.chromium.org/1869503003/ )

Reason for revert:
Speculatively reverting this since it was in the blamelist
https://chromium.googlesource.com/chromium/src/+log/39399be2c6a0063520d8d9b28...
that I came up with after analyzing the content_browsertests failures
(DownloadContentTest) on
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...
and
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...

Original issue's description:
> Convert //net and //chromecast to std::unordered_*
>
> Do //chromecast at the same time as //net since there aren't many
> uses in //chromecast and one needs to be changed to mirror a //net
> API change.
>
> BUG=576864,554287
>
> Committed: https://crrev.com/4241ad1dee5396c329a5e9379c3db3027f1b6456
> Cr-Commit-Position: refs/heads/master@{#386485}

TBR=mmenke@chromium.org,lcwu@chromium.org,davidben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864,554287

Committed: https://crrev.com/31ddcd5dea3d3dcdf9e55ba3213d84d1e904aa27
Cr-Commit-Position: refs/heads/master@{#386624}
==========

[commit-bot: I haz the power, 2016-04-12 08:20:37]

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

[davidben, 2016-04-12 14:58:37]

On 2016/04/12 08:20:37, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/31ddcd5dea3d3dcdf9e55ba3213d84d1e904aa27
> Cr-Commit-Position: refs/heads/master@{#386624}

Did this end up resolving it?

[mmenke, 2016-04-12 15:03:17]

On 2016/04/12 14:58:37, davidben (OOO 4-4 to 4-11) wrote:
> On 2016/04/12 08:20:37, commit-bot: I haz the power wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/31ddcd5dea3d3dcdf9e55ba3213d84d1e904aa27
> > Cr-Commit-Position: refs/heads/master@{#386624}
> 
> Did this end up resolving it?

Looks like it did -
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2....
 First one that didn't fail had the revert (Among other things, including a
revert of another CL for a different breakage) -
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

[kjellander_chromium, 2016-04-12 15:10:51]

On 2016/04/12 15:03:17, mmenke wrote:
> On 2016/04/12 14:58:37, davidben (OOO 4-4 to 4-11) wrote:
> > On 2016/04/12 08:20:37, commit-bot: I haz the power wrote:
> > > Patchset 1 (id:??) landed as
> > > https://crrev.com/31ddcd5dea3d3dcdf9e55ba3213d84d1e904aa27
> > > Cr-Commit-Position: refs/heads/master@{#386624}
> > 
> > Did this end up resolving it?
> 
> Looks like it did -
>
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2....
>  First one that didn't fail had the revert (Among other things, including a
> revert of another CL for a different breakage) -
>
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

Yeah, I was lucky here. I had a really hard time picking a CL to revert but I
picked this because of the changes in net/url_request/url_request_filter.{cc,h}
that seemed to show up in some of the ASan/LSan stack traces. I can't explain
why an I also couldn't repro locally unfortunately.