render_* work.

Review URL: http://codereview.chromium.org/18650

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8732 0039d316-1c4b-4281-b951-d872f2087c98

8 files changed, 172 insertions, 11 deletions

diff --git a/chrome/browser/net/dns_master.h b/chrome/browser/net/dns_master.h
index 7620d7c..f406b32 100644
--- a/chrome/browser/net/dns_master.h
+++ b/chrome/browser/net/dns_master.h
@@ -174,12 +174,17 @@ class DnsMaster {
   size_t slave_count_;  // Count of slave processes started.
   size_t running_slave_count_;  // Count of slaves process still running.
 
+  // TODO(jrg): wait for CL 15076 from _ph to come in which resolves
+  // this.  In the short term this file is hacked to be happy when
+  // included in render_process.h.
+#if defined(OS_WIN)
   // The following arrays are only initialized as
   // slave_count_ grows (up to the indicated max).
   DWORD thread_ids_[kSlaveCountMax];
   HANDLE thread_handles_[kSlaveCountMax];
   DnsSlave* slaves_[kSlaveCountMax];
-
+#endif
+  
   // shutdown_ is set to tell the slaves to terminate.
   bool shutdown_;
 
diff --git a/chrome/chrome.xcodeproj/project.pbxproj b/chrome/chrome.xcodeproj/project.pbxproj
index d466bab..e570354 100644
--- a/chrome/chrome.xcodeproj/project.pbxproj
+++ b/chrome/chrome.xcodeproj/project.pbxproj
@@ -75,6 +75,17 @@
 /* End PBXAggregateTarget section */
 
 /* Begin PBXBuildFile section */
+		3380A69B0F2E91AE004EF74F /* child_process.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFB890E9D4C9F009A6919 /* child_process.cc */; };
+		3380A69C0F2E91B1004EF74F /* child_process.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFB890E9D4C9F009A6919 /* child_process.cc */; };
+		3380A69D0F2E91D4004EF74F /* render_thread.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D640CD10EAE868600EBCFC0 /* render_thread.cc */; };
+		3380A69E0F2E91D6004EF74F /* render_thread.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D640CD10EAE868600EBCFC0 /* render_thread.cc */; };
+		3380A69F0F2E91E5004EF74F /* chrome_plugin_lib.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFB920E9D4C9F009A6919 /* chrome_plugin_lib.cc */; };
+		3380A6A00F2E91E7004EF74F /* chrome_plugin_lib.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFB920E9D4C9F009A6919 /* chrome_plugin_lib.cc */; };
+		3380A6A10F2E91F9004EF74F /* render_process.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D640CCF0EAE868600EBCFC0 /* render_process.cc */; };
+		3380A6A20F2E91FB004EF74F /* render_process.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D640CCF0EAE868600EBCFC0 /* render_process.cc */; };
+		3380A6A30F2E9207004EF74F /* ipc_sync_channel.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFBB60E9D4C9F009A6919 /* ipc_sync_channel.cc */; };
+		3380A6A40F2E9209004EF74F /* ipc_sync_channel.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D7BFBB60E9D4C9F009A6919 /* ipc_sync_channel.cc */; };
+		3380A6B60F2E9252004EF74F /* render_thread_unittest.cc in Sources */ = {isa = PBXBuildFile; fileRef = 3380A6B50F2E9252004EF74F /* render_thread_unittest.cc */; };
 		3AEA44DB19C9D93B63D7A2E4 /* url_fetcher_protect.cc in Sources */ = {isa = PBXBuildFile; fileRef = 0B7CC9C105E90E0665852528 /* url_fetcher_protect.cc */; };
 		4D1F59FE0F2A6BBB0040C1E3 /* image_diff.cc in Sources */ = {isa = PBXBuildFile; fileRef = 4D1F59FD0F2A6BBB0040C1E3 /* image_diff.cc */; };
 		4D1F5A060F2A6D170040C1E3 /* libbase.a in Frameworks */ = {isa = PBXBuildFile; fileRef = 4D7BFDC70E9D525B009A6919 /* libbase.a */; };
