Index: base/trace_event/memory_dump_manager.cc |
diff --git a/base/trace_event/memory_dump_manager.cc b/base/trace_event/memory_dump_manager.cc |
index 6f2e8b0f0f953531e84cd860745ec2657c8ad995..354d78220a3e7fa27fdde54746e1c59b6e6fd98d 100644 |
--- a/base/trace_event/memory_dump_manager.cc |
+++ b/base/trace_event/memory_dump_manager.cc |
@@ -617,11 +617,8 @@ |
const MemoryDumpProviderInfo& other) const { |
if (task_runner == other.task_runner) |
return dump_provider < other.dump_provider; |
- // TODO(ssid): Ensure that unbound providers (task_runner == nullptr) |
- // always run last. Currently it runs first so that it doesn't cause |
- // regression in perf bots due to change in measurement time. |
- // crbug.com/555584. |
- return task_runner < other.task_runner; |
+ // Ensure that unbound providers (task_runner == nullptr) always run last. |
+ return !(task_runner < other.task_runner); |
} |
MemoryDumpManager::ProcessMemoryDumpAsyncState::ProcessMemoryDumpAsyncState( |