Introduce ActiveScriptController; track active extension scripts

chrome/browser/extensions/activity_log/uma_policy.h

Index: chrome/browser/extensions/activity_log/uma_policy.h
diff --git a/chrome/browser/extensions/activity_log/uma_policy.h b/chrome/browser/extensions/activity_log/uma_policy.h
index e5ea34be863bd0abe85fc38faec34bac537e69da..5731add6d0aa83a4809f0232d47fa82905ada2a8 100644
--- a/chrome/browser/extensions/activity_log/uma_policy.h
+++ b/chrome/browser/extensions/activity_log/uma_policy.h
@@ -96,10 +96,12 @@ class UmaPolicy : public ActivityLogPolicy,
   void SetupOpenedPage(const std::string& url);
 
   // When a page is closing, remove it from the SiteMap url_status_.
-  void CleanupClosedPage(const std::string& url);
+  void CleanupClosedPage(const std::string& url,
+                         content::WebContents* web_contents);
 
   // When a page is closing, save statistics about the page to histograms.
-  void HistogramOnClose(const std::string& url);
+  void HistogramOnClose(const std::string& url,
+                        content::WebContents* web_contents);

[not at google - send to devlin, 2014/05/07 22:49:02]

seems like the URL is implied by the web contents i.e. only need the single
argument.

[Devlin, 2014/05/08 18:15:46]

The url passed here has been cleaned via CleanURL(), so it would be a waste to
duplicate that effort, I think.  Thoughts?

[not at google - send to devlin, 2014/05/08 20:47:09]

I see what you mean. maybe call the variable clean_url then. and it would be
nice, where it matters, to DCHECK_EQ(clean_url, CleanURL(clean_url));

[Devlin, 2014/05/08 23:01:00]

Done.

 
   // Standardizes the way URLs are treated.
   static std::string CleanURL(const GURL& gurl);