Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(61)

Unified Diff: third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp

Issue 2080623002: Revert "Remove OwnPtr from Blink." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp
diff --git a/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp b/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp
index 69036091dc8f859d487578fef01f86e10b7e7ef3..73b697cb371464c69159e7a5fbca0127d615d0d2 100644
--- a/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp
+++ b/third_party/WebKit/Source/modules/websockets/DocumentWebSocketChannelTest.cpp
@@ -21,10 +21,9 @@
#include "public/platform/modules/websockets/WebSocketHandleClient.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
-#include "wtf/PtrUtil.h"
+#include "wtf/OwnPtr.h"
#include "wtf/Vector.h"
#include "wtf/text/WTFString.h"
-#include <memory>
#include <stdint.h>
using testing::_;
@@ -53,7 +52,7 @@ public:
MOCK_METHOD2(didConnect, void(const String&, const String&));
MOCK_METHOD1(didReceiveTextMessage, void(const String&));
- void didReceiveBinaryMessage(std::unique_ptr<Vector<char>> payload) override
+ void didReceiveBinaryMessage(PassOwnPtr<Vector<char>> payload) override
{
didReceiveBinaryMessageMock(*payload);
}
@@ -142,7 +141,7 @@ public:
::testing::Mock::VerifyAndClearExpectations(this);
}
- std::unique_ptr<DummyPageHolder> m_pageHolder;
+ OwnPtr<DummyPageHolder> m_pageHolder;
Persistent<MockWebSocketChannelClient> m_channelClient;
MockWebSocketHandle* m_handle;
Persistent<DocumentWebSocketChannel> m_channel;
@@ -241,7 +240,7 @@ TEST_F(DocumentWebSocketChannelTest, sendBinaryInVector)
Vector<char> fooVector;
fooVector.append("foo", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(fooVector)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(fooVector)));
EXPECT_EQ(3ul, m_sumOfConsumedBufferedAmount);
}
@@ -263,22 +262,22 @@ TEST_F(DocumentWebSocketChannelTest, sendBinaryInVectorWithNullBytes)
{
Vector<char> v;
v.append("\0ar", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
}
{
Vector<char> v;
v.append("b\0z", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
}
{
Vector<char> v;
v.append("qu\0", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
}
{
Vector<char> v;
v.append("\0\0\0", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
}
EXPECT_EQ(12ul, m_sumOfConsumedBufferedAmount);
@@ -294,7 +293,7 @@ TEST_F(DocumentWebSocketChannelTest, sendBinaryInVectorNonLatin1UTF8)
Vector<char> v;
v.append("\xe7\x8b\x90", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
EXPECT_EQ(3ul, m_sumOfConsumedBufferedAmount);
}
@@ -309,7 +308,7 @@ TEST_F(DocumentWebSocketChannelTest, sendBinaryInVectorNonUTF8)
Vector<char> v;
v.append("\x80\xff\xe7", 3);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
EXPECT_EQ(3ul, m_sumOfConsumedBufferedAmount);
}
@@ -330,7 +329,7 @@ TEST_F(DocumentWebSocketChannelTest, sendBinaryInVectorNonLatin1UTF8Continuation
Vector<char> v;
v.append("\xe7\x8b\x90\xe7\x8b\x90\xe7\x8b\x90\xe7\x8b\x90\xe7\x8b\x90\xe7\x8b\x90", 18);
- channel()->sendBinaryAsCharVector(wrapUnique(new Vector<char>(v)));
+ channel()->sendBinaryAsCharVector(adoptPtr(new Vector<char>(v)));
checkpoint.Call(1);
handleClient()->didReceiveFlowControl(handle(), 16);

Powered by Google App Engine
This is Rietveld 408576698