Remove unused pieces of URLRequestJobFactory API.

net/url_request/url_request.h

 // Copyright (c) 2012 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.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_H_
 #define NET_URL_REQUEST_URL_REQUEST_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 15 matching lines @@
 #include "net/base/request_priority.h"
 #include "net/base/upload_progress.h"
 #include "net/cookies/canonical_cookie.h"
 #include "net/http/http_request_headers.h"
 #include "net/http/http_response_info.h"
 #include "net/url_request/url_request_status.h"
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 class ChildProcessSecurityPolicyTest;
 class ComponentUpdateInterceptor;

[mmenke, 2013/02/11 17:19:32]

Don't believe this is needed any more.

 class TestAutomationProvider;
 class URLRequestAutomationJob;
 
 namespace base {
 namespace debug {
 class StackTrace;
 }
 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace appcache {
 class AppCacheInterceptor;
 class AppCacheRequestHandlerTest;
 class AppCacheURLRequestJobTest;
 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace content {
 class ResourceDispatcherHostTest;
 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace extensions {
 class AutoUpdateInterceptor;

[mmenke, 2013/02/11 17:19:32]

We can remove this, too.  Think we can remove the UserScriptListenerTest as
well.

 class UserScriptListenerTest;
 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace fileapi {
 class FileSystemDirURLRequestJobTest;
 class FileSystemURLRequestJobTest;
 class FileWriterDelegateTest;
 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace policy {
 class CannedResponseInterceptor;

[mmenke, 2013/02/11 17:19:32]

This doesn't seem to be used any more, either.

 }
 
 // Temporary layering violation to allow existing users of a deprecated
 // interface.
 namespace webkit_blob {
 class BlobURLRequestJobTest;
 }
 
 namespace net {
 
@@ skipping 110 matching lines @@
     friend class content::ResourceDispatcherHostTest;
     friend class fileapi::FileSystemDirURLRequestJobTest;
     friend class fileapi::FileSystemURLRequestJobTest;
     friend class fileapi::FileWriterDelegateTest;
     friend class webkit_blob::BlobURLRequestJobTest;
 
     // Use URLRequestJobFactory::ProtocolHandler instead.
     static ProtocolFactory* RegisterProtocolFactory(const std::string& scheme,
                                                     ProtocolFactory* factory);
 
-    // Use URLRequestJobFactory::Interceptor instead.
+    // TODO(pauljensen): Remove this when AppCacheInterceptor is a
+    // ProtocolHandler, see crbug.com/161547.
     static void RegisterRequestInterceptor(Interceptor* interceptor);
     static void UnregisterRequestInterceptor(Interceptor* interceptor);
 
     DISALLOW_IMPLICIT_CONSTRUCTORS(Deprecated);
   };
 
   // The delegate's methods are called from the message loop of the thread
   // on which the request's Start() method is called. See above for the
   // ordering of callbacks.
   //
@@ skipping 633 matching lines @@
   LoadTimingInfo load_timing_info_;
 
   scoped_ptr<const base::debug::StackTrace> stack_trace_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequest);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_H_