DevTools: no way to remote debug using ToT build.

net/server/web_socket.h

+// Copyright (c) 2011 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.
+
+#ifndef NET_SERVER_WEB_SOCKET_H_
+#define NET_SERVER_WEB_SOCKET_H_
+#pragma once
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/server/http_server.h"
+
+#include <string>
+#include <vector>
+
+namespace net {
+
+class HttpServerRequestInfo;
+
+class WebSocket {
+ public:
+  static WebSocket* createWebSocket(Connection* connection,

[eroman, 2011/08/02 18:46:41]

CreateWebSocket

+                                    const HttpServerRequestInfo& request,
+                                    size_t* pos);
+
+  enum ParseResult {

[eroman, 2011/08/02 18:46:41]

Move this to the top of the public section.

+    FrameOK,
+    FrameIncomplete,
+    FrameError

[eroman, 2011/08/02 18:46:41]

Chrome is using ALL_UPER_CASE_FOR_ENUMS

+  };
+
+  virtual void Accept(const HttpServerRequestInfo& request) = 0;
+  virtual ParseResult Read(std::string* message) = 0;
+  virtual void Send(const std::string& message) = 0;

[eroman, 2011/08/02 18:46:41]

Please add a virtual destructor.

I always recommend doing this whenever there are virtual methods, to make sure
we don't accidentally leak in child class by forgetting virtual dtor.

+
+ protected:
+  WebSocket(Connection* connection);

[eroman, 2011/08/02 18:46:41]

mark this as explicit.

+  Connection* connection_;
+};
+
+}  // namespace net
+
+#endif // NET_SERVER_WEB_SOCKET_H_