Removed static factories for data, ftp, file, and about jobs.

net/url_request/url_request_http_job.cc

Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index f763bcfae6d4c64339faa106aa4a71ba6b4fea68..d845ce4fc685fc8cef2541a94e6f2b3664921d63 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -41,6 +41,7 @@
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_error_job.h"
+#include "net/url_request/url_request_job_factory.h"
 #include "net/url_request/url_request_redirect_job.h"
 #include "net/url_request/url_request_throttler_header_adapter.h"
 #include "net/url_request/url_request_throttler_manager.h"
@@ -1024,8 +1025,10 @@ bool URLRequestHttpJob::IsSafeRedirect(const GURL& location) {
   // restrict redirects to externally handled protocols.  Our consumer would
   // need to take care of those.
 
-  if (!URLRequest::IsHandledURL(location))
+  if (!(URLRequest::IsHandledURL(location) ||
+        request_->context()->job_factory()->IsHandledURL(location))) {

[mmenke, 2012/08/24 14:23:45]

nit:  Suggest

!URLRequest::IsHandledURL(location) &&
!request_->context()->job_factory()->IsHandledURL(location)

To make it more readable.

     return true;
+  }
 
   static const char* kSafeSchemes[] = {
     "http",