Preserving Content-Type header from http request in OpenURL path.

content/renderer/render_frame_impl.h

 // Copyright 2013 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 CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 860 matching lines @@
 #endif
 
   // Requests that the browser process navigates to |url|. If
   // |is_history_navigation_in_new_child| is true, the browser process should
   // look for a matching FrameNavigationEntry in the last committed entry to use
   // instead of |url|.
   void OpenURL(
       const GURL& url,
       bool uses_post,
       const scoped_refptr<ResourceRequestBodyImpl>& resource_request_body,
+      const std::string& extra_headers,

[nasko, 2016/10/06 18:15:01]

Why not keep this in the same order as the struct members?

[Łukasz Anforowicz, 2016/10/06 18:33:54]

The parameters should probably be in the same order as the order of fields in
content::OpenURLParams (so |extra_headers| follows |resource_request_body| and
|uses_post|).

FWIW, in the latest patchset I've tweaked the order of fields in
FrameHostMsg_OpenURL_Params, so that it matches (matches the order of method
parameters here + the order of fields in content::OpenURLParams struct).

Given above, there is no need to change things here, right?

       const Referrer& referrer,
       blink::WebNavigationPolicy policy,
       bool should_replace_current_entry,
       bool is_history_navigation_in_new_child);
 
   // Performs a navigation in the frame. This provides a unified function for
   // the current code path and the browser-side navigation path (in
   // development). Currently used by OnNavigate, with all *NavigationParams
   // provided by the browser. |stream_params| should be null.
   // PlzNavigate: used by OnCommitNavigation, with |common_params| and
@@ skipping 410 matching lines @@
       remote_associated_interfaces_;
 
   base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_