Separate out IPC::Message::Sender and Channel::Listener into a separate class.

I fixed the places that did IPC::Channel::Sender (I don't even know how this compiled, but I presume this means the "Sender" base class of the "Channel" which ends up being right).

We can fix the other users of this later.

BUG=
TEST=

Review URL: https://chromiumcodereview.appspot.com/10541065

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

1 files changed, 8 insertions, 11 deletions

diff --git a/ipc/ipc_message.h b/ipc/ipc_message.h
index 2c85e42..c3d4296 100644
--- a/ipc/ipc_message.h
+++ b/ipc/ipc_message.h
@@ -12,6 +12,9 @@
 #include "base/pickle.h"
 #include "ipc/ipc_export.h"
 
+// TODO(brettw) remove this when the "typedef Sender" is removed below.
+#include "ipc/ipc_sender.h"
+
 // Ipc logging adds a dependency from the 'chrome' target on all ipc message
 // classes. In a component build, this would require exporting all message
 // classes, so don't support ipc logging in the components build.
@@ -39,17 +42,11 @@ struct LogData;
 
 class IPC_EXPORT Message : public Pickle {
  public:
-  // Implemented by objects that can send IPC messages across a channel.
-  class IPC_EXPORT Sender {
-   public:
-    virtual ~Sender() {}
-
-    // Sends the given IPC message.  The implementor takes ownership of the
-    // given Message regardless of whether or not this method succeeds.  This
-    // is done to make this method easier to use.  Returns true on success and
-    // false otherwise.
-    virtual bool Send(Message* msg) = 0;
-  };
+  // IPC::Sender used to be IPC::Message::Sender which prevented forward
+  // declarations. To keep existing code compiling, we provide this backwards-
+  // compatible definition. New code should use IPC::Sender.
+  // TODO(brettw) convert users of this and delete.
+  typedef IPC::Sender Sender;
 
   enum PriorityValue {
     PRIORITY_LOW = 1,