Make BrowserMessageFilter not derive from IPC::ChannelProxy::MessageFilter. This allows us to hide …

chrome/browser/media/chrome_webrtc_browsertest.cc

 // Copyright 2013 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/command_line.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/process/launch.h"
 #include "base/process/process_metrics.h"
 #include "base/strings/string_split.h"
@@ skipping 439 matching lines @@
   // the tab is closed. We must ensure that WebRtcLogUploader::UploadLog, which
   // runs on the FILE thread, has finished after closing the tab before
   // continuing.
   // 1. Add a filter which just acts as a listener. It's added after
   //    WebRtcLoggingHandlerHost, which is the filter that posts a task for
   //    WebRtcLogUploader::UploadLog. So it's guarantueed to be removed after
   //    WebRtcLoggingHandlerHost.
   // 2. When the filter goes away post a task on the file thread to signal the
   //    event.
   base::WaitableEvent ipc_channel_closed(false, false);
-  left_tab->GetRenderProcessHost()->GetChannel()->AddFilter(
+  left_tab->GetRenderProcessHost()->AddFilter(
       new BrowserMessageFilter(base::Bind(
           &base::WaitableEvent::Signal,
           base::Unretained(&ipc_channel_closed))));
   chrome::CloseWebContents(browser(), left_tab, false);
   ASSERT_TRUE(ipc_channel_closed.TimedWait(TestTimeouts::action_timeout()));
 
   const char boundary[] = "------**--yradnuoBgoLtrapitluMklaTelgooG--**----";
 
   // Remove the compressed data, it may contain "\r\n". Just verify that its
   // size is > 0.
@@ skipping 60 matching lines @@
                multipart_lines[26].c_str());
   EXPECT_TRUE(multipart_lines[27].empty());
   EXPECT_TRUE(multipart_lines[28].empty());  // The removed zip part.
   std::string final_delimiter = boundary;
   final_delimiter += "--";
   EXPECT_STREQ(final_delimiter.c_str(), multipart_lines[29].c_str());
   EXPECT_TRUE(multipart_lines[30].empty());
 
   ASSERT_TRUE(peerconnection_server_.Stop());
 }