Substitued pattern push_back(ptr.release()) with push_back(ptr.Pass()) in directory src/ui/

Substitued pattern push_back(ptr.release()) with push_back(ptr.Pass()) in directory.
src/ui/app_list, ui/ozone,ui/message_center

BUG=457697
Committed: https://crrev.com/8022c2fcb6edd10c1f38b8962ec398bc853bb4ae
Cr-Commit-Position: refs/heads/master@{#331069}

[d.samantaray, 2015-05-15 04:28:07]

d.samantaray@samsung.com changed reviewers:
+ gcasto@chromium.org, vabr@chromium.org

[d.samantaray, 2015-05-15 04:28:08]

Uploaded the patch.https://codereview.chromium.org/1123373007
Please review.

[vabr (Chromium), 2015-05-15 08:23:58]

Please choose appropriate OWNERS reviewers
(https://www.chromium.org/developers/owners-files).

[d.samantaray, 2015-05-15 08:48:00]

d.samantaray@samsung.com changed reviewers:
+ sadrul@chromium.org, sky@chromium.org
- gcasto@chromium.org, vabr@chromium.org

[d.samantaray, 2015-05-15 08:48:00]

I have uploaded the patch. Please review.
https://codereview.chromium.org/1123373007

[d.samantaray, 2015-05-21 10:41:35]

The CQ bit was checked by d.samantaray@samsung.com

[d.samantaray, 2015-05-21 10:41:36]

The CQ bit was unchecked by d.samantaray@samsung.com

[d.samantaray, 2015-05-21 11:58:20]

Patchset #2 (id:20001) has been deleted

[d.samantaray, 2015-05-21 12:22:37]

Incorporated the changes please review.

[sadrul, 2015-05-21 14:44:53]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2015-05-21 14:44:54]

lgtm

https://codereview.chromium.org/1123373007/diff/40001/ui/app_list/search_cont...
File ui/app_list/search_controller.cc (right):

https://codereview.chromium.org/1123373007/diff/40001/ui/app_list/search_cont...
ui/app_list/search_controller.cc:125: providers_.push_back(provider.Pass());  //
Takes ownership.
Remove the comment at the end. The new code is fairly clear that ownership is
being transferred.

[sadrul, 2015-05-21 14:44:58]

The CQ bit was unchecked by sadrul@chromium.org

[d.samantaray, 2015-05-22 03:37:02]

Addressed and incorporated the review comments. Please have a look.

https://codereview.chromium.org/1123373007/diff/40001/ui/app_list/search_cont...
File ui/app_list/search_controller.cc (right):

https://codereview.chromium.org/1123373007/diff/40001/ui/app_list/search_cont...
ui/app_list/search_controller.cc:125: providers_.push_back(provider.Pass());  //
Takes ownership.
On 2015/05/21 14:44:53, sadrul wrote:
> Remove the comment at the end. The new code is fairly clear that ownership is
> being transferred.

Done.

[d.samantaray, 2015-05-22 04:54:47]

The CQ bit was checked by d.samantaray@samsung.com

[d.samantaray, 2015-05-22 04:54:48]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/1123373007/#ps60001
(title: "Addressed the review comments")

[commit-bot: I haz the power, 2015-05-22 04:55:32]

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

[commit-bot: I haz the power, 2015-05-22 05:51:48]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-05-22 05:52:36]

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