Replaced RefPtr::release with std::move in Source/core.

Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2622043003
Cr-Commit-Position: refs/heads/master@{#443409}
Committed: https://chromium.googlesource.com/chromium/src/+/52d6d437288de59def855af9bec777a5d7ac88d6

[Bugs Nash, 2017-01-11 04:46:02]

Description was changed from

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

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

to

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Bugs Nash, 2017-01-11 04:46:46]

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

[Bugs Nash, 2017-01-11 04:46:56]

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

[commit-bot: I haz the power, 2017-01-11 04:47:10]

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-01-11 04:56:34]

Description was changed from

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of 
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Bugs Nash, 2017-01-11 05:01:05]

Description was changed from

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of 
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[alancutter (OOO until 2018), 2017-01-11 05:14:19]

alancutter@chromium.org changed reviewers:
+ alancutter@chromium.org

[alancutter (OOO until 2018), 2017-01-11 05:14:20]

core/animation lgtm.

https://codereview.chromium.org/2622043003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/animation/CSSClipInterpolationType.cpp
(right):

https://codereview.chromium.org/2622043003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/animation/CSSClipInterpolationType.cpp:244:
std::move(start.nonInterpolableValue));
Yay consistency.

[commit-bot: I haz the power, 2017-01-11 07:27:06]

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

[commit-bot: I haz the power, 2017-01-11 07:27:07]

Dry run: This issue passed the CQ dry run.

[Yuta Kitamura, 2017-01-12 10:29:19]

lgtm

[Bugs Nash, 2017-01-12 20:53:50]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-01-12 20:54:29]

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-01-13 00:28:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484254430414400, "parent_rev":
"5dbc91d59a8f1fbfd4987f8539ee032fe29481c3", "commit_rev":
"52d6d437288de59def855af9bec777a5d7ac88d6"}

[commit-bot: I haz the power, 2017-01-13 00:28:42]

Description was changed from

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Replaced RefPtr::release with std::move in Source/core.

Part of the removal of PassRefPtr.

Replaces many of the calls to RefPtr::release with a std::move wrap,
which returns a RefPtr instead of a PassRefPtr. As the code currently
stands, most methods still expect a PassRefPtr and so the PassRefPtr
initialising constructor will be called passing in the RefPtr rvalue,
which calls RefPtr::release anyway.

Future patches will see these variables changed from PassRefPtr to
RefPtr type then RefPtr::release will not be used at all.

Does not handle RefPtr::release of non member variables in return
statements as these cases sometimes default to move (in the case of
returning a local variable). These will be handled in future patches.

BUG=494719
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2622043003
Cr-Commit-Position: refs/heads/master@{#443409}
Committed:
https://chromium.googlesource.com/chromium/src/+/52d6d437288de59def855af9bec7...
==========

[commit-bot: I haz the power, 2017-01-13 00:28:44]

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