Delivering the FIRST_NONEMPTY_PAINT phase changing event to base/

content/browser/profiler_controller_impl.h

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..cf2c10cb656c194e341cc44db362464b18dc2e8e 100644
--- a/content/browser/profiler_controller_impl.h
+++ b/content/browser/profiler_controller_impl.h
@@ -44,13 +44,20 @@ 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 OnProfilingPhaseCompleted(int profiling_phase) override;
 
  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.
+  static void NotifyChildProcessesOfProfilingPhaseCompletion(
+      int profiling_phase);

[Ilya Sherman, 2015/04/15 00:37:35]

Can this be moved into the anonymous namespace?

[vadimt, 2015/04/15 19:14:32]

I'd prefer this method to stay next to the similar
GetProfilerDataFromChildProcesses, and be as similar to it as possible...

 
   ProfilerSubscriber* subscriber_;