Adding DRP specfic UMA for FirstContentfulPaint

content/browser/loader/navigation_resource_throttle.h

 // Copyright 2015 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_BROWSER_LOADER_NAVIGATION_RESOURCE_THROTTLE_H_
 #define CONTENT_BROWSER_LOADER_NAVIGATION_RESOURCE_THROTTLE_H_
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/browser/resource_throttle.h"
 
 namespace net {
 class URLRequest;
 }
 
 namespace content {
+class ResourceDispatcherHostImpl;
 
 // This ResourceThrottle is used to convey throttling information to the UI
 // thread during navigations. The UI thread can then use its NavigationThrottle
 // mechanism to interact with the navigation.
 class NavigationResourceThrottle : public ResourceThrottle {
  public:
-  NavigationResourceThrottle(net::URLRequest* request);
+  NavigationResourceThrottle(net::URLRequest* request,
+                             ResourceDispatcherHostImpl* rdh);

[nasko, 2016/04/26 20:18:40]

Use full names for parameters, it isn't that much longer to use
resource_dispatcher_host. 

[RyanSturm, 2016/04/27 23:27:47]

Done.

   ~NavigationResourceThrottle() override;
 
   // ResourceThrottle overrides:
   void WillStartRequest(bool* defer) override;
   void WillRedirectRequest(const net::RedirectInfo& redirect_info,
                            bool* defer) override;
   void WillProcessResponse(bool* defer) override;
   const char* GetNameForLogging() const override;
 
  private:
   void OnUIChecksPerformed(NavigationThrottle::ThrottleCheckResult result);
 
   net::URLRequest* request_;
+  ResourceDispatcherHostImpl* rdh_;

[nasko, 2016/04/26 20:18:40]

Same here, having a longer name makes it a bit easier to read in area of the
code where the type isn't apparent.

[RyanSturm, 2016/04/27 23:27:47]

Done.

   base::WeakPtrFactory<NavigationResourceThrottle> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationResourceThrottle);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_NAVIGATION_RESOURCE_THROTTLE_H_