Chromium Code Reviews| Index: Source/modules/websockets/ThreadableWebSocketChannelClientWrapper.h |
| diff --git a/Source/modules/websockets/ThreadableWebSocketChannelClientWrapper.h b/Source/modules/websockets/ThreadableWebSocketChannelClientWrapper.h |
| index bed47dfb9db43e86426a3912466e01165ea353f2..6b7efb0190569662c54724f5e7e2da3ca2283cb4 100644 |
| --- a/Source/modules/websockets/ThreadableWebSocketChannelClientWrapper.h |
| +++ b/Source/modules/websockets/ThreadableWebSocketChannelClientWrapper.h |
| @@ -45,9 +45,9 @@ namespace WebCore { |
| class ExecutionContext; |
| class WebSocketChannelClient; |
| -class ThreadableWebSocketChannelClientWrapper : public ThreadSafeRefCounted<ThreadableWebSocketChannelClientWrapper> { |
| +class ThreadableWebSocketChannelClientWrapper : public ThreadSafeRefCountedWillBeGarbageCollectedFinalized<ThreadableWebSocketChannelClientWrapper> { |
| public: |
| - static PassRefPtr<ThreadableWebSocketChannelClientWrapper> create(WebSocketChannelClient*); |
| + static PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper> create(WebSocketChannelClient*); |
| // Subprotocol and extensions will be available when didConnect() callback is invoked. |
| String subprotocol() const; |
| @@ -68,18 +68,20 @@ public: |
| void suspend(); |
| void resume(); |
| + void trace(Visitor*) { } |
| + |
| private: |
| ThreadableWebSocketChannelClientWrapper(WebSocketChannelClient*); |
| void processPendingTasks(); |
| - static void didConnectCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>); |
| - static void didReceiveMessageCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>, const String& message); |
| - static void didReceiveBinaryDataCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>, PassOwnPtr<Vector<char> >); |
| - static void didUpdateBufferedAmountCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>, unsigned long bufferedAmount); |
| - static void didStartClosingHandshakeCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>); |
| - static void didCloseCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>, unsigned long unhandledBufferedAmount, WebSocketChannelClient::ClosingHandshakeCompletionStatus, unsigned short code, const String& reason); |
| - static void didReceiveMessageErrorCallback(ExecutionContext*, PassRefPtr<ThreadableWebSocketChannelClientWrapper>); |
| + static void didConnectCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>); |
| + static void didReceiveMessageCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>, const String& message); |
| + static void didReceiveBinaryDataCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>, PassOwnPtr<Vector<char> >); |
| + static void didUpdateBufferedAmountCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>, unsigned long bufferedAmount); |
| + static void didStartClosingHandshakeCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>); |
| + static void didCloseCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>, unsigned long unhandledBufferedAmount, WebSocketChannelClient::ClosingHandshakeCompletionStatus, unsigned short code, const String& reason); |
| + static void didReceiveMessageErrorCallback(ExecutionContext*, PassRefPtrWillBeRawPtr<ThreadableWebSocketChannelClientWrapper>); |
| WebSocketChannelClient* m_client; |
|
haraken
2014/05/07 07:45:16
This raw pointer is safe because it's explicitly c
|
| // ThreadSafeRefCounted must not have String member variables. |