Revert "MediaStreamManager: listen to and log OnThermalStateChange"

This reverts commit 9d9ec179.

Reason for revert: https://crbug.com/1087096 -- crash in OnThermalStateChange on Mac ASAN GPU bots.

Original change's description:
> MediaStreamManager: listen to and log OnThermalStateChange
> 
> crrev.com/c/2195334 just landed PowerObserver::OnThermalStateChange,
> this CL listens to it from MediaStreamManager and logs it in the
> WebRTC log.
> 
> The utility function PowerMonitorSource::DeviceThermalStateToString
> gets #if DCHECK_IS_ON() removed to be used from the new code, which
> runs also in non-dcheck_always_on builds.
> 
> Bug: 1071431
> Change-Id: I5641ae5a96e4ca9504498821ffb3ea91adbec18a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207410
> Reviewed-by: François Doray <fdoray@chromium.org>
> Reviewed-by: Guido Urdaneta <guidou@chromium.org>
> Reviewed-by: Henrik Boström <hbos@chromium.org>
> Commit-Queue: Miguel Casas <mcasas@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#771045}

TBR=fdoray@chromium.org,mcasas@chromium.org,hbos@chromium.org,guidou@chromium.org,eshr@google.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1071431
Change-Id: I998eb902d70929171dacd12e7916159d47d6787a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218477Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Commit-Queue: Robert Kroeger <rjkroege@chromium.org>
Auto-Submit: Robert Kroeger <rjkroege@chromium.org>
Cr-Commit-Position: refs/heads/master@{#772435}

base/power_monitor/power_monitor.cc

@@ -80,8 +80,10 @@ void PowerMonitor::NotifyResume() {
 void PowerMonitor::NotifyThermalStateChange(
     PowerObserver::DeviceThermalState new_state) {
   DCHECK(IsInitialized());
+#if DCHECK_IS_ON()
   DVLOG(1) << "ThermalStateChange: "
            << PowerMonitorSource::DeviceThermalStateToString(new_state);
+#endif
   GetInstance()->observers_->Notify(
       FROM_HERE, &PowerObserver::OnThermalStateChange, new_state);
 }

base/power_monitor/power_monitor_source.cc

@@ -75,8 +75,9 @@ void PowerMonitorSource::SetInitialOnBatteryPowerState(bool on_battery_power) {
   on_battery_power_ = on_battery_power;
 }
 
+#if DCHECK_IS_ON()
 // static
-const char* PowerMonitorSource::DeviceThermalStateToString(
+std::string PowerMonitorSource::DeviceThermalStateToString(
     PowerObserver::DeviceThermalState state) {
   switch (state) {
     case PowerObserver::DeviceThermalState::kUnknown:
@@ -93,5 +94,6 @@ const char* PowerMonitorSource::DeviceThermalStateToString(
   NOTREACHED();
   return "Unknown";
 }
+#endif
 
 }  // namespace base

base/power_monitor/power_monitor_source.h

@@ -31,8 +31,10 @@ class BASE_EXPORT PowerMonitorSource {
   // Is the computer currently on battery power. Can be called on any thread.
   bool IsOnBatteryPower();
 
-  static const char* DeviceThermalStateToString(
+#if DCHECK_IS_ON()
+  static std::string DeviceThermalStateToString(
       PowerObserver::DeviceThermalState state);
+#endif
 
  protected:
   friend class PowerMonitorTest;

base/power_monitor/thermal_state_observer_mac.mm

@@ -43,8 +43,10 @@ ThermalStateObserverMac::ThermalStateObserverMac(
     state = NSProcessInfoThermalStateToDeviceThermalState(nsinfo_state);
     if (state_for_testing_ != PowerObserver::DeviceThermalState::kUnknown)
       state = state_for_testing_;
+#if DCHECK_IS_ON()
     DVLOG(1) << __func__ << ": "
              << PowerMonitorSource::DeviceThermalStateToString(state);
+#endif
     state_update_callback.Run(state);
   };
 

content/browser/renderer_host/media/media_stream_manager.cc

@@ -19,7 +19,6 @@
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/power_monitor/power_monitor.h"
-#include "base/power_monitor/power_monitor_source.h"
 #include "base/rand_util.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
@@ -2101,14 +2100,6 @@ void MediaStreamManager::OnResume() {
   SendLogMessage(base::StringPrintf("OnResume([this=%p])", this));
 }
 
-void MediaStreamManager::OnThermalStateChange(
-    base::PowerObserver::DeviceThermalState new_state) {
-  const char* state_name =
-      base::PowerMonitorSource::DeviceThermalStateToString(new_state);
-  SendLogMessage(base::StringPrintf(
-      "OnThermalStateChange({this=%p}, {new_state=%s})", this, state_name));
-}
-
 void MediaStreamManager::UseFakeUIFactoryForTests(
     base::RepeatingCallback<std::unique_ptr<FakeMediaStreamUIProxy>(void)>
         fake_ui_factory) {

content/browser/renderer_host/media/media_stream_manager.h

@@ -279,8 +279,6 @@ class CONTENT_EXPORT MediaStreamManager
   // base::PowerObserver overrides.
   void OnSuspend() override;
   void OnResume() override;
-  void OnThermalStateChange(
-      base::PowerObserver::DeviceThermalState new_state) override;
 
   // Called by the tests to specify a factory for creating
   // FakeMediaStreamUIProxys to be used for generated streams.