Browser Plugin: Removed BrowserPluginWebContentsObserver (to be replaced by rewritten BrowserPlugi…

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 308dc60bd56bed2eb860b4e7de3b263dafd0c056..fe1431f7db98b1ded743ee73eb420c3c540c21b0 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -878,7 +878,7 @@ void RenderThreadImpl::OnCreateNewView(const ViewMsg_New_Params& params) {
       params.swapped_out,
       params.next_page_id,
       params.screen_info,
-      params.guest,
+      false,

[Charlie Reis, 2012/05/11 17:41:03]

Is it too soon to remove this parameter and pass in the new IDs?

[Fady Samuel, 2012/05/11 22:03:34]

Yea, I currently don't pass these to RenderViewImpl, instead I have a
BrowserPluginChannelManager that creates or looks up an existing
GuestToEmbedderChannel. I need to land BrowserPluginChannelManager and
GuestToEmbedderChannel to make that change to RenderViewImpl. This will come in
a subsequent patch.

       params.accessibility_mode);
 }