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 354d78220a3e7fa27fdde54746e1c59b6e6fd98d..6f2e8b0f0f953531e84cd860745ec2657c8ad995 100644 |
--- a/base/trace_event/memory_dump_manager.cc |
+++ b/base/trace_event/memory_dump_manager.cc |
@@ -617,8 +617,11 @@ bool MemoryDumpManager::MemoryDumpProviderInfo::operator<( |
const MemoryDumpProviderInfo& other) const { |
if (task_runner == other.task_runner) |
return dump_provider < other.dump_provider; |
- // Ensure that unbound providers (task_runner == nullptr) always run last. |
- return !(task_runner < other.task_runner); |
+ // 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; |
} |
MemoryDumpManager::ProcessMemoryDumpAsyncState::ProcessMemoryDumpAsyncState( |