Reland 8688002: PPB_TCPSocket_Private/PPB_UDPSocket_Private are exposed to Browser

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index a89b4731b834c0a54168d5272006a0a680335c7a..ee56e9a63ceedcf5c5dc67da5a23e17b72a0b499 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -599,6 +599,12 @@ bool RenderViewImpl::OnMessageReceived(const IPC::Message& message) {
     if (observer->OnMessageReceived(message))
       return true;
 
+  // Pepper plugin delegate doesn't use observers mechanism to don't implement
+  // huge interface RenderViewObserver that receives notifications about
+  // everything.

[jam, 2011/12/05 18:07:49]

None of RenderViewObserver's virtual methods are pure by design, so that
implementors only have to override the methods they care about. Please use
RenderViewObserver here.

[Dmitry Polukhin, 2011/12/05 19:09:26]

Done but it's inefficient and creates cyclic dependency in RenderViewImpl
construction order. RenderViewObserver in constructor tries to add observer to
RenderViewImple but it is not fully constructed by that point (constructor for
observers vector was not called because it is declared below pepper_delegate_).
So I had to use NULL in RenderViewObserver and add observers manually.  I can
change member order in RenderViewImpl but it's still bad to have such
construction order dependency.

[jam, 2011/12/05 19:24:43]

I think only profiling can show us that this is slow. The reason we've split off
as much code as possible into RenderViewObservers is to avoid the previous
status quo, which was everything was being dispatched in RenderView. This made
the code very hard to understand.
I think switching the member order is better in this case (it wasn't important
before, but now it matters for this).

+  if (pepper_delegate_.OnMessageReceived(message))
+    return true;
+
   bool handled = true;
   bool msg_is_ok = true;
   IPC_BEGIN_MESSAGE_MAP_EX(RenderViewImpl, message, msg_is_ok)