Add chrome channel to cert logger reports

components/certificate_reporting/error_report.cc

 // Copyright 2015 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 "components/certificate_reporting/error_report.h"
 
 #include <vector>
 
 #include "base/stl_util.h"
 #include "base/strings/string_util.h"
 #include "base/time/time.h"
 #include "components/certificate_reporting/cert_logger.pb.h"
 #include "components/network_time/network_time_tracker.h"
+#include "components/version_info/version_info.h"
 #include "net/cert/cert_status_flags.h"
 #include "net/cert/x509_certificate.h"
 #include "net/ssl/ssl_info.h"
 
 #if defined(OS_ANDROID)
 #include "net/cert/cert_verify_proc_android.h"
 #endif
 
 using network_time::NetworkTimeTracker;
 
@@ skipping 143 matching lines @@
           NETWORK_TIME_FETCHES_ON_DEMAND_ONLY;
       break;
     case NetworkTimeTracker::FETCHES_IN_BACKGROUND_AND_ON_DEMAND:
       report_behavior = CertLoggerFeaturesInfo::NetworkTimeQueryingInfo::
           NETWORK_TIME_FETCHES_IN_BACKGROUND_AND_ON_DEMAND;
       break;
   }
   network_time_info->set_network_time_query_behavior(report_behavior);
 }
 
+void ErrorReport::AddChromeChannel(const version_info::Channel channel) {
+  CertLoggerRequest::ChromeChannel code;
+  switch (channel) {
+    case version_info::Channel::STABLE:
+      code = CertLoggerRequest::STABLE;

[meacer, 2017/06/30 22:24:46]

Don't forget "break;"s after each case :)

[sperigo, 2017/06/30 23:37:27]

Done.

[sperigo, 2017/06/30 23:37:27]

Oh no! I'm embarrassed. Good catch.

+
+    case version_info::Channel::BETA:
+      code = CertLoggerRequest::BETA;
+
+    case version_info::Channel::CANARY:
+      code = CertLoggerRequest::CANARY;
+
+    case version_info::Channel::DEV:
+      code = CertLoggerRequest::DEV;
+
+    case version_info::Channel::UNKNOWN:
+      code = CertLoggerRequest::UNKNOWN;
+
+    default:
+      code = CertLoggerRequest::NONE;
+  }
+  cert_report_->set_chrome_channel(code);
+}
+
 void ErrorReport::SetIsRetryUpload(bool is_retry_upload) {
   cert_report_->set_is_retry_upload(is_retry_upload);
 }
 
 const std::string& ErrorReport::hostname() const {
   return cert_report_->hostname();
 }
 
 bool ErrorReport::is_retry_upload() const {
   return cert_report_->is_retry_upload();
 }
 
 }  // namespace certificate_reporting