@@ -1084,6 +1095,8 @@
 		0B7CC9C105E90E0665852528 /* url_fetcher_protect.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = url_fetcher_protect.cc; sourceTree = "<group>"; };
 		26D97CE692D919FEB1521E43 /* ssl_error_info.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = ssl_error_info.cc; path = ssl/ssl_error_info.cc; sourceTree = "<group>"; };
 		28AA584AB2ECFB33C7C7FD8A /* template_url_parser.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = template_url_parser.cc; sourceTree = "<group>"; };
+		3380A6B50F2E9252004EF74F /* render_thread_unittest.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = render_thread_unittest.cc; sourceTree = "<group>"; };
+		3380A6B90F2E9275004EF74F /* render_view_unittest.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = render_view_unittest.cc; sourceTree = "<group>"; };
 		3CCF8AA8A56FF8FE59F0C299 /* template_url.cc */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = template_url.cc; sourceTree = "<group>"; };
 		433B6EFB7A1D931A13C9556F /* url_fetcher_protect.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = url_fetcher_protect.h; sourceTree = "<group>"; };
 		4D1F59EA0F2A6B590040C1E3 /* image_diff */ = {isa = PBXFileReference; explicitFileType = "compiled.mach-o.executable"; includeInIndex = 0; path = image_diff; sourceTree = BUILT_PRODUCTS_DIR; };
@@ -2128,8 +2141,10 @@
 				4D640CCF0EAE868600EBCFC0 /* render_process.cc */,
 				4D640CD00EAE868600EBCFC0 /* render_process.h */,
 				4D640CD10EAE868600EBCFC0 /* render_thread.cc */,
+				3380A6B50F2E9252004EF74F /* render_thread_unittest.cc */,
 				4D640CD20EAE868600EBCFC0 /* render_thread.h */,
 				4D640CD30EAE868600EBCFC0 /* render_view.cc */,
+				3380A6B90F2E9275004EF74F /* render_view_unittest.cc */,
 				4D640CD40EAE868600EBCFC0 /* render_view.h */,
 				4D640CD50EAE868600EBCFC0 /* render_widget.cc */,
 				4D640CD60EAE868600EBCFC0 /* render_widget.h */,
@@ -4065,6 +4080,11 @@
 				E45076CF0F153871003BE099 /* localized_error.cc in Sources */,
 				4D640CF40EAE86EA00EBCFC0 /* render_dns_queue.cc in Sources */,
 				4D640CF50EAE86EF00EBCFC0 /* visitedlink_slave.cc in Sources */,
+				3380A69B0F2E91AE004EF74F /* child_process.cc in Sources */,
+				3380A69D0F2E91D4004EF74F /* render_thread.cc in Sources */,
+				3380A69F0F2E91E5004EF74F /* chrome_plugin_lib.cc in Sources */,
+				3380A6A10F2E91F9004EF74F /* render_process.cc in Sources */,
+				3380A6A30F2E9207004EF74F /* ipc_sync_channel.cc in Sources */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;
 		};
@@ -4209,6 +4229,12 @@
 				B502DA280F098056005BE90C /* visit_database_unittest.cc in Sources */,
 				4D7BFB420E9D4C35009A6919 /* visit_tracker_unittest.cc in Sources */,
 				E46C4B4C0F21098F00B393B8 /* worker_thread_ticker_unittest.cc in Sources */,
+				3380A69C0F2E91B1004EF74F /* child_process.cc in Sources */,
+				3380A69E0F2E91D6004EF74F /* render_thread.cc in Sources */,
+				3380A6A00F2E91E7004EF74F /* chrome_plugin_lib.cc in Sources */,
+				3380A6A20F2E91FB004EF74F /* render_process.cc in Sources */,
+				3380A6A40F2E9209004EF74F /* ipc_sync_channel.cc in Sources */,
+				3380A6B60F2E9252004EF74F /* render_thread_unittest.cc in Sources */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;
 		};
diff --git a/chrome/common/chrome_plugin_lib.cc b/chrome/common/chrome_plugin_lib.cc
index 1587a2d..a462adc 100644
--- a/chrome/common/chrome_plugin_lib.cc
+++ b/chrome/common/chrome_plugin_lib.cc
@@ -10,7 +10,10 @@
 #include "base/message_loop.h"
 #include "base/path_service.h"
 #include "base/perftimer.h"
