Removed PassRefPtr to RefPtr copies from unit tests.

Removed PassRefPtr to RefPtr copies from unit tests.

All instances of PassRefPtr objects being copied into RefPtr objects
need to be wrapped in std::move to avoid introducing ref churn in
future changes when all PassRefPtr objects will be replaced with RefPtr
objects. Some cases create a local RefPtr just to copy the PassRefPtr
argument into; in these cases a std::move wrap doesn't make sense as it
would result in a final state where a RefPtr argument is moved into a
local RefPtr for no reason. So to handle these cases I have converted
the PassRefPtr arguments to RefPtr in advance, and removed the local
RefPtr argument, instead using the passed RefPtr throughout the
methods.

Changed include
- added std::move wraps to PassReftr objects to force a move
- removed local RefPtr objects where PassRefPtr arguments are copied
  into them, instead change PassRefPtr argument to RefPtr that is
  directly used.

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

[Bugs Nash, 2017-05-08 02:50:27]

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

[commit-bot: I haz the power, 2017-05-08 02:50:40]

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-05-08 02:51:31]

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

[haraken, 2017-05-08 04:10:26]

LGTM

[commit-bot: I haz the power, 2017-05-08 04:52:31]

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

[commit-bot: I haz the power, 2017-05-08 04:52:32]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yuta Kitamura, 2017-05-08 05:56:10]

lgtm

[Bugs Nash, 2017-05-09 04:25:29]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-05-09 04:26:32]

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-05-09 05:48:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494303929805920, "parent_rev":
"bbfb3316d1d216a98ca569445e304aa6e6f4e6d4", "commit_rev":
"aa79e549a918ff2530db858e281eb9f1515c2d5b"}

[commit-bot: I haz the power, 2017-05-09 05:48:09]

Description was changed from

==========
Removed PassRefPtr to RefPtr copies from unit tests.

All instances of PassRefPtr objects being copied into RefPtr objects
need to be wrapped in std::move to avoid introducing ref churn in
future changes when all PassRefPtr objects will be replaced with RefPtr
objects. Some cases create a local RefPtr just to copy the PassRefPtr
argument into; in these cases a std::move wrap doesn't make sense as it
would result in a final state where a RefPtr argument is moved into a
local RefPtr for no reason. So to handle these cases I have converted
the PassRefPtr arguments to RefPtr in advance, and removed the local
RefPtr argument, instead using the passed RefPtr throughout the
methods.

Changed include
- added std::move wraps to PassReftr objects to force a move
- removed local RefPtr objects where PassRefPtr arguments are copied
  into them, instead change PassRefPtr argument to RefPtr that is
  directly used.

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

to

==========
Removed PassRefPtr to RefPtr copies from unit tests.

All instances of PassRefPtr objects being copied into RefPtr objects
need to be wrapped in std::move to avoid introducing ref churn in
future changes when all PassRefPtr objects will be replaced with RefPtr
objects. Some cases create a local RefPtr just to copy the PassRefPtr
argument into; in these cases a std::move wrap doesn't make sense as it
would result in a final state where a RefPtr argument is moved into a
local RefPtr for no reason. So to handle these cases I have converted
the PassRefPtr arguments to RefPtr in advance, and removed the local
RefPtr argument, instead using the passed RefPtr throughout the
methods.

Changed include
- added std::move wraps to PassReftr objects to force a move
- removed local RefPtr objects where PassRefPtr arguments are copied
  into them, instead change PassRefPtr argument to RefPtr that is
  directly used.

BUG=494719

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

[commit-bot: I haz the power, 2017-05-09 05:48:10]

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