Move P2P code to content namespace.

content/renderer/p2p/ipc_socket_factory.cc

Index: content/renderer/p2p/ipc_socket_factory.cc
diff --git a/content/renderer/p2p/ipc_socket_factory.cc b/content/renderer/p2p/ipc_socket_factory.cc
index c579433bf119d2ea63a16a5305e2994354bd7f02..64cc3e02e71069fd4621b0dc33f575b0647b0fc3 100644
--- a/content/renderer/p2p/ipc_socket_factory.cc
+++ b/content/renderer/p2p/ipc_socket_factory.cc
@@ -12,6 +12,8 @@
 #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
@@ -362,3 +364,5 @@ talk_base::AsyncPacketSocket* IpcPacketSocketFactory::CreateClientTcpSocket(
     return NULL;
   return socket.release();
 }
+
+}  // namespace content