I added a function "void SpdySettingsStorage::Clear()". I added a call to this function "spdy_set...

net/spdy/spdy_settings_storage.h

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_SPDY_SPDY_SETTING_STORAGE_H_
 #define NET_SPDY_SPDY_SETTING_STORAGE_H_
 #pragma once
 
 #include <map>
 #include "base/basictypes.h"
 #include "net/base/host_port_pair.h"
 #include "net/spdy/spdy_framer.h"
 
 namespace net {
 
 // SpdySettingsStorage stores SpdySettings which have been transmitted between
 // endpoints for the SPDY SETTINGS frame.
 class SpdySettingsStorage {
  public:
   SpdySettingsStorage();
   ~SpdySettingsStorage();
 
   // Get a copy of the SpdySettings stored for a host.
   // If no settings are stored, returns an empty set of settings.
+  // Since settings_map_ may be cleared, don't remember the address of the
+  // return value.
+

[wtc, 2011/05/19 22:54:37]

Nit: don't add this blank line.

willchan: since Get() returns a (const) reference to a value
in settings_map_, a caller may store the reference, which
will become invalid after a Clear() call.

We can eliminating this risk entirely by changing Get() to
return a copy of the value, but since the value (spdy::SpdySettings)
is an std::list, it seems expensive to always copy it.
So rkn and I decided to add this comment to warn the callers.
rkn verified that no existing callers store the returned
reference.

[willchan no longer on Chromium, 2011/05/19 22:59:44]

Adding a comment is indeed the right solution here.

   const spdy::SpdySettings& Get(const HostPortPair& host_port_pair) const;
 
   // Save settings for a host.
   void Set(const HostPortPair& host_port_pair,
            const spdy::SpdySettings& settings);
 
+  // Flush structure

[wtc, 2011/05/19 22:54:37]

Nit: this comment should say a little more.

+  void Clear();
+
  private:
   typedef std::map<HostPortPair, spdy::SpdySettings> SettingsMap;
 
   SettingsMap settings_map_;
 
   DISALLOW_COPY_AND_ASSIGN(SpdySettingsStorage);
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_SETTING_STORAGE_H_