NavigationSimulator: add support for GlobalRequestIds

content/public/test/navigation_simulator.cc

Index: content/public/test/navigation_simulator.cc
diff --git a/content/public/test/navigation_simulator.cc b/content/public/test/navigation_simulator.cc
index 3530e615c36ffc798ed6ea973f01fec0eea03410..48368a0a6517cc7ef6f044b8b06672c61c2a5cd1 100644
--- a/content/public/test/navigation_simulator.cc
+++ b/content/public/test/navigation_simulator.cc
@@ -10,6 +10,7 @@
 #include "content/browser/frame_host/navigation_handle_impl.h"
 #include "content/browser/frame_host/navigation_request.h"
 #include "content/common/frame_messages.h"
+#include "content/public/browser/global_request_id.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/browser_side_navigation_policy.h"
@@ -251,9 +252,12 @@ void NavigationSimulator::Commit() {
   // Note that the handle's state can be CANCELING if a throttle cancelled it
   // synchronously in PrepareForCommit.
   if (handle_->state_for_testing() < NavigationHandleImpl::CANCELING) {
+    static int request_id = 0;

[Charlie Reis, 2017/05/15 20:07:43]

Are there any concerns with this overlapping with other request IDs in tests? 
Or is that not a concern since this only gets used in unit tests?

[Charlie Harrison, 2017/05/15 21:45:54]

I think this could theoretically overlap with request IDs in unit tests. I'm not
sure how much of a problem it is though, especially since consumers know that
these navigations are not really backed by URLRequests.

We could do something fancy like start at INT_MAX and subtract if you'd like.

[Charlie Reis, 2017/05/15 22:02:38]

Maybe just start it at 1000 to reduce the chance of overlap in the common case? 
(That's assuming there isn't any code comparing request IDs to each other, of
course.)

No big deal either way-- just trying to make it slightly less likely to
encounter odd unit test problems.

[Charlie Harrison, 2017/05/16 03:41:42]

Done.

+    GlobalRequestID global_id(render_frame_host_->GetProcess()->GetID(),
+                              ++request_id);
     handle_->WillProcessResponse(
         render_frame_host_, scoped_refptr<net::HttpResponseHeaders>(),
-        net::HttpResponseInfo::ConnectionInfo(), SSLStatus(), GlobalRequestID(),
+        net::HttpResponseInfo::ConnectionInfo(), SSLStatus(), global_id,
         false /* should_replace_current_entry */, false /* is_download */,
         false /* is_stream */, base::Closure(),
         base::Callback<void(NavigationThrottle::ThrottleCheckResult)>());