Don't persist HPKP if PrivacyMode is enabled.

net/socket_stream/socket_stream.h

Index: net/socket_stream/socket_stream.h
diff --git a/net/socket_stream/socket_stream.h b/net/socket_stream/socket_stream.h
index 90aeb8c54b9bdff0f7873539c465ac848f6764f7..44d18a710dd258424d0a5a21819d6291445e0c4a 100644
--- a/net/socket_stream/socket_stream.h
+++ b/net/socket_stream/socket_stream.h
@@ -124,6 +124,7 @@ class NET_EXPORT SocketStream
   void SetUserData(const void* key, UserData* data);
 
   const GURL& url() const { return url_; }
+  GURL url_for_cookies() const { return GetURLForCookies(url_); }
   bool is_secure() const;
   const AddressList& address_list() const { return addresses_; }
   Delegate* delegate() const { return delegate_; }
@@ -180,6 +181,9 @@ class NET_EXPORT SocketStream
   // actions on alert dialog or browser cached such kinds of user actions.
   void ContinueDespiteError();
 
+  // Replace ws(s) URL Schema with http(s) for cookie policy checking.

[Ryan Sleevi, 2013/09/13 22:14:48]

1) Why is this public
2) The documentation describes what it does in terms of how it's implemented,
rather than how it should be used. ISTM the description should focus less on the
ws/https aspect.

[mef, 2013/09/16 15:00:08]

It is public to be accessible from SocketStreamJob::CreateSocketStreamJob.
Should it be something like: 'Get URL for cookie policy checking based on Web
Socket URL'?

[Ryan Sleevi, 2013/09/17 18:53:03]

// Returns the URL to be used for cookie policy checking. Note that
// this may be different than |url|; for example, the cookie policy
// for a ws:// URL is based upon the http:// scheme for that
// host:port pair.

[mef, 2013/09/17 20:38:37]

Done.

+  static GURL GetURLForCookies(const GURL& url);
+
  protected:
   friend class base::RefCountedThreadSafe<SocketStream>;
   virtual ~SocketStream();