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

content/browser/loader/resource_dispatcher_host_unittest.cc

Index: content/browser/loader/resource_dispatcher_host_unittest.cc
===================================================================
--- content/browser/loader/resource_dispatcher_host_unittest.cc	(revision 225900)
+++ content/browser/loader/resource_dispatcher_host_unittest.cc	(working copy)
@@ -176,7 +176,7 @@
                    base::Unretained(this))),
       dest_(dest),
       resource_context_(resource_context) {
-    OnChannelConnected(base::GetCurrentProcId());
+    set_peer_pid_for_testing(base::GetCurrentProcId());
   }
 
   // ResourceMessageFilter override
@@ -1804,9 +1804,6 @@
 
   // Make sure we don't hold onto the ResourceMessageFilter after it is deleted.
   GlobalRequestID first_global_request_id(first_child_id, request_id);
-  const ResourceRequestInfoImpl* info = ResourceRequestInfoImpl::ForRequest(
-      host_.GetURLRequest(first_global_request_id));
-  EXPECT_FALSE(info->filter());
 
   // This second filter is used to emulate a second process.
   scoped_refptr<ForwardingFilter> second_filter = new ForwardingFilter(