Move P2P code to content namespace.

content/renderer/p2p/ipc_socket_factory.cc

 // 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.
 
 #include "content/renderer/p2p/ipc_socket_factory.h"
 
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
 #include "content/renderer/p2p/socket_client.h"
 #include "content/renderer/p2p/socket_dispatcher.h"
 #include "jingle/glue/utils.h"
 #include "third_party/libjingle/source/talk/base/asyncpacketsocket.h"
 
+namespace content {
+
 namespace {

[jam, 2011/08/24 00:24:27]

ditto

[Sergey Ulanov, 2011/08/24 01:57:48]

Putting this code in content namespace makets it simpler, because I don't need
to specify namespace when accessing classes from content namespace.

[jam, 2011/08/24 16:30:53]

it seems a little odd having an anonymous namespace inside of another namespace.
if you don't want to keep putting content::, you can just do "using
content::P2PBlah.." for the classes that you need, like what we do with WebKit
classes

[Sergey Ulanov, 2011/08/24 17:35:12]

Actually there is a lot of code in chrome that defines anonymous namespace
inside of another namespace and it never caused any problems. Some random
examples I found (and there are many more):
 base/callback_unittest.cc
 base/json/json_reader.cc
 base/linked_list_unittest.cc
 net/base/address_list.cc
 net/base/address_list_unittest.cc
 net/socket/client_socket_factory.cc
 net/socket/tcp_client_socket_unittest.cc
 remoting/*/*

[jam, 2011/08/24 23:38:42]

to be clear, I didn't say it caused problems. if there are other places doing
this, then ok

 
 // IpcPacketSocket implements talk_base::AsyncPacketSocket interface
 // using P2PSocketClient that works over IPC-channel. It must be used
 // on the thread it was created.
 class IpcPacketSocket : public talk_base::AsyncPacketSocket,
                         public P2PSocketClient::Delegate {
  public:
   IpcPacketSocket();
   virtual ~IpcPacketSocket();
 
@@ skipping 329 matching lines @@
     return NULL;
 
   talk_base::SocketAddress crome_address;
   P2PSocketClient* socket_client = new P2PSocketClient(socket_dispatcher_);
   scoped_ptr<IpcPacketSocket> socket(new IpcPacketSocket());
   if (!socket->Init(P2P_SOCKET_TCP_CLIENT, socket_client, local_address,
                     remote_address))
     return NULL;
   return socket.release();
 }
+
+}  // namespace content