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

Side by Side Diff: components/certificate_reporting/error_report.h

Issue 2964283002: Add chrome channel to cert logger reports (Closed)
Patch Set: Revert unneeded change Created 3 years, 5 months 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_ 5 #ifndef COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_
6 #define COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_ 6 #define COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 #include "components/version_info/version_info.h"
meacer 2017/06/30 23:58:39 nit: Add a blank line between <...> and " includes
sperigo 2017/07/05 23:02:48 Done.
10 11
11 namespace base { 12 namespace base {
12 class Time; 13 class Time;
13 } // namespace base 14 } // namespace base
14 15
15 namespace network_time { 16 namespace network_time {
16 class NetworkTimeTracker; 17 class NetworkTimeTracker;
17 } // namespace network_time 18 } // namespace network_time
18 19
19 namespace net { 20 namespace net {
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 bool Serialize(std::string* output) const; 67 bool Serialize(std::string* output) const;
67 68
68 void SetInterstitialInfo(const InterstitialReason& interstitial_reason, 69 void SetInterstitialInfo(const InterstitialReason& interstitial_reason,
69 const ProceedDecision& proceed_decision, 70 const ProceedDecision& proceed_decision,
70 const Overridable& overridable, 71 const Overridable& overridable,
71 const base::Time& interstitial_time); 72 const base::Time& interstitial_time);
72 73
73 void AddNetworkTimeInfo( 74 void AddNetworkTimeInfo(
74 const network_time::NetworkTimeTracker* network_time_tracker); 75 const network_time::NetworkTimeTracker* network_time_tracker);
75 76
77 void AddChromeChannel(version_info::Channel channel);
78
76 // Sets is_retry_upload field of the protobuf to |is_retry_upload|. 79 // Sets is_retry_upload field of the protobuf to |is_retry_upload|.
77 void SetIsRetryUpload(bool is_retry_upload); 80 void SetIsRetryUpload(bool is_retry_upload);
78 81
79 // Gets the hostname to which this report corresponds. 82 // Gets the hostname to which this report corresponds.
80 const std::string& hostname() const; 83 const std::string& hostname() const;
81 84
82 // Returns true if the report has been retried. 85 // Returns true if the report has been retried.
83 bool is_retry_upload() const; 86 bool is_retry_upload() const;
84 87
85 private: 88 private:
86 std::unique_ptr<CertLoggerRequest> cert_report_; 89 std::unique_ptr<CertLoggerRequest> cert_report_;
87 }; 90 };
88 91
89 } // namespace certificate_reporting 92 } // namespace certificate_reporting
90 93
91 #endif // COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_ 94 #endif // COMPONENTS_CERTIFICATE_REPORTING_CERTIFICATE_ERROR_REPORT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698