[NoStatePrefetch] Add performance histograms.

chrome/browser/prerender/prerender_manager_factory.h

Index: chrome/browser/prerender/prerender_manager_factory.h
diff --git a/chrome/browser/prerender/prerender_manager_factory.h b/chrome/browser/prerender/prerender_manager_factory.h
index dbc5fb31bb4de5a1b919fabd5cf4df2389b170c6..46e7e67f2db756bb316e4ada0a1de322763340b4 100644
--- a/chrome/browser/prerender/prerender_manager_factory.h
+++ b/chrome/browser/prerender/prerender_manager_factory.h
@@ -9,6 +9,10 @@
 #include "base/memory/singleton.h"
 #include "components/keyed_service/content/browser_context_keyed_service_factory.h"
 
+namespace content {
+class BrowserContext;
+}
+
 class Profile;
 
 namespace prerender {
@@ -21,7 +25,8 @@ class PrerenderManager;
 class PrerenderManagerFactory : public BrowserContextKeyedServiceFactory {
  public:
   // Returns the PrerenderManager for |profile|.
-  static PrerenderManager* GetForProfile(Profile* profile);
+  static PrerenderManager* GetForBrowserContext(

[mattcary, 2016/09/14 12:33:21]

Why change this? Is it just cleanup or is there a functional reason?

[droger, 2016/09/14 14:15:42]

It's mainly a cleanup. I can do this a separate CL if this is better, or even
work around it.

I did this because I added a new call to this function in a place where I have a
BrowserContext and not a Profile.

I could convert the BrowserContext to Profile instead (like many other callers
do), but it turns out that the function does not actually need a Profile and a
BrowserContext is enough. So I figured the cleaner way is to fix the function
and all the callers to use BrowserContext.

[mattcary, 2016/09/14 14:19:39]

Ok, thanks for the context.

No need to put that in a separate CL from my perspective.

+      content::BrowserContext* context);
 
   static PrerenderManagerFactory* GetInstance();