Change Interceptors into URLRequestJobFactory::ProtocolHandlers.

webkit/tools/test_shell/test_shell.cc

Index: webkit/tools/test_shell/test_shell.cc
diff --git a/webkit/tools/test_shell/test_shell.cc b/webkit/tools/test_shell/test_shell.cc
index 70954bfba3e969ad955eb9a25eff29767a6379e6..91dd2e9b3f060d6d006b32caa228545fb3a1cd90 100644
--- a/webkit/tools/test_shell/test_shell.cc
+++ b/webkit/tools/test_shell/test_shell.cc
@@ -169,6 +169,9 @@ TestShell::~TestShell() {
     LoadURL(GURL("about:blank"));
   }
 
+  net::URLRequestFilter* filter = net::URLRequestFilter::GetInstance();

[willchan no longer on Chromium, 2012/12/11 19:23:34]

Whoa! What's this global here? I've never seen this in all my years on the team.
Let's kill this (I'm not saying you have to do this in this CL, just remarking
this so our team can keep it in mind).

[pauljensen, 2012/12/11 21:22:55]

Ya, not only is it an icky global singleton but it also uses  the
URLRequest::Deprecated::RegisterProtocolFactory() interface...  Erik and I had
already planned on changing it to be a ProtocolHandler or something similar in
the long-term of this refactoring project; a change which would likely
necessitate it not being a global singleton.

+  filter->RemoveHostnameHandler("test-shell-resource", "inspector");
+
   // Call GC twice to clean up garbage.
   CallJSGC();
   CallJSGC();