Fix recursion bug in blocking a fetch and add a test

headless/public/util/generic_url_request_job.cc

 // Copyright 2016 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 "headless/public/util/generic_url_request_job.h"
 
 #include <string.h>
 #include <algorithm>
 
 #include "base/logging.h"
@@ skipping 108 matching lines @@
 
   if (matched_resource) {
     OnFetchCompleteExtractHeaders(
         matched_resource->final_url, matched_resource->http_response_code,
         matched_resource->response_data, matched_resource->response_data_size);
   } else {
     url_fetcher_->StartFetch(rewritten_url, extra_request_headers_, this);
   }
 }
 
-void GenericURLRequestJob::OnStartError(net::Error error) {
-  DispatchStartError(error);
+void GenericURLRequestJob::OnFetchStartError(net::Error error) {
+  OnStartError(error);
 }
 
 void GenericURLRequestJob::OnFetchComplete(
     const GURL& final_url,
     int http_response_code,
     scoped_refptr<net::HttpResponseHeaders> response_headers,
     const char* body,
     size_t body_size) {
   http_response_code_ = http_response_code;
   response_headers_ = response_headers;
@@ skipping 35 matching lines @@
   return response_headers_->GetMimeType(mime_type);
 }
 
 bool GenericURLRequestJob::GetCharset(std::string* charset) {
   if (!response_headers_)
     return false;
   return response_headers_->GetCharset(charset);
 }
 
 }  // namespace headless