content: Remove 16 exit time destructors and 15 static initializers.

content/browser/net/url_request_mock_http_job.h

Index: content/browser/net/url_request_mock_http_job.h
diff --git a/content/browser/net/url_request_mock_http_job.h b/content/browser/net/url_request_mock_http_job.h
index e2c68f8293553219ce912a3edce24dde52ec2801..a556823eab917a9409cca75dd03a3cb654b6b59d 100644
--- a/content/browser/net/url_request_mock_http_job.h
+++ b/content/browser/net/url_request_mock_http_job.h
@@ -29,10 +29,10 @@ class CONTENT_EXPORT URLRequestMockHTTPJob : public net::URLRequestFileJob {
   // Adds the testing URLs to the net::URLRequestFilter.
   static void AddUrlHandler(const FilePath& base_path);
 
-  // Given the path to a file relative to base_path_, construct a mock URL.
+  // Given the path to a file relative to BasePath(), construct a mock URL.
   static GURL GetMockUrl(const FilePath& path);
 
-  // Given the path to a file relative to base_path_,
+  // Given the path to a file relative to BasePath(),
   // construct a mock URL for view source.
   static GURL GetMockViewSourceUrl(const FilePath& path);
 
@@ -48,7 +48,7 @@ class CONTENT_EXPORT URLRequestMockHTTPJob : public net::URLRequestFileJob {
 
   // This is the file path leading to the root of the directory to use as the
   // root of the http server.
-  static FilePath base_path_;
+  static FilePath& BasePath();

[Mark Mentovai, 2011/11/07 22:34:15]

You’re making this return a non-const reference, and expecting a caller to
overwrite the value. Callers may not expect that. It’s worth a comment.

private + static, does this need to be here or can it move to an anonymous
namespace in the .cc file as an implementation detail?

[Nico, 2011/11/07 23:29:21]

Done.
I didn't do it because some comments referenced it. I updated the comments and
moved it over.

 };
 
 #endif  // CONTENT_BROWSER_NET_URL_REQUEST_MOCK_HTTP_JOB_H_