Consistently suggest ephemeral port for QUIC client

net/quic/quic_stream_factory.h

 // Copyright (c) 2012 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_QUIC_QUIC_STREAM_FACTORY_H_
 #define NET_QUIC_QUIC_STREAM_FACTORY_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 212 matching lines @@
   std::vector<std::string> cannoncial_suffixes_;
 
   QuicConfig config_;
 
   JobMap active_jobs_;
   JobRequestsMap job_requests_map_;
   RequestMap active_requests_;
 
   base::WeakPtrFactory<QuicStreamFactory> weak_factory_;
 
+  // Entropy to help consistently select the same ephemeral port for repeated
+  // connections to the same server.

[wtc, 2013/12/06 23:36:44]

This comment is confusing because it describes a port suggester. It is more
important for this comment to describe the port_entropy_ member, which deals
with user profile isolation.

[jar (doing other things), 2013/12/07 00:47:58]

Extended comment.  See if this is clearer.

+  uint64 port_entropy_;
+
   DISALLOW_COPY_AND_ASSIGN(QuicStreamFactory);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_QUIC_STREAM_FACTORY_H_