Return a specific error code for unknown URL schemes.

content/browser/renderer_host/resource_dispatcher_host_unittest.cc

 // Copyright (c) 2011 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 "content/browser/renderer_host/resource_dispatcher_host.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/file_path.h"
@@ skipping 1204 matching lines @@
   EXPECT_EQ(1, host_.pending_requests());
 
   // Cancelling by other methods shouldn't work either.
   host_.CancelRequestsForProcess(render_view_id);
   EXPECT_EQ(1, host_.pending_requests());
 
   // Cancelling by context should work.
   host_.CancelRequestsForContext(&filter_->resource_context());
   EXPECT_EQ(0, host_.pending_requests());
 }
+
+TEST_F(ResourceDispatcherHostTest, UnknownURLScheme) {
+  EXPECT_EQ(0, host_.pending_requests());
+
+  SetResourceType(ResourceType::MAIN_FRAME);
+  HandleScheme("http");
+
+  MakeTestRequest(0, 1, GURL("foo://bar"));

[wtc, 2011/10/04 18:45:37]

mkosiba: I don't understand why the network stack does
not return the net::ERR_UNKNOWN_URL_SCHEME error for this
URL.

Can you set breakpoints on the two return statements in
URLRequestJobManager::CreateJob() in
net/url_request/url_request_job_manager.cc, and see if
the breakpoints are hit when you run this unit test?

  if (job_factory) {
    if (!job_factory->IsHandledProtocol(scheme)) {
      return new URLRequestErrorJob(request, ERR_UNKNOWN_URL_SCHEME);
    }
  } else if (!SupportsScheme(scheme)) {
    return new URLRequestErrorJob(request, ERR_UNKNOWN_URL_SCHEME);
  }

Thank you.

[wtc, 2011/10/04 21:34:55]

I did the experiment in gdb.  I found that
net::URLRequestJobManager::CreateJob() is not called at all.
Looking at ResourceDispatcherHost::BeginRequest(), I think
this is because we don't create a net::URLRequest object in
this case.

+
+  // Flush all pending requests.
+  while (net::URLRequestTestJob::ProcessOnePendingMessage()) {}
+
+  // Sorts out all the messages we saw by request.
+  ResourceIPCAccumulator::ClassifiedMessages msgs;
+  accum_.GetClassifiedMessages(&msgs);
+
+  // We should have gotten one RequestComplete message.
+  ASSERT_EQ(1U, msgs[0].size());
+  EXPECT_EQ(ResourceMsg_RequestComplete::ID, msgs[0][0].type());
+
+  // The RequestComplete message should have had status
+  // (FAILED, ERR_UNKNOWN_URL_SCHEME).
+  int request_id;
+  net::URLRequestStatus status;
+
+  void* iter = NULL;
+  EXPECT_TRUE(IPC::ReadParam(&msgs[0][0], &iter, &request_id));
+  EXPECT_TRUE(IPC::ReadParam(&msgs[0][0], &iter, &status));
+
+  EXPECT_EQ(1, request_id);
+  EXPECT_EQ(net::URLRequestStatus::FAILED, status.status());
+  EXPECT_EQ(net::ERR_UNKNOWN_URL_SCHEME, status.error());
+}