+#include "base/thread.h"
+#if defined(OS_WIN)
 #include "base/registry.h"
+#endif
 #include "base/string_util.h"
 #include "chrome/common/chrome_counters.h"
 #include "chrome/common/chrome_switches.h"
@@ -20,10 +23,13 @@
 
 using base::TimeDelta;
 
+// TODO(port): revisit when plugins happier
+#if defined(OS_WIN)
 const TCHAR ChromePluginLib::kRegistryChromePlugins[] =
     _T("Software\\Google\\Chrome\\Plugins");
 static const TCHAR kRegistryLoadOnStartup[] = _T("LoadOnStartup");
 static const TCHAR kRegistryPath[] = _T("Path");
+#endif
 
 typedef base::hash_map<FilePath, scoped_refptr<ChromePluginLib> >
     PluginMap;
@@ -33,7 +39,7 @@ static PluginMap* g_loaded_libs;
 
 // The thread plugins are loaded and used in, lazily initialized upon
 // the first creation call.
-static DWORD g_plugin_thread_id = 0;
+static PlatformThreadId g_plugin_thread_id = 0;
 static MessageLoop* g_plugin_thread_loop = NULL;
 
 #ifdef GEARS_STATIC_LIB
@@ -58,7 +64,7 @@ ChromePluginLib* ChromePluginLib::Create(const FilePath& filename,
   // Keep a map of loaded plugins to ensure we only load each library once.
   if (!g_loaded_libs) {
     g_loaded_libs = new PluginMap();
-    g_plugin_thread_id = ::GetCurrentThreadId();
+    g_plugin_thread_id = PlatformThread::CurrentId();
     g_plugin_thread_loop = MessageLoop::current();
   }
   DCHECK(IsPluginThread());
@@ -97,7 +103,7 @@ void ChromePluginLib::Destroy(const FilePath& filename) {
 
 // static
 bool ChromePluginLib::IsPluginThread() {
-  return ::GetCurrentThreadId() == g_plugin_thread_id;
+  return PlatformThread::CurrentId() == g_plugin_thread_id;
 }
 
 // static
@@ -116,7 +122,12 @@ void ChromePluginLib::RegisterPluginsWithNPAPI() {
     return;
   // Note: we can only access the NPAPI list because the PluginService has done
   // the locking for us.  We should not touch it anywhere else.
+#if defined(OS_WIN)
   NPAPI::PluginList::AddExtraPluginPath(path);
+#else
+  // TODO(port): plugins not yet implemented
+  NOTIMPLEMENTED();
+#endif
 }
 
 static void LogPluginLoadTime(const TimeDelta &time) {
@@ -186,7 +197,9 @@ const CPPluginFuncs& ChromePluginLib::functions() const {
 
 ChromePluginLib::ChromePluginLib(const FilePath& filename)
     : filename_(filename),
+#if defined(OS_WIN)
       module_(0),
+#endif
       initialized_(false),
       CP_VersionNegotiate_(NULL),
       CP_Initialize_(NULL) {
@@ -235,6 +248,11 @@ int ChromePluginLib::CP_Test(void* param) {
 }
 
 bool ChromePluginLib::Load() {
+#if !defined(OS_WIN)
+  // TODO(port): plugins not yet implemented
+  NOTIMPLEMENTED();
+  return false;
+#else
   DCHECK(module_ == 0);
 #ifdef GEARS_STATIC_LIB
   FilePath path;
@@ -267,6 +285,7 @@ bool ChromePluginLib::Load() {
       (GetProcAddress(module_, "CP_Test"));
 
   return true;
+#endif
 }
 
 void ChromePluginLib::Unload() {
@@ -278,8 +297,10 @@ void ChromePluginLib::Unload() {
   if (initialized_)
     CP_Shutdown();
 
+#if defined(OS_WIN)
   if (module_) {
     FreeLibrary(module_);
     module_ = 0;
   }
+#endif
 }
diff --git a/chrome/renderer/render_process.cc b/chrome/renderer/render_process.cc
index 69b56b2..648c8a8 100644
--- a/chrome/renderer/render_process.cc
+++ b/chrome/renderer/render_process.cc
@@ -72,12 +72,20 @@ bool RenderProcess::GlobalInit(const std::wstring &channel_name) {
 
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
   if (command_line.HasSwitch(switches::kJavaScriptFlags)) {
+  // TODO(port)
+  // Try and limit what we pull in for our non-Win unit test bundle
+#if defined(OS_WIN) || (!defined(UNIT_TEST))
     webkit_glue::SetJavaScriptFlags(
       command_line.GetSwitchValue(switches::kJavaScriptFlags));
+#endif
   }
   if (command_line.HasSwitch(switches::kPlaybackMode) ||
       command_line.HasSwitch(switches::kRecordMode)) {
+  // TODO(port)
+  // Try and limit what we pull in for our non-Win unit test bundle
+#if defined(OS_WIN) || (!defined(UNIT_TEST))
       webkit_glue::SetRecordPlaybackMode(true);
+#endif
   }
 
   if (command_line.HasSwitch(switches::kInProcessPlugins) ||
@@ -211,8 +219,12 @@ void RenderProcess::ScheduleCacheClearer() {
 }
 
 void RenderProcess::Cleanup() {
+  // TODO(port)
+  // Try and limit what we pull in for our non-Win unit test bundle
+#if defined(OS_WIN) || (!defined(UNIT_TEST))
 #ifndef NDEBUG
   // log important leaked objects
   webkit_glue::CheckForLeaks();
 #endif
+#endif
 }
diff --git a/chrome/renderer/render_thread.cc b/chrome/renderer/render_thread.cc
index 08ccd27..d39800f 100644
--- a/chrome/renderer/render_thread.cc
+++ b/chrome/renderer/render_thread.cc
@@ -2,7 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#if defined(OS_WIN)
 #include <windows.h>
+#endif
 #include <algorithm>
 
 #include "chrome/renderer/render_thread.h"
@@ -11,7 +13,15 @@
 #include "chrome/common/chrome_plugin_lib.h"
 #include "chrome/common/ipc_logging.h"
 #include "chrome/common/notification_service.h"
+// TODO(port)
+#if defined(OS_WIN)
 #include "chrome/plugin/plugin_channel.h"
+#else
+#include <vector>
+#include "base/scoped_handle.h"
+#include "chrome/plugin/plugin_channel_base.h"
+#include "webkit/glue/weburlrequest.h"
+#endif
 #include "chrome/renderer/net/render_dns_master.h"
 #include "chrome/renderer/render_process.h"
 #include "chrome/renderer/render_view.h"
@@ -19,6 +29,7 @@
 #include "chrome/renderer/visitedlink_slave.h"
 #include "webkit/glue/cache_manager.h"
 
+
 RenderThread* g_render_thread;
 
 static const unsigned int kCacheStatsDelayMS = 2000 /* milliseconds */;
@@ -31,8 +42,8 @@ static const size_t kStackSize = 1024 * 1024;
 
 RenderThread::RenderThread(const std::wstring& channel_name)
     : Thread("Chrome_RenderThread"),
-      channel_name_(channel_name),
       owner_loop_(MessageLoop::current()),
+      channel_name_(channel_name),
       visited_link_slave_(NULL),
       user_script_slave_(NULL),
       render_dns_master_(NULL),
@@ -71,7 +82,12 @@ void RenderThread::RemoveFilter(IPC::ChannelProxy::MessageFilter* filter) {
 }
 
 void RenderThread::Resolve(const char* name, size_t length) {
+// TODO(port)
+#if defined(OS_WIN)
   return render_dns_master_->Resolve(name, length);
+#else
+  NOTIMPLEMENTED();
+#endif
 }
 
 void RenderThread::AddRoute(int32 routing_id,
@@ -101,13 +117,20 @@ void RenderThread::Init() {
       IPC::Channel::MODE_CLIENT, this, NULL, owner_loop_, true,
       RenderProcess::GetShutDownEvent()));
 
+#if defined(OS_WIN)
   // The renderer thread should wind-up COM.
   CoInitialize(0);
+#endif
 
   visited_link_slave_ = new VisitedLinkSlave();
   user_script_slave_ = new UserScriptSlave();
 
+// TODO(port)
+#if defined(OS_WIN)
   render_dns_master_.reset(new RenderDnsMaster());
+#else
+  NOTIMPLEMENTED();
+#endif
 
 #ifdef IPC_MESSAGE_LOG_ENABLED
   IPC::Logging::current()->SetIPCSender(this);
@@ -122,8 +145,11 @@ void RenderThread::CleanUp() {
   // it caches a pointer to this thread.
   channel_.reset();
 
+// TODO(port)
+#if defined(OS_WIN)
   // Clean up plugin channels before this thread goes away.
   PluginChannelBase::CleanupChannels();
+#endif
 
 #ifdef IPC_MESSAGE_LOG_ENABLED
   IPC::Logging::current()->SetIPCSender(NULL);
@@ -137,7 +163,9 @@ void RenderThread::CleanUp() {
   delete user_script_slave_;
   user_script_slave_ = NULL;
 
+#if defined(OS_WIN)
   CoUninitialize();
+#endif
 }
 
 void RenderThread::OnUpdateVisitedLinks(base::SharedMemoryHandle table) {
@@ -160,7 +188,11 @@ void RenderThread::OnMessageReceived(const IPC::Message& msg) {
     IPC_BEGIN_MESSAGE_MAP(RenderThread, msg)
       IPC_MESSAGE_HANDLER(ViewMsg_VisitedLink_NewTable, OnUpdateVisitedLinks)
       IPC_MESSAGE_HANDLER(ViewMsg_SetNextPageID, OnSetNextPageID)
+      // TODO(port): removed from render_messages_internal.h;
+      // is there a new non-windows message I should add here?
+#if defined(OS_WIN)
       IPC_MESSAGE_HANDLER(ViewMsg_New, OnCreateNewView)
+#endif
       IPC_MESSAGE_HANDLER(ViewMsg_SetCacheCapacities, OnSetCacheCapacities)
       IPC_MESSAGE_HANDLER(ViewMsg_GetCacheResourceStats,
                           OnGetCacheResourceStats)
@@ -192,9 +224,14 @@ void RenderThread::OnPluginMessage(const FilePath& plugin_path,
 void RenderThread::OnSetNextPageID(int32 next_page_id) {
   // This should only be called at process initialization time, so we shouldn't
   // have to worry about thread-safety.
+  // TODO(port)
+#if defined(OS_WIN)
   RenderView::SetNextPageID(next_page_id);
+#endif
 }
 
+// TODO(port)
+#if defined(OS_WIN)
 void RenderThread::OnCreateNewView(HWND parent_hwnd,
                                    HANDLE modal_dialog_event,
                                    const WebPreferences& webkit_prefs,
@@ -207,23 +244,39 @@ void RenderThread::OnCreateNewView(HWND parent_hwnd,
       this, parent_hwnd, waitable_event, MSG_ROUTING_NONE, webkit_prefs,
       new SharedRenderViewCounter(0), view_id);
 }
+#endif
 
 void RenderThread::OnSetCacheCapacities(size_t min_dead_capacity,
                                         size_t max_dead_capacity,
                                         size_t capacity) {
+#if defined(OS_WIN)
   CacheManager::SetCapacities(min_dead_capacity, max_dead_capacity, capacity);
+#else
+  // TODO(port)
+  NOTIMPLEMENTED();
+#endif
 }
 
 void RenderThread::OnGetCacheResourceStats() {
+#if defined(OS_WIN)
   CacheManager::ResourceTypeStats stats;
   CacheManager::GetResourceTypeStats(&stats);
   Send(new ViewHostMsg_ResourceTypeStats(stats));
+#else
+  // TODO(port)
+  NOTIMPLEMENTED();
+#endif
 }
 
 void RenderThread::InformHostOfCacheStats() {
+#if defined(OS_WIN)
   CacheManager::UsageStats stats;
   CacheManager::GetUsageStats(&stats);
   Send(new ViewHostMsg_UpdatedCacheStats(stats));
+#else
+  // TODO(port)
+  NOTIMPLEMENTED();
+#endif
 }
 
 void RenderThread::InformHostOfCacheStatsLater() {
diff --git a/chrome/renderer/render_thread_unittest.cc b/chrome/renderer/render_thread_unittest.cc
new file mode 100644
index 0000000..0aadda6
--- /dev/null
+++ b/chrome/renderer/render_thread_unittest.cc
@@ -0,0 +1,47 @@
+// Copyright (c) 2008 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 "base/string_util.h"
+#include "base/waitable_event.h"
+#include "chrome/common/render_messages.h"
+#include "chrome/renderer/render_thread.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+
+class RenderThreadTest : public testing::Test {
+ public:
+  virtual void SetUp() {
+    // Must have a message loop to create a RenderThread()
+    message_loop_ = new MessageLoop(MessageLoop::TYPE_DEFAULT);
+  }
+  
+  virtual void TearDown() {
+    delete message_loop_;
+  }
+
+ private:
+  MessageLoop *message_loop_;
+  
+};
+
+
+TEST_F(RenderThreadTest, TestGlobal) {
+  ASSERT_FALSE(g_render_thread);
+  {
+    RenderThread thread(EmptyWString());
+    ASSERT_TRUE(g_render_thread);
+  }
+  ASSERT_FALSE(g_render_thread);
+}
+
+TEST_F(RenderThreadTest, TestVisitedMsg) {
+  RenderThread thread(EmptyWString());
+  IPC::Message* msg = new ViewMsg_VisitedLink_NewTable(NULL);
+  ASSERT_TRUE(msg);
+  // Message goes nowhere, but this confirms Init() has happened.
+  // Unusually (?), RenderThread() Start()s itself in it's constructor.
+  thread.Send(msg);
+  delete msg;
+}
+
diff --git a/chrome/renderer/render_view_unittest.cc b/chrome/renderer/render_view_unittest.cc
index ffb61ad..c3b2fd3 100644
--- a/chrome/renderer/render_view_unittest.cc
+++ b/chrome/renderer/render_view_unittest.cc
@@ -16,6 +16,8 @@ namespace {
 const int32 kRouteId = 5;
 const int32 kOpenerId = 7;
 
+};
+
 class RenderViewTest : public testing::Test {
  public:
   RenderViewTest() {}
@@ -86,7 +88,6 @@ class RenderViewTest : public testing::Test {
   scoped_refptr<RenderView> view_;
 };
 
-}  // namespace
 
 TEST_F(RenderViewTest, OnLoadAlternateHTMLText) {
   // Test a new navigation.
@@ -96,7 +97,7 @@ TEST_F(RenderViewTest, OnLoadAlternateHTMLText) {
 
   // We should have gotten two different types of start messages in the
   // following order.
-  ASSERT_EQ(2, render_thread_.sink().message_count());
+  ASSERT_EQ((size_t)2, render_thread_.sink().message_count());
   const IPC::Message* msg = render_thread_.sink().GetMessageAt(0);
   EXPECT_EQ(ViewHostMsg_DidStartLoading::ID, msg->type());
 
diff --git a/chrome/renderer/renderer.scons b/chrome/renderer/renderer.scons
index 9911a02..30b5b0e 100644
--- a/chrome/renderer/renderer.scons
+++ b/chrome/renderer/renderer.scons
@@ -104,8 +104,6 @@ if env.Bit('linux'):
       'external_js_object.cc',
       'net/render_dns_master.cc',
       'plugin_channel_host.cc',
-      'render_process.cc',
-      'render_thread.cc',
       'render_view.cc',
       'render_widget.cc',
       'renderer_main.cc',
@@ -120,8 +118,6 @@ if env.Bit('mac'):
       'external_js_object.cc',
       'net/render_dns_master.cc',
       'plugin_channel_host.cc',
-      'render_process.cc',
-      'render_thread.cc',
       'render_view.cc',
       'render_widget.cc',
       'renderer_glue.cc',