Fix for FullTabModeIE_ChromeFrameDeleteCookieTest and issues with deleting persistent cookies as reported in bug 30786.

TEST=Run FullTabModeIE_ChromeFrameDeleteCookieTest.
BUG=32546,30786

Review URL: http://codereview.chromium.org/551101

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36831 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 20 insertions, 0 deletions

diff --git a/chrome_frame/test/html_util_unittests.cc b/chrome_frame/test/html_util_unittests.cc
index 7884638..5670055 100644
--- a/chrome_frame/test/html_util_unittests.cc
+++ b/chrome_frame/test/html_util_unittests.cc
@@ -385,3 +385,23 @@ TEST(HttpCookieTest, IdentifyDuplicateCookieTest) {
                   "BLAH", header_cookies));
 }
 
+TEST(HttpCookieTest, SetCookiePathToRootIfNotPresentTest) {
+  struct TestCase {
+    std::string input;
+    std::string expected;
+  } test_cases[] = {
+    { "", "" },
+    { "Cookie=value", "Cookie=value; path=/" },
+    { "Cookie=value;", "Cookie=value; path=/" },
+    { "Cookie=value; ", "Cookie=value; path=/" },
+    { " Cookie=value; ", "Cookie=value; path=/" },
+    { "Cookie=", "Cookie=; path=/" },
+    { "Cookie=foo; path=/bar", "Cookie=foo; path=/bar" },
+  };
+
+  for (int i = 0; i < arraysize(test_cases); ++i) {
+    std::string& cookie(test_cases[i].input);
+    URLRequestAutomationJob::SetCookiePathToRootIfNotPresent(&cookie);
+    EXPECT_EQ(cookie, test_cases[i].expected);
+  }
+}