Fix unique_ptr usage in handling OAuth2TokenService::StartRequest result

Fix unique_ptr usage in handling OAuth2TokenService::StartRequest result

Calling release() on a temporary unique_ptr to reset another unique_ptr with
resulting raw pointer is verbose and obscure. It also loses custom deleter
(see C++11 standard, clause 20.7.1.2.3.1).

BUG=none

Review-Url: https://codereview.chromium.org/2755403002
Cr-Commit-Position: refs/heads/master@{#458047}
Committed: https://chromium.googlesource.com/chromium/src/+/e32318796eaf782ee8afe8ec047b1288f756f644

[bsazonov, 2017-03-20 10:27:42]

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

[commit-bot: I haz the power, 2017-03-20 10:27:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-20 11:20:33]

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

[commit-bot: I haz the power, 2017-03-20 11:20:34]

Dry run: This issue passed the CQ dry run.

[bsazonov, 2017-03-20 12:08:20]

bsazonov@chromium.org changed reviewers:
+ msarda@chromium.org

[bsazonov, 2017-03-20 12:08:20]

Mihai, please take a look.

[msarda, 2017-03-20 12:33:04]

lgtm

[bsazonov, 2017-03-20 13:01:38]

The CQ bit was checked by bsazonov@chromium.org

[commit-bot: I haz the power, 2017-03-20 13:01:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-20 13:05:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490014898557630, "parent_rev":
"9783cf1b1bfe51a505ef158e2db0959b9fc1e115", "commit_rev":
"e32318796eaf782ee8afe8ec047b1288f756f644"}

[commit-bot: I haz the power, 2017-03-20 13:06:22]

Description was changed from

==========
Fix unique_ptr usage in handling OAuth2TokenService::StartRequest result

Calling release() on a temporary unique_ptr to reset another unique_ptr with
resulting raw pointer is verbose and obscure. It also loses custom deleter
(see C++11 standard, clause 20.7.1.2.3.1).

BUG=none
==========

to

==========
Fix unique_ptr usage in handling OAuth2TokenService::StartRequest result

Calling release() on a temporary unique_ptr to reset another unique_ptr with
resulting raw pointer is verbose and obscure. It also loses custom deleter
(see C++11 standard, clause 20.7.1.2.3.1).

BUG=none

Review-Url: https://codereview.chromium.org/2755403002
Cr-Commit-Position: refs/heads/master@{#458047}
Committed:
https://chromium.googlesource.com/chromium/src/+/e32318796eaf782ee8afe8ec047b...
==========

[commit-bot: I haz the power, 2017-03-20 13:06:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e32318796eaf782ee8afe8ec047b...