Add metrics for DHCP WPAD feature.

net/proxy/dhcp_proxy_script_fetcher_win.h

Index: net/proxy/dhcp_proxy_script_fetcher_win.h
diff --git a/net/proxy/dhcp_proxy_script_fetcher_win.h b/net/proxy/dhcp_proxy_script_fetcher_win.h
index 3783971c8c0188243a0852a49df04fbe01db75a6..06e8a4b4e67424edaf1afad8571ad05b9f0cca84 100644
--- a/net/proxy/dhcp_proxy_script_fetcher_win.h
+++ b/net/proxy/dhcp_proxy_script_fetcher_win.h
@@ -12,6 +12,7 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/threading/non_thread_safe.h"
+#include "base/time.h"
 #include "base/timer.h"
 #include "net/proxy/dhcp_proxy_script_fetcher.h"
 
@@ -44,6 +45,7 @@ class DhcpProxyScriptFetcherWin
 
  protected:
   // Event/state transition handlers
+  void CancelImpl(bool user_initiated);
   void OnFetcherDone(int result);
   void OnWaitTimer();
   void TransitionToDone();
@@ -113,6 +115,9 @@ class DhcpProxyScriptFetcherWin
 
   base::OneShotTimer<DhcpProxyScriptFetcherWin> wait_timer_;
 
+  // Time |Fetch()| was last called, 0 if never.
+  base::TimeTicks fetch_start_time_;

[jar (doing other things), 2011/05/23 19:37:52]

nit: Data members should be private, and accessors and mutators can be public or
protected.

I'm surprised to see a pile of such declarations here... and that should be
changed sometime... but you probably shouldn't add to the violations (the
corrected style is very helpful in debugging).

[Jói, 2011/05/24 00:58:38]

I've made this member private (as well as the implicit constructors). I've also
added a TODO in this file for me to clean this up better, and added that close
to the top of my TODO list (right after the item about moving other users of
UMA_HISTOGRAM_CUSTOM_ENUMERATION that are transforming from a static array of
values to using the new helper function).

+
   scoped_refptr<URLRequestContext> url_request_context_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(DhcpProxyScriptFetcherWin);