Added move constructor to PassRefPtr.

third_party/WebKit/Source/wtf/PassRefPtrTest.cpp

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "PassRefPtr.h"

[Yuta Kitamura, 2016/08/29 05:03:47]

nit: We spell out the include path fully; no relative paths.

+
+#include "testing/gtest/include/gtest/gtest.h"
+#include "wtf/RefCounted.h"
+
+namespace WTF {
+
+class RefCountedClass : public RefCounted<RefCountedClass> {
+};
+
+TEST(PassRefPtrTest, MoveConstructor)
+{
+    PassRefPtr<RefCountedClass> oldPassRefPtr = adoptRef(new RefCountedClass());
+    RefCountedClass* rawPtr = oldPassRefPtr.get();
+    EXPECT_EQ(rawPtr->refCount(), 1);
+    PassRefPtr<RefCountedClass> newPassRefPtr = PassRefPtr<RefCountedClass>(std::move(oldPassRefPtr));

[Yuta Kitamura, 2016/08/29 05:03:47]

nit: This actually causes a move construction AND an assignment.
You probably just want a move construction, like this:

    PassRefPtr<...> newPassRefPtr(std::move(oldPassRefPtr));

+    EXPECT_EQ(oldPassRefPtr.get(), nullptr);
+    EXPECT_EQ(newPassRefPtr.get(), rawPtr);
+    EXPECT_EQ(rawPtr->refCount(), 1);
+}
+
+} // namespace WTF