commit 46a1acfe9dbf55999b39dd2f9b1917c1beb9639c
author olli.raula <olli.raula@intel.com> Wed Nov 04 08:45:52 2015
committer Commit bot <commit-bot@chromium.org> Wed Nov 04 08:46:28 2015
tree 094ddeb3e15e426008f63a58a34c7c7c42cb23cd
parent cd992c08b465526201070ee57904452b46ad63ca

Make WebSocketEncoder::CreateClient return scoped_ptr

Make WebSocketEncoder::CreateClient return scoped_ptr
increase pointer safety and because all consumers(only in unittests) are using scoped_ptr already.

Review URL: https://codereview.chromium.org/1425593005

Cr-Commit-Position: refs/heads/master@{#357782}

net/server/web_socket_encoder.h

diff --git a/net/server/web_socket_encoder.h b/net/server/web_socket_encoder.h
index 1eb749f..097df0e 100644
--- a/net/server/web_socket_encoder.h
+++ b/net/server/web_socket_encoder.h
@@ -32,8 +32,8 @@
   static scoped_ptr<WebSocketEncoder> CreateServer(
       const std::string& extensions,
       WebSocketDeflateParameters* params);
-  // TODO(yhirano): Return a scoped_ptr instead of a raw pointer.
-  static WebSocketEncoder* CreateClient(const std::string& response_extensions);
+  static scoped_ptr<WebSocketEncoder> CreateClient(
+      const std::string& response_extensions);
 
   WebSocket::ParseResult DecodeFrame(const base::StringPiece& frame,
                                      int* bytes_consumed,