Allow multiple logging::LogMessage{Handler,Listener}s

remoting/client/plugin/chromoting_instance.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 "remoting/client/plugin/chromoting_instance.h"
 
 #include <nacl_io/nacl_io.h>
 #include <sys/mount.h>
 
 #include <string>
@@ skipping 1016 matching lines @@
   data->SetDouble("maxRoundtripLatency", perf_tracker_.round_trip_ms().Max());
   PostLegacyJsonMessage("onPerfStats", std::move(data));
 }
 
 // static
 void ChromotingInstance::RegisterLogMessageHandler() {
   base::AutoLock lock(g_logging_lock.Get());
 
   // Set up log message handler.
   // This is not thread-safe so we need it within our lock.
-  logging::SetLogMessageHandler(&LogToUI);
+  logging::PushLogMessageHandler(&LogToUI);

[Dan Beam, 2016/06/09 18:52:50]

does this call ever need to be balanced by a PopLogMessageHandler()?

[wychen, 2016/06/09 21:04:17]

I was a bit curious when I changed this. The handler wasn't reset to nullptr
ever, so I guess not.

 }
 
 void ChromotingInstance::RegisterLoggingInstance() {
   base::AutoLock lock(g_logging_lock.Get());
   g_logging_instance = pp_instance();
 }
 
 void ChromotingInstance::UnregisterLoggingInstance() {
   base::AutoLock lock(g_logging_lock.Get());
 
@@ skipping 70 matching lines @@
   if (is_custom_counts_histogram) {
     uma.HistogramCustomCounts(histogram_name, value, histogram_min,
                               histogram_max, histogram_buckets);
   } else {
     uma.HistogramCustomTimes(histogram_name, value, histogram_min,
                              histogram_max, histogram_buckets);
   }
 }
 
 }  // namespace remoting