Shut down Profile's URLRequestContextGetters safely.

chrome/browser/net/chrome_url_request_context_getter.h

Index: chrome/browser/net/chrome_url_request_context_getter.h
diff --git a/chrome/browser/net/chrome_url_request_context_getter.h b/chrome/browser/net/chrome_url_request_context_getter.h
index d9cff5020855d36eef7533b31ed1f921d49f3a61..31b5c309e46d52de76cfcee94f8fb033fa186e5e 100644
--- a/chrome/browser/net/chrome_url_request_context_getter.h
+++ b/chrome/browser/net/chrome_url_request_context_getter.h
@@ -79,9 +79,9 @@ class ChromeURLRequestContextGetter : public net::URLRequestContextGetter {
       const ProfileIOData* profile_io_data,
       const StoragePartitionDescriptor& partition_descriptor);
 
-  // Discard reference to URLRequestContext.
-  // Access only from the IO thread.
-  void Invalidate();
+  // Discard reference to URLRequestContext and inform observers of shutdown.
+  // Must be called before destruction. May only be called on IO thread.
+  void NotifyContextShuttingDown();

[davidben, 2015/05/14 17:09:26]

Maybe keep this called Invalidate or rename it to something else? I think having
a non-virtual override is probably better avoided. (They also do subtly
different things anyway. The base class's one is "By the way, I just made the
context start returning null. Please tell the observers". The subclass is "By
the way, that factory I constructed you with needs to be destroyed because it
references a dying ProfileIOData. Please deal."

[mmenke, 2015/05/14 19:23:52]

Done - was on the fence about naming, happy to defer to your opinion.  Switched
back to Invalidate.

 
  private:
   ~ChromeURLRequestContextGetter() override;