New PeerConnection handler in Chrome to support latest PeerConnection draft

content/renderer/renderer_webkitplatformsupport_impl.cc

Index: content/renderer/renderer_webkitplatformsupport_impl.cc
diff --git a/content/renderer/renderer_webkitplatformsupport_impl.cc b/content/renderer/renderer_webkitplatformsupport_impl.cc
index c7e274782cde98582fd08da29feb1239d521e2aa..fc89387ef492864e09ca78e2671586138cf60606 100644
--- a/content/renderer/renderer_webkitplatformsupport_impl.cc
+++ b/content/renderer/renderer_webkitplatformsupport_impl.cc
@@ -96,6 +96,8 @@ using WebKit::WebPeerConnection00Handler;
 using WebKit::WebPeerConnection00HandlerClient;
 using WebKit::WebPeerConnectionHandler;
 using WebKit::WebPeerConnectionHandlerClient;
+using WebKit::WebRTCPeerConnectionHandler;
+using WebKit::WebRTCPeerConnectionHandlerClient;
 using WebKit::WebSerializedScriptValue;
 using WebKit::WebStorageNamespace;
 using WebKit::WebString;
@@ -713,6 +715,19 @@ RendererWebKitPlatformSupportImpl::createPeerConnection00Handler(
   return render_view->CreatePeerConnectionHandlerJsep(client);
 }
 
+WebRTCPeerConnectionHandler*
+RendererWebKitPlatformSupportImpl::createRTCPeerConnectionHandler(
+    WebRTCPeerConnectionHandlerClient* client) {
+  WebFrame* web_frame = WebFrame::frameForCurrentContext();
+  if (web_frame) {

[Ronghua Wu (Left Chromium), 2012/08/14 00:59:03]

I think the format in createPeerConnection00Handler is clearer:
if (!web_frame) {
  return NULL;
}

[perkj_chrome, 2012/08/14 09:15:40]

please talk to your colleague next to you :->
Ok - will change it back.

please also note the indentation of the method names that don't fit on one line
in this file. Seems like some uses this indentation and some indent the method
names.

On 2012/08/14 00:59:03, Ronghua Wu wrote:

[Ronghua Wu (Left Chromium), 2012/08/15 00:57:32]

:) This is the format preferred in libjingle and I also personally think it's
clearer. If chromium has another preference I'm ok with both.

+    RenderViewImpl* view = RenderViewImpl::FromWebView(web_frame->view());
+    if (view) {
+      return view->CreateRTCPeerConnectionHandler(client);
+    }
+  }
+  return NULL;
+}
+
 //------------------------------------------------------------------------------
 
 WebMediaStreamCenter*