Forwarding POST body into renderer after a cross-site transfer.

content/public/browser/navigation_handle.h

 // Copyright 2015 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_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 
 #include <memory>
 
 #include "content/common/content_export.h"
@@ skipping 71 matching lines @@
   // browser process. Corresponds to Navigation Timing API.
   virtual const base::TimeTicks& NavigationStart() = 0;
 
   // Parameters available at network request start time ------------------------
   //
   // The following parameters are only available when the network request is
   // made for the navigation (or at commit time if no network request is made).
   // This corresponds to NavigationThrottle::WillSendRequest. They should not
   // be queried before that.
 
-  // Whether the navigation is a POST or a GET. This may change during the
-  // navigation when encountering a server redirect.
-  virtual bool IsPost() = 0;
+  // HTTP method of the navigation (i.e. a POST or a GET). This may change
+  // during the navigation when encountering a server redirect.
+  virtual const std::string& GetMethod() = 0;

[Łukasz Anforowicz, 2016/06/01 16:28:38]

Hmmm... does the IsPost -> GetMethod change still look good in light of the
discussion we had in the other, WIP CL [1]?  I believe that we can only deal
with "GET" and "POST" here, right?

[1] https://codereview.chromium.org/2022483003/

[Charlie Reis, 2016/06/01 23:46:32]

Heh.  I think we just need to pick an approach and document why, so that we
don't flip flop between them any further.  :)  I'm ok with either approach.

If we can't navigate to something other than a GET or a POST and we want to
enforce that embedders don't try to, then keeping it as a bool seems reasonable.
 Let's document it if so.

[Łukasz Anforowicz, 2016/06/02 22:07:04]

In this case, let me go back to NavigationHandle::IsPost :-).  I've also added a
comment explaining why we have IsPost rather than GetMethod.

 
   // Returns a sanitized version of the referrer for this request.
   virtual const Referrer& GetReferrer() = 0;
 
   // Whether the navigation was initiated by a user gesture. Note that this
   // will return false for browser-initiated navigations.
   // TODO(clamy): when PlzNavigate launches, this should return true for
   // browser-initiated navigations.
   virtual bool HasUserGesture() = 0;
 
@@ skipping 75 matching lines @@
 
   // The NavigationData that the embedder returned from
   // ResourceDispatcherHostDelegate::GetNavigationData during commit. This will
   // be a clone of the NavigationData.
   virtual NavigationData* GetNavigationData() = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_