dbus: Stop collecting DBus.RejectedSignalCount histogram

dbus/signal_sender_verification_unittest.cc

 // Copyright (c) 2012 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 <memory>
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/histogram_samples.h"
@@ skipping 179 matching lines @@
 TEST_F(SignalSenderVerificationTest, TestSignalAccepted) {
   const char kMessage[] = "hello, world";
   // Send the test signal from the exported object.
   test_service_->SendTestSignal(kMessage);
   // Receive the signal with the object proxy. The signal is handled in
   // SignalSenderVerificationTest::OnTestSignal() in the main thread.
   WaitForTestSignal();
   ASSERT_EQ(kMessage, test_signal_string_);
 }
 
-// Disabled, http://crbug.com/407063 .
-TEST_F(SignalSenderVerificationTest, DISABLED_TestSignalRejected) {
-  // To make sure the histogram instance is created.
-  UMA_HISTOGRAM_COUNTS("DBus.RejectedSignalCount", 0);
-  base::HistogramBase* reject_signal_histogram =
-        base::StatisticsRecorder::FindHistogram("DBus.RejectedSignalCount");
-  std::unique_ptr<base::HistogramSamples> samples1(
-      reject_signal_histogram->SnapshotSamples());
-
+TEST_F(SignalSenderVerificationTest, TestSignalRejected) {
   const char kNewMessage[] = "hello, new world";
   test_service2_->SendTestSignal(kNewMessage);
 
   // This test tests that our callback is NOT called by the ObjectProxy.
   // Sleep to have message delivered to the client via the D-Bus service.
-  base::PlatformThread::Sleep(TestTimeouts::action_timeout());
-
-  std::unique_ptr<base::HistogramSamples> samples2(
-      reject_signal_histogram->SnapshotSamples());
+  base::PlatformThread::Sleep(TestTimeouts::tiny_timeout());
 
   ASSERT_EQ("", test_signal_string_);
-  EXPECT_EQ(samples1->TotalCount() + 1, samples2->TotalCount());
 }
 
 TEST_F(SignalSenderVerificationTest, TestOwnerChanged) {
   const char kMessage[] = "hello, world";
 
   // Send the test signal from the exported object.
   test_service_->SendTestSignal(kMessage);
   // Receive the signal with the object proxy. The signal is handled in
   // SignalSenderVerificationTest::OnTestSignal() in the main thread.
   WaitForTestSignal();
@@ skipping 167 matching lines @@
 
   // Now the second service owns the name.
   const char kNewMessage[] = "hello, new world";
 
   test_service2_->SendTestSignal(kNewMessage);
   WaitForTestSignal();
   ASSERT_EQ(kNewMessage, test_signal_string_);
 }
 
 }  // namespace dbus