Convert //cc from scoped_ptr to std::unique_ptr.

Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/60bc3bcfa8b7141986c5e4b3c357c7e03393f230
Cr-Commit-Position: refs/heads/master@{#386260}

[danakj, 2016-04-08 00:36:31]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
BUG=554298
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[danakj, 2016-04-08 00:37:17]

The CQ bit was checked by danakj@chromium.org

[vmpstr, 2016-04-08 00:53:55]

lgtm, thanks (I looked at animation and spot checked other places).

https://codereview.chromium.org/1866203004/diff/1/cc/animation/element_animat...
File cc/animation/element_animations.cc (right):

https://codereview.chromium.org/1866203004/diff/1/cc/animation/element_animat...
cc/animation/element_animations.cc:66: : players_list_(base::WrapUnique(new
PlayersList())),
I don't think you need this?

[danakj, 2016-04-08 00:58:47]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 00:59:01]

https://codereview.chromium.org/1866203004/diff/1/cc/animation/element_animat...
File cc/animation/element_animations.cc (right):

https://codereview.chromium.org/1866203004/diff/1/cc/animation/element_animat...
cc/animation/element_animations.cc:66: : players_list_(base::WrapUnique(new
PlayersList())),
On 2016/04/08 00:53:55, vmpstr wrote:
> I don't think you need this?

Done.

[commit-bot: I haz the power, 2016-04-08 00:59:16]

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

[commit-bot: I haz the power, 2016-04-08 01:07:22]

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

[commit-bot: I haz the power, 2016-04-08 01:07:25]

Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[danakj, 2016-04-08 18:25:54]

danakj@chromium.org changed reviewers:
+ piman@chromium.org

[danakj, 2016-04-08 18:25:55]

+piman TBR for content and ui/compositor IWYU for scoped_ptr.h

[danakj, 2016-04-08 18:26:43]

On 2016/04/08 18:25:55, danakj wrote:
> +piman TBR for content and ui/compositor IWYU for scoped_ptr.h

+sky TBR for ui/

[danakj, 2016-04-08 18:27:04]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[danakj, 2016-04-08 18:27:04]

danakj@chromium.org changed reviewers:
+ sky@chromium.org

[danakj, 2016-04-08 18:28:28]

On 2016/04/08 18:26:43, danakj wrote:
> On 2016/04/08 18:25:55, danakj wrote:
> > +piman TBR for content and ui/compositor IWYU for scoped_ptr.h
> 
> +sky TBR for ui/

^

[danakj, 2016-04-08 18:28:33]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 18:28:35]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps40001
(title: "scopedptrcc: iwyu")

[commit-bot: I haz the power, 2016-04-08 18:29:30]

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

[commit-bot: I haz the power, 2016-04-08 18:47:43]

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

[commit-bot: I haz the power, 2016-04-08 18:47:46]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[danakj, 2016-04-08 20:10:12]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 20:10:14]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps60001
(title: "scopedptrcc: iwyu-android")

[commit-bot: I haz the power, 2016-04-08 20:10:35]

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

[commit-bot: I haz the power, 2016-04-08 20:17:03]

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

[commit-bot: I haz the power, 2016-04-08 20:17:06]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  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_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[danakj, 2016-04-08 20:33:37]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[danakj, 2016-04-08 20:33:37]

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

[danakj, 2016-04-08 20:46:47]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
==========

[danakj, 2016-04-08 20:47:06]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 20:47:06]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2016-04-08 20:47:08]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps70001
(title: "scopedptrcc: rebase")

[commit-bot: I haz the power, 2016-04-08 20:47:38]

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

[danakj, 2016-04-08 21:05:46]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 21:05:48]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps90001
(title: "scopedptrcc: .")

[danakj, 2016-04-08 21:05:48]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-04-08 21:06:31]

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

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

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

[commit-bot: I haz the power, 2016-04-08 21:28:17]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[danakj, 2016-04-08 21:40:47]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 21:40:49]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps100001
(title: "scopedptrcc: moreiwyu")

[commit-bot: I haz the power, 2016-04-08 21:41:10]

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

[commit-bot: I haz the power, 2016-04-08 22:00:51]

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

[commit-bot: I haz the power, 2016-04-08 22:00:54]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[danakj, 2016-04-08 22:08:26]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 22:08:28]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps110001
(title: "scopedptrcc: .")

[commit-bot: I haz the power, 2016-04-08 22:08:47]

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

[commit-bot: I haz the power, 2016-04-08 22:15:55]

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

[commit-bot: I haz the power, 2016-04-08 22:15:59]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[danakj, 2016-04-08 23:06:39]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-04-08 23:06:41]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1866203004/#ps120001
(title: "scopedptrcc: rebase")

[commit-bot: I haz the power, 2016-04-08 23:07:07]

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

[commit-bot: I haz the power, 2016-04-09 00:25:03]

Committed patchset #13 (id:120001)

[commit-bot: I haz the power, 2016-04-09 00:26:44]

Description was changed from

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Convert //cc from scoped_ptr to std::unique_ptr.

R=enne, vmpstr
TBR=piman,sky, skyostil
BUG=554298
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/60bc3bcfa8b7141986c5e4b3c357c7e03393f230
Cr-Commit-Position: refs/heads/master@{#386260}
==========

[commit-bot: I haz the power, 2016-04-09 00:26:47]

Patchset 13 (id:??) landed as
https://crrev.com/60bc3bcfa8b7141986c5e4b3c357c7e03393f230
Cr-Commit-Position: refs/heads/master@{#386260}