Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

Committed: https://crrev.com/5e45bd7cbb42f0c5251b140e374ce9589a00a110
Cr-Commit-Position: refs/heads/master@{#387200}

[Matt Giuca, 2016-04-04 01:21:33]

Patchset #4 (id:60001) has been deleted

[Matt Giuca, 2016-04-11 01:45:04]

Description was changed from

==========
Border::CreateBorderPainter now takes a scoped_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.
==========

to

==========
Border::CreateBorderPainter now takes a scoped_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

[Matt Giuca, 2016-04-11 01:55:42]

Description was changed from

==========
Border::CreateBorderPainter now takes a scoped_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

to

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

[Matt Giuca, 2016-04-11 01:55:56]

Description was changed from

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

to

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

[Matt Giuca, 2016-04-11 01:57:02]

This is definitely going to conflict with a CL that I suspect is coming in the
next few days to convert ui/views to unique_ptr, so I will wait until that lands
to avoid conflicting with it.

Also it is a bit weird adding unique_ptr into a class that already uses
scoped_ptr everywhere else. It will make more sense after it is converted.

[Matt Giuca, 2016-04-13 06:53:38]

mgiuca@chromium.org changed reviewers:
+ pkasting@chromium.org, sadrul@chromium.org

[Matt Giuca, 2016-04-13 06:53:38]

sadrul: Please review changes in ui/views.
pkasting: TBR for dropdown_bar_view mechanical change.

[sadrul, 2016-04-13 15:21:14]

cool. lgtm

[Peter Kasting, 2016-04-13 19:59:42]

LGTM

[Matt Giuca, 2016-04-14 00:14:10]

Description was changed from

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

TBR=pkasting@chromium.org
==========

to

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.
==========

[Matt Giuca, 2016-04-14 00:43:27]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-04-14 00:43:54]

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

[commit-bot: I haz the power, 2016-04-14 01:36:21]

Description was changed from

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.
==========

to

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.
==========

[commit-bot: I haz the power, 2016-04-14 01:36:22]

Committed patchset #7 (id:140001)

[commit-bot: I haz the power, 2016-04-14 01:38:35]

Description was changed from

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.
==========

to

==========
Border::CreateBorderPainter now takes a unique_ptr, not a raw pointer.

Now the ownership is enforced by the type, rather than having to be
explained in a comment.

Committed: https://crrev.com/5e45bd7cbb42f0c5251b140e374ce9589a00a110
Cr-Commit-Position: refs/heads/master@{#387200}
==========

[commit-bot: I haz the power, 2016-04-14 01:38:36]

Patchset 7 (id:??) landed as
https://crrev.com/5e45bd7cbb42f0c5251b140e374ce9589a00a110
Cr-Commit-Position: refs/heads/master@{#387200}