SdchObserver: add OnDictionary{Added,Removed}

net/base/sdch_observer.h

Index: net/base/sdch_observer.h
diff --git a/net/base/sdch_observer.h b/net/base/sdch_observer.h
index b74a090146a6a2ae01f796154f2137f48d5d353f..0a28d8c99c19b77984d0756db35fb4d85cadada9 100644
--- a/net/base/sdch_observer.h
+++ b/net/base/sdch_observer.h
@@ -22,9 +22,14 @@ class NET_EXPORT SdchObserver {
  public:
   virtual ~SdchObserver();
 
-  // TODO(rdsmith): Add Added/Removed signals.  These are only needed if
-  // we end up with an implementation in which more than one observer
-  // generates Add/Removed events; otherwise, tracking can be done internally.
+  // A dictionary has been added to the observed manager.
+  virtual void OnDictionaryAdded(SdchManager* manager,
+                                 const GURL& dictionary_url,
+                                 const std::string& server_hash) = 0;
+
+  // A dictionary has been removed from the observed manager.

[mmenke, 2015/05/12 15:46:42]

Should mention this isn't called on clear.

[Elly Fong-Jones, 2015/05/14 19:29:28]

Done.

+  virtual void OnDictionaryRemoved(SdchManager* manager,
+                                   const std::string& server_hash) = 0;
 
   // TODO(rdsmith): Add signal that an Avail-Dictionary header was generated.
   // Should be added if/when an observer wants to use it to fine-tune