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

Unified Diff: content/browser/profiler_controller_impl.h

Issue 1021053003: Delivering the FIRST_NONEMPTY_PAINT phase changing event to base/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@phase_splitting
Patch Set: jar@ comments. Created 5 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/profiler_controller_impl.h
diff --git a/content/browser/profiler_controller_impl.h b/content/browser/profiler_controller_impl.h
index d62f38955dd42c4b0c56b1cf0acf18d08933435e..f7382470313b96a6cf8e19dae874f3bd1e91cd0e 100644
--- a/content/browser/profiler_controller_impl.h
+++ b/content/browser/profiler_controller_impl.h
@@ -44,13 +44,19 @@ class ProfilerControllerImpl : public ProfilerController {
// ProfilerController implementation:
void Register(ProfilerSubscriber* subscriber) override;
void Unregister(const ProfilerSubscriber* subscriber) override;
- void GetProfilerData(int sequence_number) override;
+ void GetProfilerData(int sequence_number,
+ int current_profiling_phase) override;
+ void OnProfilingPhaseCompletion(int profiling_phase) override;
Ilya Sherman 2015/04/07 01:15:31 nit: "Completion" -> "Completed"
vadimt 2015/04/07 21:44:14 Done.
private:
friend struct DefaultSingletonTraits<ProfilerControllerImpl>;
// Contact child processes and get their profiler data.
- void GetProfilerDataFromChildProcesses(int sequence_number);
+ void GetProfilerDataFromChildProcesses(int sequence_number,
+ int current_profiling_phase);
+
+ // Contact child processes and notify them of a profiling phase completion.
+ void NotifyChildProcessesOfProfilingPhaseCompletion(int profiling_phase);
ProfilerSubscriber* subscriber_;

Powered by Google App Engine
This is Rietveld 408576698