ScopedPtrMap -> std::map from apps, ash, media, ui

ScopedPtrMap -> std::map from apps, ash, media, ui

C++ 11 enables containers that contain move-only type, scoped_ptr.
So, Use std::map<key, scoped_ptr<Foo>> instead of ScopedPtrMap.
Also use std::move() instead of scoped_ptr::Pass().

TBR=avi@chromium.org
BUG=554291, 557422
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/918cc2e94b4592f9211390784de4bb8534b59cc4
Cr-Commit-Position: refs/heads/master@{#360745}

[limasdf, 2015-11-19 01:58:34]

Patchset #1 (id:1) has been deleted

[limasdf, 2015-11-19 01:58:55]

Patchset #1 (id:20001) has been deleted

[limasdf, 2015-11-19 01:59:01]

Patchset #1 (id:40001) has been deleted

[limasdf, 2015-11-19 01:59:08]

Patchset #1 (id:60001) has been deleted

[limasdf, 2015-11-19 01:59:13]

Patchset #1 (id:80001) has been deleted

[limasdf, 2015-11-19 02:01:54]

Description was changed from

==========
[WIP] Remove ScopedPtrMap from /media

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
ScopedPtrMap -> std::map from apps, ash, media, ui

C++ 11 enables containers that contain move-only type, scoped_ptr.
So, Use std::map<key, scoped_ptr<Foo>> instead of ScopedPtrMap.
Also use std::move() instead of scoped_ptr::Pass().

BUG=554291, 557422
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[limasdf, 2015-11-19 02:01:55]

limasdf@gmail.com changed reviewers:
+ avi@chromium.org, benwells@chromium.org, oshima@chromium.org,
xhwang@chromium.org

[limasdf, 2015-11-19 02:02:23]

Please take a look. it's fairly mechanical.

[oshima, 2015-11-19 02:03:21]

ash lgtm

[xhwang, 2015-11-19 06:32:07]

media lgtm

[benwells, 2015-11-19 19:17:49]

lgtm

[limasdf, 2015-11-19 23:00:01]

Description was changed from

==========
ScopedPtrMap -> std::map from apps, ash, media, ui

C++ 11 enables containers that contain move-only type, scoped_ptr.
So, Use std::map<key, scoped_ptr<Foo>> instead of ScopedPtrMap.
Also use std::move() instead of scoped_ptr::Pass().

BUG=554291, 557422
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
ScopedPtrMap -> std::map from apps, ash, media, ui

C++ 11 enables containers that contain move-only type, scoped_ptr.
So, Use std::map<key, scoped_ptr<Foo>> instead of ScopedPtrMap.
Also use std::move() instead of scoped_ptr::Pass().

TBR=avi@chromium.org
BUG=554291, 557422
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[limasdf, 2015-11-19 23:00:07]

The CQ bit was checked by limasdf@gmail.com

[limasdf, 2015-11-19 23:00:07]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/1458553006/#ps120001
(title: "fix compile")

[commit-bot: I haz the power, 2015-11-19 23:05:01]

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

[Avi (use Gerrit), 2015-11-19 23:14:27]

lgtm

[commit-bot: I haz the power, 2015-11-19 23:30:49]

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

[commit-bot: I haz the power, 2015-11-19 23:30:50]

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

[limasdf, 2015-11-20 01:33:40]

The CQ bit was checked by limasdf@gmail.com

[commit-bot: I haz the power, 2015-11-20 01:39:30]

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

[commit-bot: I haz the power, 2015-11-20 01:59:54]

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

[commit-bot: I haz the power, 2015-11-20 01:59:55]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[limasdf, 2015-11-20 02:22:13]

The CQ bit was checked by limasdf@gmail.com

[limasdf, 2015-11-20 02:22:13]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org,
avi@chromium.org, benwells@chromium.org, xhwang@chromium.org
Link to the patchset: https://codereview.chromium.org/1458553006/#ps140001
(title: "Fix ash_unittests compile error")

[commit-bot: I haz the power, 2015-11-20 02:24:02]

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

[commit-bot: I haz the power, 2015-11-20 04:29:05]

Committed patchset #3 (id:140001)

[commit-bot: I haz the power, 2015-11-20 04:30:13]

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