Support POST in browser navaigation.

chrome/browser/ui/browser_navigator.h

Index: chrome/browser/ui/browser_navigator.h
===================================================================
--- chrome/browser/ui/browser_navigator.h	(revision 215313)
+++ chrome/browser/ui/browser_navigator.h	(working copy)
@@ -7,6 +7,8 @@
 
 #include <string>
 
+#include "base/memory/ref_counted.h"
+#include "base/memory/ref_counted_memory.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/global_request_id.h"
@@ -22,7 +24,7 @@
 
 namespace content {
 class WebContents;
-}
+}  // namespace content

[jam, 2013/08/03 05:23:22]

ditto.

in general, please avoid making stylistic nits like this, the old code was fine

[Johnny(Jianning) Ding, 2013/08/03 18:05:42]

Done.

 
 namespace chrome {
 
@@ -63,6 +65,15 @@
   GURL url;
   content::Referrer referrer;
 
+  // Indicates whether this navigation will be sent using POST.
+  // The POST method is limited support for basic POST data by leveraging
+  // NavigationController::LOAD_TYPE_BROWSER_INITIATED_HTTP_POST.
+  // It is not for things like file uploads.
+  bool uses_post;
+
+  // The post data when the navigation uses POST.
+  scoped_refptr<base::RefCountedMemory> browser_initiated_post_data;

[joth, 2013/08/03 01:21:11]

note you could fwd declare rather than include ref_counted_memory

[Johnny(Jianning) Ding, 2013/08/03 01:44:53]

I tried to include ref_counted_memory, but it ends up needing to add 
#include "base/memory/ref_counted_memory.h" to all files which include
"browser_navigator.h". So I gave up.

[joth, 2013/08/03 01:55:47]

Ah yes. the "normal" solution would be to give it explicit copy c'tor and
assignment operator defined inside the .cc file, but I think your approach is
probably just as good. (avoids the need to remember to include assigning every
single member into those methods)

+
   // Extra headers to add to the request for this page.  Headers are
   // represented as "<name>: <value>" and separated by \r\n.  The entire string
   // is terminated by \r\n.  May be empty if no extra headers are needed.