Make SdchManager per-profile.

net/filter/sdch_filter.h

Index: net/filter/sdch_filter.h
diff --git a/net/filter/sdch_filter.h b/net/filter/sdch_filter.h
index 5b401e464e8933b840463f450a94884fd4714157..325f0af3119edad21f090080f860a772ef1eaaa8 100644
--- a/net/filter/sdch_filter.h
+++ b/net/filter/sdch_filter.h
@@ -95,6 +95,13 @@ class NET_EXPORT_PRIVATE SdchFilter : public Filter {
   // That char* data is part of the dictionary_ we hold a reference to.
   scoped_refptr<SdchManager::Dictionary> dictionary_;
 
+  // We keep a copy of the URLRequestContext for use in the destructor, (at
+  // which point GetURLRequestContext() will likely return null because of
+  // the disassociation of the URLRequest from the URLRequestJob).  This is
+  // safe because the URLRequestJob (and any filters) are guaranteed to be
+  // deleted before the URLRequestContext is destroyed.
+  const URLRequestContext * const url_request_context_;

[jar (doing other things), 2014/06/13 18:28:48]

nit: no space before the asterisk. (like lines 43, 44, 66).

The style guide specifically asks that you not put a space on both sides.... and
leaves either side as allowed... but...

Chromium tends to standardize on putting the space *only* after the asterisk
(despite the fact that the parsing binds the asterisk more tightly to what is on
its right :-/).

[Randy Smith (Not in Mondays), 2014/06/13 19:40:12]

Whoops; done.

+
   // The decoder may demand a larger output buffer than the target of
   // ReadFilteredData so we buffer the excess output between calls.
   std::string dest_buffer_excess_;