Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(192)

Unified Diff: content/browser/frame_host/navigation_controller_impl_browsertest.cc

Issue 2088793002: Revert of OpenURL post data handling. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@post-data-my-stuff
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/browser/frame_host/navigator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/frame_host/navigation_controller_impl_browsertest.cc
diff --git a/content/browser/frame_host/navigation_controller_impl_browsertest.cc b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
index 9b5f8063fda4a7ff5d1c3d6a05c4280a39ffb2e2..158527000472114097e1fc947c08aa6980436980 100644
--- a/content/browser/frame_host/navigation_controller_impl_browsertest.cc
+++ b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
@@ -29,7 +29,6 @@
#include "content/public/browser/web_contents_observer.h"
#include "content/public/common/bindings_policy.h"
#include "content/public/common/browser_side_navigation_policy.h"
-#include "content/public/common/renderer_preferences.h"
#include "content/public/common/url_constants.h"
#include "content/public/test/browser_test_utils.h"
#include "content/public/test/content_browser_test.h"
@@ -4579,41 +4578,4 @@
}
}
-// Tests that POST body is not lost when decidePolicyForNavigation tells the
-// renderer to route the request via FrameHostMsg_OpenURL sent to the browser.
-// See also https://crbug.com/344348.
-IN_PROC_BROWSER_TEST_F(NavigationControllerBrowserTest, PostViaOpenUrlMsg) {
- GURL main_url(
- embedded_test_server()->GetURL("/form_that_posts_to_echoall.html"));
- EXPECT_TRUE(NavigateToURL(shell(), main_url));
-
- // Ask the renderer to go through OpenURL FrameHostMsg_OpenURL IPC message.
- // Without this, the test wouldn't repro https://crbug.com/344348.
- shell()
- ->web_contents()
- ->GetMutableRendererPrefs()
- ->browser_handles_all_top_level_requests = true;
- shell()->web_contents()->GetRenderViewHost()->SyncRendererPrefs();
-
- // Submit the form.
- TestNavigationObserver form_post_observer(shell()->web_contents(), 1);
- EXPECT_TRUE(ExecuteScript(shell()->web_contents(),
- "document.getElementById('form').submit();"));
- form_post_observer.Wait();
-
- // Verify that we arrived at the expected location.
- GURL target_url(embedded_test_server()->GetURL("/echoall"));
- EXPECT_EQ(target_url, shell()->web_contents()->GetLastCommittedURL());
-
- // Verify that POST body was correctly passed to the server and ended up in
- // the body of the page.
- std::string body;
- EXPECT_TRUE(ExecuteScriptAndExtractString(
- shell()->web_contents(),
- "window.domAutomationController.send("
- "document.getElementsByTagName('pre')[0].innerText);",
- &body));
- EXPECT_EQ("text=value\n", body);
-}
-
} // namespace content
« no previous file with comments | « no previous file | content/browser/frame_host/navigator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698