Made PassRefPtr move only and replaced new copies with moves.

Removed PassRefPtr copy constructor.

Added move wraps for remaining PassRefPtr copies (newly introduced).

BUG=640449

Review-Url: https://codereview.chromium.org/2437883002
Cr-Commit-Position: refs/heads/master@{#427020}

third_party/WebKit/Source/core/css/PropertyRegistry.cpp

@@ -13,8 +13,8 @@ void PropertyRegistry::registerProperty(
     const CSSValue* initial,
     PassRefPtr<CSSVariableData> initialVariableData) {
   DCHECK(!registration(name));
-  m_registrations.set(
-      name, new Registration(syntax, inherits, initial, initialVariableData));
+  m_registrations.set(name, new Registration(syntax, inherits, initial,
+                                             std::move(initialVariableData)));
 }
 
 const PropertyRegistry::Registration* PropertyRegistry::registration(

third_party/WebKit/Source/core/style/StyleNonInheritedVariables.h

@@ -30,7 +30,7 @@ class StyleNonInheritedVariables {
 
   void setVariable(const AtomicString& name,
                    PassRefPtr<CSSVariableData> value) {
-    m_data.set(name, value);
+    m_data.set(name, std::move(value));
   }
   CSSVariableData* getVariable(const AtomicString& name) const;
   void removeVariable(const AtomicString&);

third_party/WebKit/Source/modules/fetch/BlobBytesConsumer.cpp

@@ -36,7 +36,7 @@ BlobBytesConsumer::BlobBytesConsumer(ExecutionContext* executionContext,
 
 BlobBytesConsumer::BlobBytesConsumer(ExecutionContext* executionContext,
                                      PassRefPtr<BlobDataHandle> blobDataHandle)
-    : BlobBytesConsumer(executionContext, blobDataHandle, nullptr) {}
+    : BlobBytesConsumer(executionContext, std::move(blobDataHandle), nullptr) {}
 
 BlobBytesConsumer::~BlobBytesConsumer() {}
 
@@ -265,7 +265,8 @@ BlobBytesConsumer* BlobBytesConsumer::createForTesting(
     ExecutionContext* executionContext,
     PassRefPtr<BlobDataHandle> blobDataHandle,
     ThreadableLoader* loader) {
-  return new BlobBytesConsumer(executionContext, blobDataHandle, loader);
+  return new BlobBytesConsumer(executionContext, std::move(blobDataHandle),
+                               loader);
 }
 
 ThreadableLoader* BlobBytesConsumer::createLoader() {

third_party/WebKit/Source/modules/fetch/FormDataBytesConsumer.cpp

@@ -212,7 +212,7 @@ FormDataBytesConsumer::FormDataBytesConsumer(const void* data, size_t size)
 FormDataBytesConsumer::FormDataBytesConsumer(
     ExecutionContext* executionContext,
     PassRefPtr<EncodedFormData> formData)
-    : FormDataBytesConsumer(executionContext, formData, nullptr) {}
+    : FormDataBytesConsumer(executionContext, std::move(formData), nullptr) {}
 
 FormDataBytesConsumer::FormDataBytesConsumer(
     ExecutionContext* executionContext,
@@ -220,10 +220,10 @@ FormDataBytesConsumer::FormDataBytesConsumer(
     BytesConsumer* consumer)
     : m_impl(isSimple(formData.get())
                  ? static_cast<BytesConsumer*>(
-                       new SimpleFormDataBytesConsumer(formData))
+                       new SimpleFormDataBytesConsumer(std::move(formData)))
                  : static_cast<BytesConsumer*>(
                        new ComplexFormDataBytesConsumer(executionContext,
-                                                        formData,
+                                                        std::move(formData),
                                                         consumer))) {}
 
 }  // namespace blink

third_party/WebKit/Source/modules/fetch/FormDataBytesConsumer.h

@@ -29,7 +29,8 @@ class FormDataBytesConsumer final : public BytesConsumer {
       ExecutionContext* executionContext,
       PassRefPtr<EncodedFormData> formData,
       BytesConsumer* consumer) {
-    return new FormDataBytesConsumer(executionContext, formData, consumer);
+    return new FormDataBytesConsumer(executionContext, std::move(formData),
+                                     consumer);
   }
 
   // BytesConsumer implementation

third_party/WebKit/Source/wtf/PassRefPtr.h

@@ -71,11 +71,6 @@ class PassRefPtr {
   PassRefPtr() : m_ptr(nullptr) {}
   PassRefPtr(std::nullptr_t) : m_ptr(nullptr) {}
   PassRefPtr(T* ptr) : m_ptr(ptr) { refIfNotNull(ptr); }
-  // It somewhat breaks the type system to allow transfer of ownership out of
-  // a const PassRefPtr. However, it makes it much easier to work with
-  // PassRefPtr temporaries, and we don't have a need to use real const
-  // PassRefPtrs anyway.
-  PassRefPtr(const PassRefPtr& o) : m_ptr(o.leakRef()) {}
   PassRefPtr(PassRefPtr&& o) : m_ptr(o.leakRef()) {}
   template <typename U>
   PassRefPtr(const PassRefPtr<U>& o, EnsurePtrConvertibleArgDecl(U, T))