Delivering the FIRST_NONEMPTY_PAINT phase changing event to base/

content/public/browser/profiler_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_PROFILER_CONTROLLER_H_
 #define CONTENT_PUBLIC_BROWSER_PROFILER_CONTROLLER_H_
 
 #include <set>
 #include <string>
 
@@ skipping 24 matching lines @@
   // OnProfilerDataCollected is returning profiler data from a child process.
   // This is called on UI thread.
   virtual void Register(ProfilerSubscriber* subscriber) = 0;
 
   // Unregister the subscriber so that it will not be called when for example
   // OnProfilerDataCollected is returning profiler data from a child process.
   // Safe to call even if caller is not the current subscriber.
   virtual void Unregister(const ProfilerSubscriber* subscriber) = 0;
 
   // Contact all processes and get their profiler data.
-  virtual void GetProfilerData(int sequence_number) = 0;
+  // |current_profiling_phase| is the number of the current profiling phase.

[Ilya Sherman, 2015/04/07 01:15:31]

nit: s/number/0-indexed identifier, or something like that.

[vadimt, 2015/04/07 21:44:15]

Done.

+  virtual void GetProfilerData(int sequence_number,
+                               int current_profiling_phase) = 0;
+
+  // Contact child processes and notify them of the profiling phase
+  // |profiling_phase| completion.

[Ilya Sherman, 2015/04/07 01:15:31]

nit: "notify them of the profiling phase completion" -> "notify them that the
|profiling_phase| has completed".

[vadimt, 2015/04/07 21:44:15]

Done.

+  virtual void OnProfilingPhaseCompletion(int profiling_phase) = 0;

[Ilya Sherman, 2015/04/07 01:15:31]

nit: "Completion" -> "Completed"

 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_PROFILER_CONTROLLER_H_