DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true

Committed: https://crrev.com/229b1f8945646f00a834ec88bc97fe738561ff0a
Cr-Commit-Position: refs/heads/master@{#395765}

[pfeldman, 2016-05-24 18:20:44]

pfeldman@chromium.org changed reviewers:
+ dgozman@chromium.org

[pfeldman, 2016-05-24 18:24:40]

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

[pfeldman, 2016-05-24 18:25:13]

The CQ bit was unchecked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-05-24 18:25:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2004313003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2004313003/20001

[pfeldman, 2016-05-24 18:52:46]

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

[commit-bot: I haz the power, 2016-05-24 18:53:27]

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

[dgozman, 2016-05-24 19:10:46]

lgtm

[haraken, 2016-05-24 19:40:17]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org, yutak@chromium.org

[haraken, 2016-05-24 19:40:18]

+yutak FYI

[pfeldman, 2016-05-24 19:55:22]

On 2016/05/24 19:40:18, haraken wrote:
> +yutak FYI

thanks. we briefly discussed it with yutak.

[commit-bot: I haz the power, 2016-05-24 20:02:36]

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

[commit-bot: I haz the power, 2016-05-24 20:02:37]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[pfeldman, 2016-05-24 21:38:09]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-05-24 21:38:42]

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

[commit-bot: I haz the power, 2016-05-24 22:33:14]

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

[commit-bot: I haz the power, 2016-05-24 22:33:16]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[pfeldman, 2016-05-24 22:39:58]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-05-24 22:39:59]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2004313003/#ps60001
(title: "rebaselined")

[commit-bot: I haz the power, 2016-05-24 22:40:49]

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

[commit-bot: I haz the power, 2016-05-25 00:58:08]

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

[commit-bot: I haz the power, 2016-05-25 00:58:10]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[pfeldman, 2016-05-25 01:11:42]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-05-25 01:12:10]

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

[pfeldman, 2016-05-25 01:14:17]

Description was changed from

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
==========

to

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true
==========

[pfeldman, 2016-05-25 01:14:24]

The CQ bit was unchecked by pfeldman@chromium.org

[pfeldman, 2016-05-25 01:14:26]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-05-25 01:14:55]

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

[pfeldman, 2016-05-25 01:18:56]

The CQ bit was unchecked by pfeldman@chromium.org

[pfeldman, 2016-05-25 01:19:09]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-05-25 01:19:53]

Description was changed from

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true
==========

to

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true
==========

[commit-bot: I haz the power, 2016-05-25 01:19:55]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-25 01:21:28]

Description was changed from

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true
==========

to

==========
DevTools: migrate from OwnPtr to std::unique_ptr for inspector protocol classes.

BUG=580337
NOTRY=true

Committed: https://crrev.com/229b1f8945646f00a834ec88bc97fe738561ff0a
Cr-Commit-Position: refs/heads/master@{#395765}
==========

[commit-bot: I haz the power, 2016-05-25 01:21:30]

Patchset 4 (id:??) landed as
https://crrev.com/229b1f8945646f00a834ec88bc97fe738561ff0a
Cr-Commit-Position: refs/heads/master@{#395765}

[Yuta Kitamura, 2016-05-25 03:55:23]

This is great!