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

content/renderer/render_frame_impl.h

Index: content/renderer/render_frame_impl.h
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index 58b4b93db25272585d5e2326ddbad736044d9f4a..91c6ff7c5f6e9bc94121d5604f5b7216432ca193 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -878,6 +878,7 @@ class CONTENT_EXPORT RenderFrameImpl
       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,