Navigation transitions: Block first response until after transitions have run.

content/browser/loader/cross_site_resource_handler.h

Index: content/browser/loader/cross_site_resource_handler.h
diff --git a/content/browser/loader/cross_site_resource_handler.h b/content/browser/loader/cross_site_resource_handler.h
index df6c7a39bbf4aacbd00a548c68811ade0ee10d07..a33884771631c459b9f12a86ca06643d168628e6 100644
--- a/content/browser/loader/cross_site_resource_handler.h
+++ b/content/browser/loader/cross_site_resource_handler.h
@@ -53,6 +53,11 @@ class CrossSiteResourceHandler : public LayeredResourceHandler {
   CONTENT_EXPORT static void SetLeakRequestsForTesting(
       bool leak_requests_for_testing);
 
+  // Navigations are deferred at response started to parse out any navigation

[nasko, 2014/05/28 22:59:00]

nit: OnResponseStarted

[shatch, 2014/05/29 21:41:22]

Done.

+  // transition link headers, and give the navigation transition (if it exists)
+  // a chance to run.
+  void ResumeResponseDeferredAtStart(int request_id);
+
  private:
   // Prepare to render the cross-site response in a new RenderViewHost, by
   // telling the old RenderViewHost to run its onunload handler.
@@ -66,6 +71,12 @@ class CrossSiteResourceHandler : public LayeredResourceHandler {
                                      ResourceResponse* response,
                                      bool* defer);
 
+  bool OnNavigationTransitionResponseStarted(int request_id,

[nasko, 2014/05/28 22:59:00]

style: parameters should be on separate lines.

[shatch, 2014/05/29 21:41:22]

Done.

+      ResourceResponse* response, bool* defer);
+
+  bool OnNormalResponseStarted(int request_id, ResourceResponse* response,

[nasko, 2014/05/28 22:59:00]

style: parameters should be on separate lines.

[shatch, 2014/05/29 21:41:22]

Done.

+      bool* defer);
+
   void ResumeOrTransfer(bool is_transfer);
   void ResumeIfDeferred();