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..65ca7a1c666cd3be8ac037bb5ff455989dbfce61 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,18 @@ RendererWebKitPlatformSupportImpl::createPeerConnection00Handler(
   return render_view->CreatePeerConnectionHandlerJsep(client);
 }
 
+WebRTCPeerConnectionHandler*
+RendererWebKitPlatformSupportImpl::createRTCPeerConnectionHandler(
+    WebRTCPeerConnectionHandlerClient* client) {
+  WebFrame* web_frame = WebFrame::frameForCurrentContext();
+  if (!web_frame)
+    return NULL;
+  RenderViewImpl* render_view = RenderViewImpl::FromWebView(web_frame->view());
+  if (!render_view)
+    return NULL;
+  return render_view->CreateRTCPeerConnectionHandler(client);

[wjia(left Chromium), 2012/08/12 17:06:52]

How about:
if (web_frame) {
  RenderViewImpl* render_view = RenderViewImpl::FromWebView(web_frame->view());
  if (render_view)
    return render_view->CreateRTCPeerConnectionHandler(client);
}

return NULL;

[perkj_chrome, 2012/08/13 07:35:46]

Sounds good. In a number of reviews I have had comments about prefering short
exits. But your suggestion looks nicer here.
On 2012/08/12 17:06:52, wjia wrote:

+}
+
 //------------------------------------------------------------------------------
 
 WebMediaStreamCenter*