net: Socket pools prioritize requests with ignore_limits

net/socket/client_socket_pool_base.h

Index: net/socket/client_socket_pool_base.h
===================================================================
--- net/socket/client_socket_pool_base.h	(revision 203448)
+++ net/socket/client_socket_pool_base.h	(working copy)
@@ -443,6 +443,10 @@
   typedef std::map<const ClientSocketHandle*, CallbackResultPair>
       PendingCallbackMap;
 
+  // Inserts the request into the queue based on order they will receive
+  // sockets.  Sockets which ignore the socket pool limits are first.  Then

[eroman, 2013/06/04 20:42:21]

[no action necessary] my impression is that it is more common to use one space
after periods in chromium code.

nevertheless, +1 for adding more comments.

[mmenke, 2013/06/05 04:32:24]

You're definitely right about single space comments being more common, but there
are a large number of double space comments all over the place, including in
this file.  In this case, I just moved the comment from the cc file and updated
it - and it was using double spaces before (Which is, admittedly, what I
generally use myself as well).

While I think there are some places where comment consistency matters
(Capitalization of SPDY, for instance), I think it's a bit too late to worry
about comment spacing.  That having been said, I have gone ahead and switched to
single spaces in this CL.

+  // requests are sorted by priority, with higher priorities closer to the
+  // front.  Older requests are prioritized over requests of equal priority.
   static void InsertRequestIntoQueue(const Request* r,
                                      RequestQueue* pending_requests);
   static const Request* RemoveRequestFromQueue(const RequestQueue::iterator& it,