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

Revert of Preserving Content-Type header from http request in OpenURL path. (patchset #13 id:240001 of https://codereview.chromium.org/2355023002/ )

Reason for revert:
This CL caused a regression - https://crbug.com/655568.

Original issue's description:
> Preserving extra http request headers in OpenURL navigation path.
>
> This CL makes sure that extra http request headers (e.g. in case of HTTP
> POST, the Content-Type: multipart/form-data; boundary=... header) are
> preserved when navigation uses the "OpenURL" code path.
>
> BUG=648648
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/f097ee58fe6bb4c329999f03253bc4849cc50302
> Cr-Commit-Position: refs/heads/master@{#424588}

TBR=creis@chromium.org,nasko@chromium.org,mmenke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=648648
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/375969abb781814249ca4666b90a916f52b15fa2
Cr-Commit-Position: refs/heads/master@{#425473}

[Łukasz Anforowicz, 2016-10-14 15:22:38]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2016-10-14 15:22:38]

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

[commit-bot: I haz the power, 2016-10-14 15:23:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 16:48:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 16:48:28]

Failed to apply patch for content/browser/frame_host/navigator_impl.cc:
While running git apply --index -3 -p1;
  error: patch failed: content/browser/frame_host/navigator_impl.cc:842
  Falling back to three-way merge...
  Applied patch to 'content/browser/frame_host/navigator_impl.cc' with
conflicts.
  U content/browser/frame_host/navigator_impl.cc

Patch:       content/browser/frame_host/navigator_impl.cc
Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc
b/content/browser/frame_host/navigator_impl.cc
index
531e06b0cf9d13b5130d4d88ba3b0ddf23d68725..571387d591b0916a709bc081c52a1f5741d191a5
100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -692,7 +692,6 @@
     const GURL& url,
     bool uses_post,
     const scoped_refptr<ResourceRequestBodyImpl>& body,
-    const std::string& extra_headers,
     SiteInstance* source_site_instance,
     const Referrer& referrer,
     WindowOpenDisposition disposition,
@@ -738,7 +737,6 @@
                        true /* is_renderer_initiated */);
   params.uses_post = uses_post;
   params.post_data = body;
-  params.extra_headers = extra_headers;
   params.source_site_instance = source_site_instance;
   if (redirect_chain.size() > 0)
     params.redirect_chain = redirect_chain;
@@ -781,8 +779,7 @@
     const GlobalRequestID& transferred_global_request_id,
     bool should_replace_current_entry,
     const std::string& method,
-    scoped_refptr<ResourceRequestBodyImpl> post_body,
-    const std::string& extra_headers) {
+    scoped_refptr<ResourceRequestBodyImpl> post_body) {
   // |method != "POST"| should imply absence of |post_body|.
   if (method != "POST" && post_body) {
     NOTREACHED();
@@ -842,7 +839,6 @@
     CHECK(SiteIsolationPolicy::UseSubframeNavigationEntries());
     if (controller_->GetLastCommittedEntry()) {
       entry = controller_->GetLastCommittedEntry()->Clone();
-      entry->set_extra_headers(extra_headers);
       entry->SetPageID(-1);
     } else {
       // If there's no last committed entry, create an entry for about:blank
@@ -851,7 +847,7 @@
       entry = NavigationEntryImpl::FromNavigationEntry(
           controller_->CreateNavigationEntry(
               GURL(url::kAboutBlankURL), referrer_to_use, page_transition,
-              is_renderer_initiated, extra_headers,
+              is_renderer_initiated, std::string(),
               controller_->GetBrowserContext()));
     }
     entry->AddOrUpdateFrameEntry(
@@ -864,7 +860,7 @@
     entry = NavigationEntryImpl::FromNavigationEntry(
         controller_->CreateNavigationEntry(
             dest_url, referrer_to_use, page_transition, is_renderer_initiated,
-            extra_headers, controller_->GetBrowserContext()));
+            std::string(), controller_->GetBrowserContext()));
     entry->root_node()->frame_entry->set_source_site_instance(
         static_cast<SiteInstanceImpl*>(source_site_instance));
     entry->SetRedirectChain(redirect_chain);

[Łukasz Anforowicz, 2016-10-14 18:34:35]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2016-10-14 18:35:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 18:47:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 18:47:44]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Łukasz Anforowicz, 2016-10-14 20:12:59]

