media: Use stl utilities from the base namespace

media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org

Committed: https://crrev.com/03a07cb6e22221efeadb1212418eaee92976a3da
Cr-Commit-Position: refs/heads/master@{#411662}

[Sami, 2016-08-10 10:29:30]

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

[commit-bot: I haz the power, 2016-08-10 10:29:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sami, 2016-08-10 13:32:05]

Description was changed from

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
==========

to

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
==========

[Sami, 2016-08-10 13:32:06]

skyostil@chromium.org changed reviewers:
+ chcunningham@chromium.org

[Sami, 2016-08-10 13:32:21]

PTAL.

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

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

[commit-bot: I haz the power, 2016-08-10 14:30:31]

Dry run: This issue passed the CQ dry run.

[Sami, 2016-08-10 16:16:46]

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

[commit-bot: I haz the power, 2016-08-10 16:17:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chcunningham, 2016-08-10 18:41:31]

On 2016/08/10 13:32:21, Sami wrote:
> PTAL.

How are we currently supporting both base:: and non base:: calls? For instance,
I find that https://cs.chromium.org/chromium/src/base/stl_util.h?rcl=0&l=40
STLDeleteContainerPoiters is only in the global namesapce... I'm sure I'm
missing something

[commit-bot: I haz the power, 2016-08-10 19:58:56]

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

[commit-bot: I haz the power, 2016-08-10 19:58:57]

Dry run: This issue passed the CQ dry run.

[Sami, 2016-08-11 09:13:44]

On 2016/08/10 18:41:31, chcunningham wrote:
> On 2016/08/10 13:32:21, Sami wrote:
> > PTAL.
> 
> How are we currently supporting both base:: and non base:: calls? For
instance,
> I find that https://cs.chromium.org/chromium/src/base/stl_util.h?rcl=0&l=40
> STLDeleteContainerPoiters is only in the global namesapce... I'm sure I'm
> missing something

Ah, looks like code search hasn't updated yet. We've got these global aliases in
place temporarily:

https://chromium.googlesource.com/chromium/src/+/master/base/stl_util.h#262

I'll remove them as soon as all the call sites have been fixed.

[chcunningham1, 2016-08-11 17:10:36]

chcunningham@google.com changed reviewers:
+ chcunningham@google.com

[chcunningham1, 2016-08-11 17:10:37]

lgtm

[Sami, 2016-08-12 10:55:34]

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

[commit-bot: I haz the power, 2016-08-12 10:56:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sami, 2016-08-12 12:27:53]

The CQ bit was unchecked by skyostil@chromium.org

[Sami, 2016-08-12 12:27:58]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2016-08-12 12:27:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
chcunningham@google.com
Link to the patchset: https://codereview.chromium.org/2234753002/#ps40001
(title: "Rebased")

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

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-08-12 12:31:57]

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

[commit-bot: I haz the power, 2016-08-12 12:31:58]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Sami, 2016-08-12 14:35:52]

Description was changed from

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
==========

to

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org
==========

[Sami, 2016-08-12 14:36:44]

Looks like the presubmit freaked out since your lg2m came from @google.com
instead of @chromium.org. I'm assuming this is still fine to land, so TBR :)

[Sami, 2016-08-12 14:36:53]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2016-08-12 14:37:02]

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-08-12 15:51:01]

Description was changed from

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org
==========

to

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org
==========

[commit-bot: I haz the power, 2016-08-12 15:51:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-12 15:52:44]

Description was changed from

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org
==========

to

==========
media: Use stl utilities from the base namespace

The utilities in base/stl_util.h have been moved from the global
into the base namespace. This patch updates the call sites accordingly.

No functional changes.

BUG=636301
TBR=chcunningham@chromium.org

Committed: https://crrev.com/03a07cb6e22221efeadb1212418eaee92976a3da
Cr-Commit-Position: refs/heads/master@{#411662}
==========

[commit-bot: I haz the power, 2016-08-12 15:52:45]

Patchset 3 (id:??) landed as
https://crrev.com/03a07cb6e22221efeadb1212418eaee92976a3da
Cr-Commit-Position: refs/heads/master@{#411662}