Replaced PassRefPtr copies with moves in Source/web and Source/wtf.

third_party/WebKit/Source/wtf/HashMap.h

Index: third_party/WebKit/Source/wtf/HashMap.h
diff --git a/third_party/WebKit/Source/wtf/HashMap.h b/third_party/WebKit/Source/wtf/HashMap.h
index 776ccba0eea01f503b60cd67f3b8c90f2b415131..568ece55c547eb9b76d4210570a963024ddf0b18 100644
--- a/third_party/WebKit/Source/wtf/HashMap.h
+++ b/third_party/WebKit/Source/wtf/HashMap.h
@@ -252,7 +252,7 @@ struct HashMapTranslator {
     template <typename T, typename U, typename V> static void translate(T& location, U&& key, V&& mapped)
     {
         location.key = std::forward<U>(key);
-        ValueTraits::ValueTraits::store(std::forward<V>(mapped), location.value);
+        ValueTraits::ValueTraits::store(std::move(std::forward<V>(mapped)), location.value);

[Yuta Kitamura, 2016/09/05 06:46:09]

Nesting std::move and std::forward does not make sense; forward()
is a generalized version of move(), thus you probably don't want
to touch here.

I guess the real error is in somewhere else, likely in a call
site of HashMap::add() or the like.

[Bugs Nash, 2016/09/14 05:23:29]

Done

     }
 };
 
@@ -371,7 +371,7 @@ HashMap<T, U, V, W, X, Y>::set(IncomingKeyType&& key, IncomingMappedType&& mappe
         // to set the mapped value.
         //
         // It's safe to call std::forward again, because |mapped| isn't moved if there's an existing entry.
-        MappedTraits::store(std::forward<IncomingMappedType>(mapped), result.storedValue->value);
+        MappedTraits::store(std::move(std::forward<IncomingMappedType>(mapped)), result.storedValue->value);

[Yuta Kitamura, 2016/09/05 06:46:09]

Ditto.

[Bugs Nash, 2016/09/14 05:23:29]

Done

     }
     return result;
 }