Remove ChildThread dependency from Chrome. In the future we might want to expose it as an interfa...

chrome/browser/importer/external_process_importer_bridge.h

Index: chrome/browser/importer/external_process_importer_bridge.h
===================================================================
--- chrome/browser/importer/external_process_importer_bridge.h	(revision 111804)
+++ chrome/browser/importer/external_process_importer_bridge.h	(working copy)
@@ -13,13 +13,10 @@
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/importer/importer_bridge.h"
 #include "chrome/browser/importer/profile_writer.h"
+#include "ipc/ipc_message.h"

[Dirk Pranke, 2011/11/29 01:27:24]

Can't you forward-declare IPC::Message:;Sender here, and only #include
ipc_message in the .cc file?

[jam, 2011/11/29 01:46:13]

unfortunately we can't forward declare it because it's an inner class. another
reason why we shouldn't have inner classes :)
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Nested_Classes

 
 class GURL;
 
-namespace IPC {
-class Message;
-}
-
 namespace base {
 class DictionaryValue;
 }
@@ -31,8 +28,9 @@
 // profile.
 class ExternalProcessImporterBridge : public ImporterBridge {
  public:
-  explicit ExternalProcessImporterBridge(
-      const base::DictionaryValue& localized_strings);
+  ExternalProcessImporterBridge(
+      const base::DictionaryValue& localized_strings,
+      IPC::Message::Sender* sender);
 
   // Begin ImporterBridge implementation:
   virtual void AddBookmarks(
@@ -75,6 +73,8 @@
   // bundle isn't available to the external process.
   scoped_ptr<base::DictionaryValue> localized_strings_;
 
+  IPC::Message::Sender* sender_;
+
   DISALLOW_COPY_AND_ASSIGN(ExternalProcessImporterBridge);
 };