Add page-level aggregation of SubresourceFilter time metrics.

components/subresource_filter/content/renderer/subresource_filter_agent.cc

Index: components/subresource_filter/content/renderer/subresource_filter_agent.cc
diff --git a/components/subresource_filter/content/renderer/subresource_filter_agent.cc b/components/subresource_filter/content/renderer/subresource_filter_agent.cc
index 05405c00b9a3b88998bed812659f29d7dc615c96..bcdf5c92f53c6ff3997c823d2bcc265939e57ada 100644
--- a/components/subresource_filter/content/renderer/subresource_filter_agent.cc
+++ b/components/subresource_filter/content/renderer/subresource_filter_agent.cc
@@ -113,6 +113,12 @@ void SubresourceFilterAgent::RecordHistogramsOnLoadFinished() {
         statistics.evaluation_total_cpu_duration,
         base::TimeDelta::FromMicroseconds(1), base::TimeDelta::FromSeconds(10),
         50);
+
+    render_frame()->Send(

[engedy, 2016/12/19 10:18:42]

Note that the unit test for this class is quite tricky. There won't be a render
frame in the unittest, see explanation in:

https://cs.chromium.org/chromium/src/components/subresource_filter/content/re...

This means that we will need to put this into a virtual method so that it can be
mocked out. It probably makes sense to either add the reported durations as
arguments so they can be verified during tests, or find another way to verify
them.

[pkalinnikov, 2016/12/21 15:28:46]

Done.

+        new SubresourceFilterHostMsg_DidAggregateEvaluationDuration(
+            render_frame()->GetRoutingID(),
+            statistics.evaluation_total_wall_duration,
+            statistics.evaluation_total_cpu_duration));
   }
 }