Commit a1e48f14 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

Remove reference to SystemMonitor on iOS.

Bug was closed as WontFix, so removing the corresponding commented
out code.

Bug: 228014
Change-Id: I9d4e389d319523b40b79b21ac94497ff2c0241d6
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/893265Reviewed-by: default avatarPeter Lee <pkl@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532875}
parent 053e0ec2
...@@ -14,7 +14,6 @@ ...@@ -14,7 +14,6 @@
namespace base { namespace base {
class PowerMonitor; class PowerMonitor;
class SystemMonitor;
} // namespace base } // namespace base
namespace web { namespace web {
...@@ -72,7 +71,6 @@ class WebMainLoop { ...@@ -72,7 +71,6 @@ class WebMainLoop {
// The MessageLoop and NetworkChangeNotifier are not owned by the WebMainLoop // The MessageLoop and NetworkChangeNotifier are not owned by the WebMainLoop
// but still need to be destroyed in correct order so use ScopedClosureRunner. // but still need to be destroyed in correct order so use ScopedClosureRunner.
base::ScopedClosureRunner destroy_message_loop_; base::ScopedClosureRunner destroy_message_loop_;
std::unique_ptr<base::SystemMonitor> system_monitor_;
std::unique_ptr<base::PowerMonitor> power_monitor_; std::unique_ptr<base::PowerMonitor> power_monitor_;
base::ScopedClosureRunner destroy_network_change_notifier_; base::ScopedClosureRunner destroy_network_change_notifier_;
......
...@@ -17,7 +17,6 @@ ...@@ -17,7 +17,6 @@
#include "base/power_monitor/power_monitor.h" #include "base/power_monitor/power_monitor.h"
#include "base/power_monitor/power_monitor_device_source.h" #include "base/power_monitor/power_monitor_device_source.h"
#include "base/process/process_metrics.h" #include "base/process/process_metrics.h"
#include "base/system_monitor/system_monitor.h"
#include "base/task_scheduler/scheduler_worker_pool_params.h" #include "base/task_scheduler/scheduler_worker_pool_params.h"
#include "base/task_scheduler/task_scheduler.h" #include "base/task_scheduler/task_scheduler.h"
#include "base/threading/sequenced_worker_pool.h" #include "base/threading/sequenced_worker_pool.h"
...@@ -77,11 +76,8 @@ void WebMainLoop::MainMessageLoopStart() { ...@@ -77,11 +76,8 @@ void WebMainLoop::MainMessageLoopStart() {
InitializeMainThread(); InitializeMainThread();
#if 0 // TODO(crbug.com/807279): Do we need PowerMonitor on iOS, or can we get rid
// TODO(crbug.com/228014): SystemMonitor is not working properly on iOS. // of it?
system_monitor_.reset(new base::SystemMonitor);
#endif
// TODO(rohitrao): Do we need PowerMonitor on iOS, or can we get rid of it?
std::unique_ptr<base::PowerMonitorSource> power_monitor_source( std::unique_ptr<base::PowerMonitorSource> power_monitor_source(
new base::PowerMonitorDeviceSource()); new base::PowerMonitorDeviceSource());
power_monitor_.reset(new base::PowerMonitor(std::move(power_monitor_source))); power_monitor_.reset(new base::PowerMonitor(std::move(power_monitor_source)));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment