Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Side by Side Diff: components/metrics/profiler/tracking_synchronizer.cc

Issue 985773002: Introducing phased profiling framework (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@write_to_file
Patch Set: isherman@ comments Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "components/metrics/profiler/tracking_synchronizer.h" 5 #include "components/metrics/profiler/tracking_synchronizer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/threading/thread.h" 9 #include "base/threading/thread.h"
10 #include "base/tracked_objects.h" 10 #include "base/tracked_objects.h"
11 #include "components/metrics/profiler/tracking_synchronizer_observer.h" 11 #include "components/metrics/profiler/tracking_synchronizer_observer.h"
12 #include "components/variations/variations_associated_data.h"
12 #include "content/public/browser/browser_thread.h" 13 #include "content/public/browser/browser_thread.h"
13 #include "content/public/browser/profiler_controller.h" 14 #include "content/public/browser/profiler_controller.h"
14 #include "content/public/common/process_type.h" 15 #include "content/public/common/process_type.h"
15 16
16 using base::TimeTicks; 17 using base::TimeTicks;
17 using content::BrowserThread; 18 using content::BrowserThread;
18 19
19 namespace metrics { 20 namespace metrics {
20 21
21 namespace { 22 namespace {
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 }; 173 };
173 174
174 // static 175 // static
175 base::LazyInstance 176 base::LazyInstance
176 <TrackingSynchronizer::RequestContext::RequestContextMap>::Leaky 177 <TrackingSynchronizer::RequestContext::RequestContextMap>::Leaky
177 TrackingSynchronizer::RequestContext::outstanding_requests_ = 178 TrackingSynchronizer::RequestContext::outstanding_requests_ =
178 LAZY_INSTANCE_INITIALIZER; 179 LAZY_INSTANCE_INITIALIZER;
179 180
180 // TrackingSynchronizer methods and members. 181 // TrackingSynchronizer methods and members.
181 182
182 TrackingSynchronizer::TrackingSynchronizer() 183 TrackingSynchronizer::TrackingSynchronizer(const base::TimeTicks& now)
183 : last_used_sequence_number_(kNeverUsableSequenceNumber) { 184 : last_used_sequence_number_(kNeverUsableSequenceNumber),
185 start_time_(now) {
184 DCHECK(!g_tracking_synchronizer); 186 DCHECK(!g_tracking_synchronizer);
185 g_tracking_synchronizer = this; 187 g_tracking_synchronizer = this;
188 phase_start_times_.push_back(now);
186 content::ProfilerController::GetInstance()->Register(this); 189 content::ProfilerController::GetInstance()->Register(this);
187 } 190 }
188 191
189 TrackingSynchronizer::~TrackingSynchronizer() { 192 TrackingSynchronizer::~TrackingSynchronizer() {
190 content::ProfilerController::GetInstance()->Unregister(this); 193 content::ProfilerController::GetInstance()->Unregister(this);
191 194
192 // Just in case we have any pending tasks, clear them out. 195 // Just in case we have any pending tasks, clear them out.
193 RequestContext::OnShutdown(); 196 RequestContext::OnShutdown();
194 197
195 g_tracking_synchronizer = NULL; 198 g_tracking_synchronizer = NULL;
(...skipping 29 matching lines...) Expand all
225 if (!request) 228 if (!request)
226 return; 229 return;
227 request->AddProcessesPending(pending_processes); 230 request->AddProcessesPending(pending_processes);
228 request->SetReceivedProcessGroupCount(end); 231 request->SetReceivedProcessGroupCount(end);
229 request->DeleteIfAllDone(); 232 request->DeleteIfAllDone();
230 } 233 }
231 234
232 void TrackingSynchronizer::OnProfilerDataCollected( 235 void TrackingSynchronizer::OnProfilerDataCollected(
233 int sequence_number, 236 int sequence_number,
234 const tracked_objects::ProcessDataSnapshot& profiler_data, 237 const tracked_objects::ProcessDataSnapshot& profiler_data,
235 int process_type) { 238 content::ProcessType process_type) {
236 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 239 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
237 DecrementPendingProcessesAndSendData(sequence_number, profiler_data, 240 DecrementPendingProcessesAndSendData(sequence_number, profiler_data,
238 process_type); 241 process_type);
239 } 242 }
240 243
241 int TrackingSynchronizer::RegisterAndNotifyAllProcesses( 244 int TrackingSynchronizer::RegisterAndNotifyAllProcesses(
242 const base::WeakPtr<TrackingSynchronizerObserver>& callback_object) { 245 const base::WeakPtr<TrackingSynchronizerObserver>& callback_object) {
243 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 246 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
244 247
245 int sequence_number = GetNextAvailableSequenceNumber(); 248 int sequence_number = GetNextAvailableSequenceNumber();
246 249
247 RequestContext* request = 250 RequestContext* request =
248 RequestContext::Register(sequence_number, callback_object); 251 RequestContext::Register(sequence_number, callback_object);
249 252
250 // Increment pending process count for sending browser's profiler data. 253 // Increment pending process count for sending browser's profiler data.
251 request->IncrementProcessesPending(); 254 request->IncrementProcessesPending();
252 255
253 // Get profiler data from renderer and browser child processes. 256 // Get profiler data from renderer and browser child processes.
254 content::ProfilerController::GetInstance()->GetProfilerData(sequence_number); 257 content::ProfilerController::GetInstance()->GetProfilerData(sequence_number);
255 258
256 // Send profiler_data from browser process. 259 // Send process data snapshot from browser process.
257 tracked_objects::ProcessDataSnapshot process_data; 260 tracked_objects::ProcessDataSnapshot process_data_snapshot;
258 tracked_objects::ThreadData::Snapshot(&process_data); 261 tracked_objects::ThreadData::Snapshot(&process_data_snapshot);
259 DecrementPendingProcessesAndSendData(sequence_number, process_data, 262 DecrementPendingProcessesAndSendData(sequence_number, process_data_snapshot,
260 content::PROCESS_TYPE_BROWSER); 263 content::PROCESS_TYPE_BROWSER);
261 264
262 return sequence_number; 265 return sequence_number;
263 } 266 }
264 267
268 void TrackingSynchronizer::SendData(
269 const tracked_objects::ProcessDataSnapshot& profiler_data,
270 content::ProcessType process_type,
271 const base::TimeTicks& now,
272 TrackingSynchronizerObserver* observer) const {
273 // We are going to loop though past profiling phases and notify the request
274 // about each phase that is contained in profiler_data. past_events
275 // will track the set of past profiling events as we go.
276 ProfilerEvents past_events;
277
278 // Go through all completed phases, and through the current one. The current
279 // one is not in phase_completion_events_sequence_, but note the <=
280 // comparison.
281 for (size_t phase = 0; phase <= phase_completion_events_sequence_.size();
282 ++phase) {
283 tracked_objects::PhasedProcessDataSnapshotMap::const_iterator it =
284 profiler_data.phased_process_data_snapshots.find(phase);
285
286 if (it != profiler_data.phased_process_data_snapshots.end()) {
287 // If the phase is contained in the received snapshot, notify the
288 // request.
289 const base::TimeDelta phase_start =
290 phase_start_times_[phase] - start_time_;
291 const base::TimeDelta phase_finish =
292 (phase + 1 < phase_start_times_.size() ? phase_start_times_[phase + 1]
293 : now) -
294 start_time_;
295 observer->ReceivedProfilerData(it->second, profiler_data.process_id,
296 process_type, phase, phase_start,
297 phase_finish, past_events);
298 }
299
300 if (phase != phase_completion_events_sequence_.size()) {
301 past_events.push_back(phase_completion_events_sequence_[phase]);
302 }
303 }
304 }
305
265 void TrackingSynchronizer::DecrementPendingProcessesAndSendData( 306 void TrackingSynchronizer::DecrementPendingProcessesAndSendData(
266 int sequence_number, 307 int sequence_number,
267 const tracked_objects::ProcessDataSnapshot& profiler_data, 308 const tracked_objects::ProcessDataSnapshot& profiler_data,
268 int process_type) { 309 content::ProcessType process_type) {
269 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 310 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
270 311
271 RequestContext* request = RequestContext::GetRequestContext(sequence_number); 312 RequestContext* request = RequestContext::GetRequestContext(sequence_number);
272 if (!request) 313 if (!request)
273 return; 314 return;
274 315
275 if (request->callback_object_.get()) { 316 TrackingSynchronizerObserver* observer =
276 request->callback_object_ 317 request->callback_object_.get();
277 ->ReceivedProfilerData(profiler_data, process_type); 318 if (observer)
278 } 319 SendData(profiler_data, process_type, base::TimeTicks::Now(), observer);
279 320
280 // Delete request if we have heard back from all child processes. 321 // Delete request if we have heard back from all child processes.
281 request->DecrementProcessesPending(); 322 request->DecrementProcessesPending();
282 request->DeleteIfAllDone(); 323 request->DeleteIfAllDone();
283 } 324 }
284 325
285 int TrackingSynchronizer::GetNextAvailableSequenceNumber() { 326 int TrackingSynchronizer::GetNextAvailableSequenceNumber() {
286 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 327 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
287 328
288 ++last_used_sequence_number_; 329 ++last_used_sequence_number_;
289 330
290 // Watch out for wrapping to a negative number. 331 // Watch out for wrapping to a negative number.
291 if (last_used_sequence_number_ < 0) 332 if (last_used_sequence_number_ < 0)
292 last_used_sequence_number_ = 1; 333 last_used_sequence_number_ = 1;
293 return last_used_sequence_number_; 334 return last_used_sequence_number_;
294 } 335 }
295 336
296 } // namespace metrics 337 } // namespace metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698