| 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(
|
|
|