Description was changed from

==========
Revert of Preserving Content-Type header from http request in OpenURL path.
(patchset #13 id:240001 of https://codereview.chromium.org/2355023002/ )

Reason for revert:
This CL caused a regression - https://crbug.com/655568.

Original issue's description:
> Preserving extra http request headers in OpenURL navigation path.
> 
> This CL makes sure that extra http request headers (e.g. in case of HTTP
> POST, the Content-Type: multipart/form-data; boundary=... header) are
> preserved when navigation uses the "OpenURL" code path.
> 
> BUG=648648
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/f097ee58fe6bb4c329999f03253bc4849cc50302
> Cr-Commit-Position: refs/heads/master@{#424588}

TBR=creis@chromium.org,nasko@chromium.org,mmenke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=648648
==========

to

==========
Revert of Preserving Content-Type header from http request in OpenURL path.
(patchset #13 id:240001 of https://codereview.chromium.org/2355023002/ )

Reason for revert:
This CL caused a regression - https://crbug.com/655568.

Original issue's description:
> Preserving extra http request headers in OpenURL navigation path.
> 
> This CL makes sure that extra http request headers (e.g. in case of HTTP
> POST, the Content-Type: multipart/form-data; boundary=... header) are
> preserved when navigation uses the "OpenURL" code path.
> 
> BUG=648648
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/f097ee58fe6bb4c329999f03253bc4849cc50302
> Cr-Commit-Position: refs/heads/master@{#424588}

TBR=creis@chromium.org,nasko@chromium.org,mmenke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=648648
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Łukasz Anforowicz, 2016-10-14 20:14:29]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2016-10-14 20:15:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 21:37:12]

Committed patchset #2 (id:230001)

[commit-bot: I haz the power, 2016-10-14 21:40:46]

Description was changed from

==========
Revert of Preserving Content-Type header from http request in OpenURL path.
(patchset #13 id:240001 of https://codereview.chromium.org/2355023002/ )

Reason for revert:
This CL caused a regression - https://crbug.com/655568.

Original issue's description:
> Preserving extra http request headers in OpenURL navigation path.
> 
> This CL makes sure that extra http request headers (e.g. in case of HTTP
> POST, the Content-Type: multipart/form-data; boundary=... header) are
> preserved when navigation uses the "OpenURL" code path.
> 
> BUG=648648
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/f097ee58fe6bb4c329999f03253bc4849cc50302
> Cr-Commit-Position: refs/heads/master@{#424588}

TBR=creis@chromium.org,nasko@chromium.org,mmenke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=648648
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert of Preserving Content-Type header from http request in OpenURL path.
(patchset #13 id:240001 of https://codereview.chromium.org/2355023002/ )

Reason for revert:
This CL caused a regression - https://crbug.com/655568.

Original issue's description:
> Preserving extra http request headers in OpenURL navigation path.
>
> This CL makes sure that extra http request headers (e.g. in case of HTTP
> POST, the Content-Type: multipart/form-data; boundary=... header) are
> preserved when navigation uses the "OpenURL" code path.
>
> BUG=648648
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/f097ee58fe6bb4c329999f03253bc4849cc50302
> Cr-Commit-Position: refs/heads/master@{#424588}

TBR=creis@chromium.org,nasko@chromium.org,mmenke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=648648
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/375969abb781814249ca4666b90a916f52b15fa2
Cr-Commit-Position: refs/heads/master@{#425473}
==========

[commit-bot: I haz the power, 2016-10-14 21:40:47]

Patchset 2 (id:??) landed as
https://crrev.com/375969abb781814249ca4666b90a916f52b15fa2
Cr-Commit-Position: refs/heads/master@{#425473}