Eliminate MetricsProvider::RecordCurrentState()

chrome/browser/metrics/plugin_metrics_provider.h

Index: chrome/browser/metrics/plugin_metrics_provider.h
diff --git a/chrome/browser/metrics/plugin_metrics_provider.h b/chrome/browser/metrics/plugin_metrics_provider.h
index baabcf746df0bc5a1f499815ca0898d56e7078d0..c89d57c6876019efbba429ac64726d38029baf19 100644
--- a/chrome/browser/metrics/plugin_metrics_provider.h
+++ b/chrome/browser/metrics/plugin_metrics_provider.h
@@ -42,7 +42,7 @@ class PluginMetricsProvider : public metrics::MetricsProvider,
       metrics::SystemProfileProto* system_profile_proto) OVERRIDE;
   virtual void ProvideStabilityMetrics(
       metrics::SystemProfileProto* system_profile_proto) OVERRIDE;
-  virtual void RecordCurrentState() OVERRIDE;
+  void RecordCurrentState();

[Alexei Svitkine (slow), 2014/08/06 15:10:01]

Now that this is no longer an implementation of the interface, please add a
comment for the method and move it to the private section.

[gayane -on leave until 09-2017, 2014/08/06 19:38:27]

Done.

 
   // Notifies the provider about an error loading the plugin at |plugin_path|.
   void LogPluginLoadingError(const base::FilePath& plugin_path);
@@ -50,6 +50,15 @@ class PluginMetricsProvider : public metrics::MetricsProvider,
   // Sets this provider's list of plugins, exposed for testing.
   void SetPluginsForTesting(const std::vector<content::WebPluginInfo>& plugins);
 
+  // Posts a delayed task for RecordCurrentState. Returns true if new task is
+  // posted and false if there was one already waiting for execution.
+  // The param delay_sec is for unit tests.
+  bool RecordCurrentStateWithDelay(int delay_ms);

[Alexei Svitkine (slow), 2014/08/06 15:10:01]

The new methods you're adding shouldn't be public. Instead, make them private
and you can set the new test to be a friend. (See variations_service.h for an
example how to do that with the FRIEND_TEST macros).

[gayane -on leave until 09-2017, 2014/08/06 19:38:27]

Done.

+
+  // If a delayed RecordCurrnetState task exists than cancels it, calls

[Alexei Svitkine (slow), 2014/08/06 15:10:01]

nit: than -> then

[gayane -on leave until 09-2017, 2014/08/06 19:38:27]

Done.

+  // RecordCurrentState immediately and returns true. Otherwise returns false.
+  bool RecordCurrentStateIfPending();
+
   // Returns true if process of type |type| should be counted as a plugin
   // process, and false otherwise.
   static bool IsPluginProcess(int process_type);