diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-02 19:16:07 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-02 19:16:07 +0000 |
| commit | 4b580bf3020b1e0eaf5b7efad50896b4c62474c5 (patch) | |
| tree | 94f893bf3422dccda5e4bc05e4f8b8cca0d62638 /chrome/test/automation | |
| parent | e669998f4edaa156aafbe2fb93b3e3dae1ebd06c (diff) | |
| download | chromium_src-4b580bf3020b1e0eaf5b7efad50896b4c62474c5.zip chromium_src-4b580bf3020b1e0eaf5b7efad50896b4c62474c5.tar.gz chromium_src-4b580bf3020b1e0eaf5b7efad50896b4c62474c5.tar.bz2 | |
Add a base class for objects that want to filter messages on the IO thread. I'll switch the filters to it in future separate changes.
I've also taken out the special case for an initial filter from the IPC classes. The reason it existed was that there was a race condition of some messages not being filtered if a filter is added after construction but before launching the peer process. Taking it out allows us to add more than one filter and makes things a little cleaner.
Review URL: http://codereview.chromium.org/5513001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68043 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/automation')
| -rw-r--r-- | chrome/test/automation/automation_proxy.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/test/automation/automation_proxy.cc b/chrome/test/automation/automation_proxy.cc index a33f095..cd5d9af 100644 --- a/chrome/test/automation/automation_proxy.cc +++ b/chrome/test/automation/automation_proxy.cc @@ -157,10 +157,10 @@ void AutomationProxy::InitializeChannel(const std::string& channel_id, use_named_interface ? IPC::Channel::MODE_NAMED_CLIENT : IPC::Channel::MODE_SERVER, this, // we are the listener - new AutomationMessageFilter(this), thread_->message_loop(), true, shutdown_event_.get())); + channel_->AddFilter(new AutomationMessageFilter(this)); } void AutomationProxy::InitializeHandleTracker() { |