To fix the cross-site post submission bug.

chrome/browser/extensions/isolated_app_browsertest.cc

 // Copyright (c) 2012 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.
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/automation/automation_util.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 57 matching lines @@
 
  private:
   virtual void SetUpCommandLine(CommandLine* command_line) {
     ExtensionBrowserTest::SetUpCommandLine(command_line);
     command_line->AppendSwitch(switches::kEnableExperimentalExtensionApis);
   }
 };
 
 }  // namespace
 
+IN_PROC_BROWSER_TEST_F(IsolatedAppTest, CrossSiteDataPost) {

[irobert, 2012/11/09 19:07:57]

This Test is for the DataType Post Submission.
I am still trying to figure out the test for FileType Post submission. I think
we have to simulate the file upload request using native C++ code instead of
javascript since it cannot construct a FileType post data.

+  host_resolver()->AddRule("*", "127.0.0.1");
+  ASSERT_TRUE(test_server()->Start());
+
+  ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("isolated_apps/app1")));
+
+  GURL base_url = test_server()->GetURL("files/extensions/isolated_apps/");
+  GURL::Replacements replace_host;
+  std::string host_str("localhost");  // Must stay in scope with replace_host.
+  replace_host.SetHostStr(host_str);
+  base_url = base_url.ReplaceComponents(replace_host);
+  std::string resolve_url ="app1/main.html";
+  ui_test_utils::NavigateToURLWithDisposition(
+      browser(), base_url.Resolve(resolve_url),
+      CURRENT_TAB, ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
+
+  WebContents* tab0 = chrome::GetWebContentsAt(browser(), 0);
+  pid_t old_process = tab0->GetRenderProcessHost()->GetHandle();
+
+  std::string data = "post \0\ndata";
+  GURL echo_url = test_server()->GetURL("echoall");
+  std::string jsSubmit = "(function submitform() {";
+  jsSubmit.append("var form = document.createElement(\"form\");");
+  jsSubmit.append("form.setAttribute(\"method\", \"POST\");");
+  jsSubmit.append("form.setAttribute(\"action\", \"");
+  jsSubmit.append(echo_url.spec().c_str());
+  jsSubmit.append("\");var hiddenField = document.createElement(\"input\");");
+  jsSubmit.append("hiddenField.setAttribute(\"type\", \"hidden\");");
+  jsSubmit.append("hiddenField.setAttribute(\"name\", \"data\");");
+  jsSubmit.append("hiddenField.setAttribute(\"value\", \"" + data + "\");");
+  jsSubmit.append("form.appendChild(hiddenField);");
+  jsSubmit.append("document.body.appendChild(form);");
+  jsSubmit.append("form.submit();");
+  jsSubmit.append("})()");
+
+  ASSERT_TRUE(ExecuteJavaScript(
+      chrome::GetWebContentsAt(browser(), 0)->GetRenderViewHost(),
+      L"",
+      ASCIIToWide(jsSubmit)));
+
+  const string16 expected_title = ASCIIToUTF16("echoall");
+  content::TitleWatcher title_watcher(tab0, expected_title);
+  string16 actual_title = title_watcher.WaitAndGetTitle();
+  EXPECT_EQ(expected_title, actual_title);
+
+  // Make Sure we are in the new process
+  WebContents* tab1 = chrome::GetWebContentsAt(browser(), 0);
+  EXPECT_NE(old_process,
+            tab1->GetRenderProcessHost()->GetHandle());
+
+  std::string getResult = "window.domAutomationController.send(";
+  getResult.append("document.getElementsByTagName('pre')[0].firstChild.data);");
+  std::string value;
+  ASSERT_TRUE(ExecuteJavaScriptAndExtractString(
+      chrome::GetWebContentsAt(browser(), 0)->GetRenderViewHost(),
+      L"",
+      ASCIIToWide(getResult),
+      &value));
+  EXPECT_EQ("data="+data+"\n", value);
+}
+
 // Tests that cookies set within an isolated app are not visible to normal
 // pages or other apps.
 //
 // TODO(ajwong): Also test what happens if an app spans multiple sites in its
 // extent.  These origins should also be isolated, but still have origin-based
 // separation as you would expect.
 IN_PROC_BROWSER_TEST_F(IsolatedAppTest, CookieIsolation) {
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
@@ skipping 343 matching lines @@
   EXPECT_EQ("ss_app2", result);
 
   ui_test_utils::NavigateToURLWithDisposition(
       browser(), base_url.Resolve("non_app/main.html"),
       CURRENT_TAB, ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
   ASSERT_TRUE(ExecuteJavaScriptAndExtractString(
       chrome::GetWebContentsAt(browser(), 0)->GetRenderViewHost(),
       L"", kRetrieveSessionStorage.c_str(), &result));
   EXPECT_EQ("ss_normal", result);
 }