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

content/renderer/render_thread_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 860 matching lines @@
       params.web_preferences,
       new SharedRenderViewCounter(0),
       params.view_id,
       params.surface_id,
       params.session_storage_namespace_id,
       params.frame_name,
       false,
       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);
 }
 
 GpuChannelHost* RenderThreadImpl::EstablishGpuChannelSync(
     content::CauseForGpuLaunch cause_for_gpu_launch) {
   if (gpu_channel_.get()) {
     // Do nothing if we already have a GPU channel or are already
     // establishing one.
     if (gpu_channel_->state() == GpuChannelHost::kUnconnected ||
         gpu_channel_->state() == GpuChannelHost::kConnected)
@@ skipping 78 matching lines @@
 
 scoped_refptr<base::MessageLoopProxy>
 RenderThreadImpl::GetFileThreadMessageLoopProxy() {
   DCHECK(message_loop() == MessageLoop::current());
   if (!file_thread_.get()) {
     file_thread_.reset(new base::Thread("Renderer::FILE"));
     file_thread_->Start();
   }
   return file_thread_->message_loop_proxy();
 }