Chromium Code Reviews
DescriptionReplaced RefPtr::release with std::move in Source/platform.
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
Review-Url: https://codereview.chromium.org/2628773002
Cr-Commit-Position: refs/heads/master@{#443416}
Committed: https://chromium.googlesource.com/chromium/src/+/3ec92ab7b54be886a9b6849c0c6d1ddda99439ee
Patch Set 1 #
Messages
Total messages: 14 (9 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||