Add new Cronet exception class for QUIC errors

net/test/url_request/url_request_failed_job.cc

 // 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.
 
 #include "net/test/url_request/url_request_failed_job.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 106 matching lines @@
   // If we have headers, get the response code from them.
   if (response_info_.headers)
     return response_info_.headers->response_code();
   return URLRequestJob::GetResponseCode();
 }
 
 void URLRequestFailedJob::GetResponseInfo(HttpResponseInfo* info) {
   *info = response_info_;
 }
 
+void URLRequestFailedJob::PopulateNetErrorDetails(
+    NetErrorDetails* details) const {
+  details->quic_connection_error = QUIC_INTERNAL_ERROR;

[pauljensen, 2016/06/17 12:05:59]

I don't think we should set |quic_connection_error| for all errors, that seems
somewhat incorrect, for example for non-QUIC requests.  How about adding "if
(net_error_ == ERR_QUIC_PROTOCOL_ERROR)"?

[mgersh, 2016/06/29 23:01:09]

Done.

+}
+
 // static
 void URLRequestFailedJob::AddUrlHandler() {
   return AddUrlHandlerForHostname(kMockHostname);
 }
 
 // static
 void URLRequestFailedJob::AddUrlHandlerForHostname(
     const std::string& hostname) {
   URLRequestFilter* filter = URLRequestFilter::GetInstance();
   // Add |hostname| to URLRequestFilter for HTTP and HTTPS.
@@ skipping 44 matching lines @@
       NotifyStartError(URLRequestStatus(URLRequestStatus::FAILED, net_error_));
       return;
     }
     return;
   }
   response_info_.headers = new net::HttpResponseHeaders("HTTP/1.1 200 OK");
   NotifyHeadersComplete();
 }
 
 }  // namespace net