Add atomic sequence number to ChannelIDService and check that in URLRequestHttpJob logging

net/ssl/channel_id_service.h

Index: net/ssl/channel_id_service.h
diff --git a/net/ssl/channel_id_service.h b/net/ssl/channel_id_service.h
index eb4a92ae6f5ed83dfedb7308c7544c816bff9a9d..beda9910f1bc71c8b8756d378f6f55969fa144b1 100644
--- a/net/ssl/channel_id_service.h
+++ b/net/ssl/channel_id_service.h
@@ -129,6 +129,10 @@ class NET_EXPORT ChannelIDService
   // Returns the backing ChannelIDStore.
   ChannelIDStore* GetChannelIDStore();
 
+  // Returns an ID that is unique across all instances of ChannelIDService in
+  // this process. TODO(nharper): remove this once crbug.com/548423 is resolved.
+  int GetUniqueID() { return id_; }

[mattm, 2016/03/09 21:33:01]

const method

[nharper, 2016/03/09 21:52:07]

Done.

+
   // Public only for unit testing.
   int channel_id_count();
   uint64_t requests() const { return requests_; }
@@ -171,6 +175,7 @@ class NET_EXPORT ChannelIDService
 
   scoped_ptr<ChannelIDStore> channel_id_store_;
   scoped_refptr<base::TaskRunner> task_runner_;
+  int id_;

[mattm, 2016/03/09 21:33:01]

can this be const?

[nharper, 2016/03/09 21:52:07]

Yes. Done.

 
   // inflight_ maps from a server to an active generation which is taking
   // place.