Disable UKM on Android

chrome/browser/metrics/chrome_metrics_services_manager_client.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 "chrome/browser/metrics/chrome_metrics_services_manager_client.h"
 
 #include "base/command_line.h"
 #include "base/feature_list.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/task_scheduler/post_task.h"
+#include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
 #include "chrome/browser/metrics/chrome_metrics_service_client.h"
 #include "chrome/browser/metrics/variations/chrome_variations_service_client.h"
 #include "chrome/browser/metrics/variations/ui_string_overrider_factory.h"
+#include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_otr_state.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "components/metrics/enabled_state_provider.h"
 #include "components/metrics/metrics_state_manager.h"
 #include "components/prefs/pref_service.h"
 #include "components/rappor/rappor_service_impl.h"
 #include "components/variations/service/variations_service.h"
 #include "components/variations/variations_associated_data.h"
 #include "components/version_info/version_info.h"
@@ skipping 253 matching lines @@
         base::Bind(&GoogleUpdateSettings::LoadMetricsClientInfo));
   }
   return metrics_state_manager_.get();
 }
 
 bool ChromeMetricsServicesManagerClient::IsMetricsReportingForceEnabled() {
   return ChromeMetricsServiceClient::IsMetricsReportingForceEnabled();
 }
 
 bool ChromeMetricsServicesManagerClient::IsIncognitoSessionActive() {
-  return chrome::IsIncognitoSessionActive();
+#if defined(OS_ANDROID)
+  // TODO(crbug/739971) On Android, we don't get notifications from TabModel
+  // when incognito tabs are opened, so this won't get re-evaluated reliably
+  // yet.  Assume there always an incognito tab open, which will keep UKM
+  // disabled.
+  return false;

[rkaplow, 2017/07/07 13:53:41]

shouldn't this be true?

[Steven Holte, 2017/07/07 19:08:41]

Yes, fixed.

+#else
+  // Depending directly on BrowserList, since that is the implementation
+  // that we get correct notifications for.
+  return chrome::BrowserList::IsIncognitoSessionActive();
+#endif
 }