Remove OwnPtr::release().

Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
Committed: https://crrev.com/0030f9c2a1dec3f165d6a23aff64292fec2067f1
Cr-Commit-Position: refs/heads/master@{#394722}

[Yuta Kitamura, 2016-05-18 07:46:07]

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

[commit-bot: I haz the power, 2016-05-18 07:46:24]

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

[Yuta Kitamura, 2016-05-18 07:46:41]

Description was changed from

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
==========

to

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
==========

[Yuta Kitamura, 2016-05-18 07:46:41]

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

[Yuta Kitamura, 2016-05-18 07:46:42]

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

[Yuta Kitamura, 2016-05-18 07:48:24]

Note: I sent a PSA to blink-dev, and will wait for feedbacks
until tomorrow before actually landing this.

tzik: PTAL?
+haraken for modules/ approval.

[haraken, 2016-05-18 07:49:43]

LGTM assuming there is no objection on the PSA.

[tzik, 2016-05-18 07:54:34]

lgtm

[commit-bot: I haz the power, 2016-05-18 08:13:18]

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

[commit-bot: I haz the power, 2016-05-18 08:13:18]

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

[Yuta Kitamura, 2016-05-18 09:42:34]

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

[commit-bot: I haz the power, 2016-05-18 09:42:54]

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

[commit-bot: I haz the power, 2016-05-18 10:57:23]

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

[commit-bot: I haz the power, 2016-05-18 10:57:24]

Dry run: This issue passed the CQ dry run.

[Yuta Kitamura, 2016-05-19 07:53:20]

The CQ bit was checked by yutak@chromium.org

[Yuta Kitamura, 2016-05-19 07:53:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, tzik@chromium.org
Link to the patchset: https://codereview.chromium.org/1987203002/#ps40001
(title: "Merge with trunk for landing.")

[commit-bot: I haz the power, 2016-05-19 07:53:43]

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

[commit-bot: I haz the power, 2016-05-19 09:15:25]

Description was changed from

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
==========

to

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
==========

[commit-bot: I haz the power, 2016-05-19 09:15:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-19 09:16:30]

Description was changed from

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660
==========

to

==========
Remove OwnPtr::release().

Having OwnPtr::release() can be confusing when we convert OwnPtrs to
std::unique_ptrs, because std::unique_ptr also has release() member
function with a different meaning.

Existing uses of OwnPtr::release() are already mostly removed; this
patch takes care of release() calls from recent changes, and actually
removes the method.

BUG=611660

Committed: https://crrev.com/0030f9c2a1dec3f165d6a23aff64292fec2067f1
Cr-Commit-Position: refs/heads/master@{#394722}
==========

[commit-bot: I haz the power, 2016-05-19 09:16:32]

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