Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(259)

Side by Side Diff: chrome/browser/net/chrome_fraudulent_certificate_reporter.cc

Issue 51953002: [Net] Add a priority parameter to URLRequest's constructor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile error from rebase Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/net/chrome_fraudulent_certificate_reporter.h" 5 #include "chrome/browser/net/chrome_fraudulent_certificate_reporter.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/base64.h" 9 #include "base/base64.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "base/stl_util.h" 11 #include "base/stl_util.h"
12 #include "base/time/time.h" 12 #include "base/time/time.h"
13 #include "chrome/browser/net/cert_logger.pb.h" 13 #include "chrome/browser/net/cert_logger.pb.h"
14 #include "net/base/load_flags.h" 14 #include "net/base/load_flags.h"
15 #include "net/base/request_priority.h"
15 #include "net/base/upload_bytes_element_reader.h" 16 #include "net/base/upload_bytes_element_reader.h"
16 #include "net/base/upload_data_stream.h" 17 #include "net/base/upload_data_stream.h"
17 #include "net/cert/x509_certificate.h" 18 #include "net/cert/x509_certificate.h"
18 #include "net/ssl/ssl_info.h" 19 #include "net/ssl/ssl_info.h"
19 #include "net/url_request/url_request_context.h" 20 #include "net/url_request/url_request_context.h"
20 21
21 namespace chrome_browser_net { 22 namespace chrome_browser_net {
22 23
23 // TODO(palmer): Switch to HTTPS when the error handling delegate is more 24 // TODO(palmer): Switch to HTTPS when the error handling delegate is more
24 // sophisticated. Ultimately we plan to attempt the report on many transports. 25 // sophisticated. Ultimately we plan to attempt the report on many transports.
(...skipping 25 matching lines...) Expand all
50 for (size_t i = 0; i < pem_encoded_chain.size(); ++i) 51 for (size_t i = 0; i < pem_encoded_chain.size(); ++i)
51 *cert_chain += pem_encoded_chain[i]; 52 *cert_chain += pem_encoded_chain[i];
52 53
53 std::string out; 54 std::string out;
54 request.SerializeToString(&out); 55 request.SerializeToString(&out);
55 return out; 56 return out;
56 } 57 }
57 58
58 net::URLRequest* ChromeFraudulentCertificateReporter::CreateURLRequest( 59 net::URLRequest* ChromeFraudulentCertificateReporter::CreateURLRequest(
59 net::URLRequestContext* context) { 60 net::URLRequestContext* context) {
60 net::URLRequest* request = context->CreateRequest(upload_url_, this); 61 net::URLRequest* request =
62 context->CreateRequest(upload_url_, net::DEFAULT_PRIORITY, this);
61 request->set_load_flags(net::LOAD_DO_NOT_SEND_COOKIES | 63 request->set_load_flags(net::LOAD_DO_NOT_SEND_COOKIES |
62 net::LOAD_DO_NOT_SAVE_COOKIES); 64 net::LOAD_DO_NOT_SAVE_COOKIES);
63 return request; 65 return request;
64 } 66 }
65 67
66 void ChromeFraudulentCertificateReporter::SendReport( 68 void ChromeFraudulentCertificateReporter::SendReport(
67 const std::string& hostname, 69 const std::string& hostname,
68 const net::SSLInfo& ssl_info, 70 const net::SSLInfo& ssl_info,
69 bool sni_available) { 71 bool sni_available) {
70 // We do silent/automatic reporting ONLY for Google properties. For other 72 // We do silent/automatic reporting ONLY for Google properties. For other
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 LOG(WARNING) << "Certificate upload HTTP status: " 117 LOG(WARNING) << "Certificate upload HTTP status: "
116 << request->GetResponseCode(); 118 << request->GetResponseCode();
117 } 119 }
118 RequestComplete(request); 120 RequestComplete(request);
119 } 121 }
120 122
121 void ChromeFraudulentCertificateReporter::OnReadCompleted( 123 void ChromeFraudulentCertificateReporter::OnReadCompleted(
122 net::URLRequest* request, int bytes_read) {} 124 net::URLRequest* request, int bytes_read) {}
123 125
124 } // namespace chrome_browser_net 126 } // namespace chrome_browser_net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698