Removed calls to RefPtr::Release in return statements with auto move.

Removed calls to RefPtr::Release in return statements with auto move.

Removed calls to RefPtr::Release in return statements that automatically
move the object returned (i.e. where the RefPtr returned is local to the
method). These methods still return a PassRefPtr and so the RefPtr will
be implicitly converted to a PassRefPtr. Future patches will change the
return types to RefPtr.

BUG=494719
Review-Url: https://codereview.chromium.org/2957513002
Cr-Commit-Position: refs/heads/master@{#482178}
Committed: https://chromium.googlesource.com/chromium/src/+/2083513f84c5d1c914275031d0064875e27b28f8

[Bugs Nash, 2017-06-23 02:08:03]

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

[commit-bot: I haz the power, 2017-06-23 02:08:16]

Dry run: CQ is trying da patch.

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

[Bugs Nash, 2017-06-23 02:08:43]

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

[commit-bot: I haz the power, 2017-06-23 02:23:18]

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

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

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[haraken, 2017-06-23 02:25:37]

LGTM assuming that this won't have any performance impact.

[Yuta Kitamura, 2017-06-23 06:29:08]

LGTM

The changes in this file do not change the runtime behavior,
so the performance will stay the same. (I expect the
compilers will generate the completely identical assemblies.)

You don't have to test performance effect of this CL locally.

[Bugs Nash, 2017-06-25 22:27:56]

rebased

[Bugs Nash, 2017-06-25 22:28:04]

The CQ bit was checked by bugsnash@chromium.org

[Bugs Nash, 2017-06-25 22:28:04]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, yutak@chromium.org
Link to the patchset: https://codereview.chromium.org/2957513002/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2017-06-25 22:28:14]

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-06-26 00:19:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498429684001810,
"parent_rev": "20fca46b45d56973beb701dfcd4f77a433077a39", "commit_rev":
"2083513f84c5d1c914275031d0064875e27b28f8"}

[commit-bot: I haz the power, 2017-06-26 00:19:50]

Description was changed from

==========
Removed calls to RefPtr::Release in return statements with auto move.

Removed calls to RefPtr::Release in return statements that automatically
move the object returned (i.e. where the RefPtr returned is local to the
method). These methods still return a PassRefPtr and so the RefPtr will
be implicitly converted to a PassRefPtr. Future patches will change the
return types to RefPtr.

BUG=494719
==========

to

==========
Removed calls to RefPtr::Release in return statements with auto move.

Removed calls to RefPtr::Release in return statements that automatically
move the object returned (i.e. where the RefPtr returned is local to the
method). These methods still return a PassRefPtr and so the RefPtr will
be implicitly converted to a PassRefPtr. Future patches will change the
return types to RefPtr.

BUG=494719

Review-Url: https://codereview.chromium.org/2957513002
Cr-Commit-Position: refs/heads/master@{#482178}
Committed:
https://chromium.googlesource.com/chromium/src/+/2083513f84c5d1c914275031d006...
==========

[commit-bot: I haz the power, 2017-06-26 00:19:52]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2083513f84c5d1c914275031d006...