[subresource_filter] Add metrics for all NavigationThrottle delays

components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h

Index: components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
diff --git a/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h b/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
index e0c8aefc902946f5b5d09c20c893f2ee1f255d8c..a3b8dc6b37d371081f7a3d8d17119d960e86a5c6 100644
--- a/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
+++ b/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
@@ -7,6 +7,7 @@
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "base/time/time.h"
 #include "components/subresource_filter/content/browser/async_document_subresource_filter.h"
 #include "content/public/browser/navigation_throttle.h"
 
@@ -46,6 +47,10 @@ class SubframeNavigationFilteringThrottle : public content::NavigationThrottle {
   // Must outlive this class.
   AsyncDocumentSubresourceFilter* parent_frame_filter_;
 
+  base::TimeTicks last_defer_time_;

[shivanisha, 2017/05/05 18:44:50]

nit: Rename to defer_start_timestamp_ to emphasize it is actually a timestamp?

[Charlie Harrison, 2017/05/05 18:56:07]

What about last_defer_timestamp_? Since we defer not only at the start but also
during redirects.

+  base::TimeDelta total_defer_time_;
+  bool disallowed_ = false;
+
   base::WeakPtrFactory<SubframeNavigationFilteringThrottle> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SubframeNavigationFilteringThrottle);