Changed when the UMA metric Ash.NumberOfVisibleWindowsInPrimaryDisplay is recorded.

ash/test/user_metrics_recorder_test_api.h

+// 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.
+
+#ifndef ASH_TEST_USER_METRICS_RECORDER_TEST_API_H_
+#define ASH_TEST_USER_METRICS_RECORDER_TEST_API_H_
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace ash {
+
+class UserMetricsRecorder;
+
+namespace test {
+
+// Test API to access internals of the UserMetricsRecorder class.
+class UserMetricsRecorderTestAPI {
+ public:
+  explicit UserMetricsRecorderTestAPI(
+      UserMetricsRecorder* user_metrics_recorder);
+  ~UserMetricsRecorderTestAPI();
+
+  // Accessor to UserMetricsRecorder::UserMetricsRecorder(bool) constructor.
+  static scoped_ptr<UserMetricsRecorder> CreateUserMetricsRecorder(
+      bool record_periodic_metrics);
+
+  // Accessor to UserMetricsRecorder::RecordPeriodicMetrics.

[pkotwicz, 2015/04/21 16:15:17]

Nit: Add "()" add the end to make it obvious that this is an accessor to a
private function

[bruthig, 2015/04/24 20:00:12]

Done.

+  void RecordPeriodicMetrics();

[pkotwicz, 2015/04/21 16:15:17]

Nit: Add "()" add the end to make it obvious that this is an accessor to a
private function

[bruthig, 2015/04/24 20:00:12]

Assuming this comment was added by mistake...

+
+  // Accessor to UserMetricsRecorder::IsUserActiveInMultiWindowEnvironment.

[pkotwicz, 2015/04/21 16:15:17]

Nit: Add "()" add the end to make it obvious that this is an accessor to a
private function

[bruthig, 2015/04/24 20:00:12]

Done.

+  bool IsUserActiveInMultiWindowEnvironment() const;
+
+ private:
+  // Not owned.
+  UserMetricsRecorder* user_metrics_recorder_;
+
+  DISALLOW_COPY_AND_ASSIGN(UserMetricsRecorderTestAPI);
+};
+
+}  // namespace test
+}  // namespace ash
+
+#endif  // ASH_TEST_USER_METRICS_RECORDER_TEST_API_H_