Updating GetChannel in BlimpMetricsServiceClient so metrics will display

blimp/engine/app/blimp_metrics_service_client.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 "blimp/engine/app/blimp_metrics_service_client.h"
 
 #include "base/bind.h"
 #include "base/i18n/rtl.h"
 #include "base/lazy_instance.h"
 #include "base/memory/ptr_util.h"
@@ skipping 100 matching lines @@
 std::string BlimpMetricsServiceClient::GetApplicationLocale() {
   return base::i18n::GetConfiguredLocale();
 }
 
 bool BlimpMetricsServiceClient::GetBrand(std::string* brand_code) {
   // Blimp doesn't use brand codes.
   return false;
 }
 
 metrics::SystemProfileProto::Channel BlimpMetricsServiceClient::GetChannel() {
-  // Blimp engine does not have channel info yet.
-  return metrics::SystemProfileProto::CHANNEL_UNKNOWN;
+  // Blimp engine does not have channel info yet, however metrics data with
+  // CHANNEL_UNKNOWN is filtered in metrics visualization tools since the value
+  // typically implies a non-official build.
+  // Using CHANNEL_CANARY as a work around until channel is set here.
+  return metrics::SystemProfileProto::CHANNEL_CANARY;
 }
 
 std::string BlimpMetricsServiceClient::GetVersionString() {
   return version_info::GetVersionNumber();
 }
 
 void BlimpMetricsServiceClient::OnLogUploadComplete() {}
 
 void BlimpMetricsServiceClient::InitializeSystemProfileMetrics(
     const base::Closure& done_callback) {
@@ skipping 26 matching lines @@
 BlimpMetricsServiceClient::GetMetricsReportingDefaultState() {
   return metrics::EnableMetricsDefault::OPT_IN;
 }
 
 bool BlimpMetricsServiceClient::IsConsentGiven() {
   return true;
 }
 
 }  // namespace engine
 }  // namespace blimp