Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/trace_event/memory_dump_manager.h" | 5 #include "base/trace_event/memory_dump_manager.h" |
| 6 | 6 |
| 7 #include <algorithm> | 7 #include <algorithm> |
| 8 | 8 |
| 9 #include "base/atomic_sequence_num.h" | 9 #include "base/atomic_sequence_num.h" |
| 10 #include "base/compiler_specific.h" | 10 #include "base/compiler_specific.h" |
| (...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 150 // static | 150 // static |
| 151 void MemoryDumpManager::SetInstanceForTesting(MemoryDumpManager* instance) { | 151 void MemoryDumpManager::SetInstanceForTesting(MemoryDumpManager* instance) { |
| 152 if (instance) | 152 if (instance) |
| 153 instance->skip_core_dumpers_auto_registration_for_testing_ = true; | 153 instance->skip_core_dumpers_auto_registration_for_testing_ = true; |
| 154 g_instance_for_testing = instance; | 154 g_instance_for_testing = instance; |
| 155 } | 155 } |
| 156 | 156 |
| 157 MemoryDumpManager::MemoryDumpManager() | 157 MemoryDumpManager::MemoryDumpManager() |
| 158 : delegate_(nullptr), | 158 : delegate_(nullptr), |
| 159 memory_tracing_enabled_(0), | 159 memory_tracing_enabled_(0), |
| 160 tracing_process_id_(kInvalidTracingProcessId), | |
| 160 skip_core_dumpers_auto_registration_for_testing_(false) { | 161 skip_core_dumpers_auto_registration_for_testing_(false) { |
| 161 g_next_guid.GetNext(); // Make sure that first guid is not zero. | 162 g_next_guid.GetNext(); // Make sure that first guid is not zero. |
| 162 } | 163 } |
| 163 | 164 |
| 164 MemoryDumpManager::~MemoryDumpManager() { | 165 MemoryDumpManager::~MemoryDumpManager() { |
| 165 base::trace_event::TraceLog::GetInstance()->RemoveEnabledStateObserver(this); | 166 base::trace_event::TraceLog::GetInstance()->RemoveEnabledStateObserver(this); |
| 166 } | 167 } |
| 167 | 168 |
| 168 void MemoryDumpManager::Initialize() { | 169 void MemoryDumpManager::Initialize() { |
| 169 TRACE_EVENT0(kTraceCategory, "init"); // Add to trace-viewer category list. | 170 TRACE_EVENT0(kTraceCategory, "init"); // Add to trace-viewer category list. |
| (...skipping 255 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 425 } | 426 } |
| 426 } | 427 } |
| 427 | 428 |
| 428 void MemoryDumpManager::OnTraceLogDisabled() { | 429 void MemoryDumpManager::OnTraceLogDisabled() { |
| 429 AutoLock lock(lock_); | 430 AutoLock lock(lock_); |
| 430 periodic_dump_timer_.Stop(); | 431 periodic_dump_timer_.Stop(); |
| 431 subtle::NoBarrier_Store(&memory_tracing_enabled_, 0); | 432 subtle::NoBarrier_Store(&memory_tracing_enabled_, 0); |
| 432 session_state_ = nullptr; | 433 session_state_ = nullptr; |
| 433 } | 434 } |
| 434 | 435 |
| 436 // static | |
| 437 int MemoryDumpManager::ChildProcessIdToTracingProcessId(int child_process_id) { | |
| 438 return 1000 + child_process_id; | |
|
picksi
2015/06/24 11:12:43
Can we use a named const to explain the 1000, e.g.
jam
2015/06/24 16:38:15
why is this guessable? i.e. why not a hash?
Primiano Tucci (use gerrit)
2015/06/24 17:47:34
+1
| |
| 439 } | |
| 440 | |
| 435 MemoryDumpManager::MemoryDumpProviderInfo::MemoryDumpProviderInfo( | 441 MemoryDumpManager::MemoryDumpProviderInfo::MemoryDumpProviderInfo( |
| 436 const scoped_refptr<SingleThreadTaskRunner>& task_runner) | 442 const scoped_refptr<SingleThreadTaskRunner>& task_runner) |
| 437 : task_runner(task_runner), consecutive_failures(0), disabled(false) { | 443 : task_runner(task_runner), consecutive_failures(0), disabled(false) { |
| 438 } | 444 } |
| 439 MemoryDumpManager::MemoryDumpProviderInfo::~MemoryDumpProviderInfo() { | 445 MemoryDumpManager::MemoryDumpProviderInfo::~MemoryDumpProviderInfo() { |
| 440 } | 446 } |
| 441 | 447 |
| 442 } // namespace trace_event | 448 } // namespace trace_event |
| 443 } // namespace base | 449 } // namespace base |
| OLD | NEW |