Replace std::string::find with base::StartsWith when comparing cookie paths.

net/cookies/cookie_store_unittest.h

 // 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.
 
 #ifndef NET_COOKIES_COOKIE_STORE_UNITTEST_H_
 #define NET_COOKIES_COOKIE_STORE_UNITTEST_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 1282 matching lines @@
 
   ASSERT_TRUE(++it != cookies.end());
   EXPECT_EQ(this->http_foo_com_.host(), it->Domain());
   EXPECT_EQ("/", it->Path());
   EXPECT_EQ("C", it->Name());
   EXPECT_EQ("D", it->Value());
 
   ASSERT_TRUE(++it == cookies.end());
 }
 
+TYPED_TEST_P(CookieStoreTest, DeleteCookieAsync) {
+  scoped_refptr<CookieStore> cs(this->GetCookieStore());
+
+  EXPECT_TRUE(
+      this->SetCookie(cs.get(), this->http_www_google_.url(), "A=A1; path=/"));
+  EXPECT_TRUE(this->SetCookie(cs.get(), this->http_www_google_.url(),
+                              "A=A2; path=/foo"));
+  EXPECT_TRUE(this->SetCookie(cs.get(), this->http_www_google_.url(),
+                              "A=A3; path=/bar"));

[mmenke, 2016/03/02 20:02:01]

I tried adding a "path=/fo" test, but iOS apparently has different matching
behavior here (Applies to getting cookies, too, I think)

[Mike West, 2016/03/03 18:56:28]

What? What does iOS do?

[mmenke, 2016/03/03 19:02:58]

iOS seems to return (Or at least delete, think return, too, though) a cookie set
on "/fo" when passed a path of "/foo"

+  EXPECT_TRUE(
+      this->SetCookie(cs.get(), this->http_www_google_.url(), "B=B1; path=/"));
+  EXPECT_TRUE(this->SetCookie(cs.get(), this->http_www_google_.url(),
+                              "B=B2; path=/foo"));
+  EXPECT_TRUE(this->SetCookie(cs.get(), this->http_www_google_.url(),
+                              "B=B3; path=/bar"));
+
+  this->DeleteCookie(cs.get(), this->http_www_google_.AppendPath("foo/bar"),
+                     "A");
+
+  CookieList cookies = this->GetAllCookies(cs.get());
+  size_t expected_size = 4;
+  EXPECT_EQ(expected_size, cookies.size());
+  for (const auto& cookie : cookies) {
+    EXPECT_NE("A1", cookie.Value());
+    EXPECT_NE("A2", cookie.Value());
+  }
+}
+
 TYPED_TEST_P(CookieStoreTest, DeleteCanonicalCookieAsync) {
   scoped_refptr<CookieStore> cs(this->GetCookieStore());
 
   // Set two cookies with the same name, and make sure both are set.
   EXPECT_TRUE(
       this->SetCookie(cs.get(), this->http_www_google_.url(), "A=B;Path=/foo"));
   EXPECT_TRUE(
       this->SetCookie(cs.get(), this->http_www_google_.url(), "A=C;Path=/bar"));
   EXPECT_EQ(2u, this->GetAllCookies(cs.get()).size());
   EXPECT_EQ("A=B", this->GetCookies(cs.get(), this->www_google_foo_.url()));
@@ skipping 74 matching lines @@
                            HttpOnlyTest,
                            TestCookieDeletion,
                            TestDeleteAll,
                            TestDeleteAllCreatedBetween,
                            TestDeleteAllCreatedBetweenForHost,
                            TestSecure,
                            NetUtilCookieTest,
                            OverwritePersistentCookie,
                            CookieOrdering,
                            GetAllCookiesAsync,
+                           DeleteCookieAsync,
                            DeleteCanonicalCookieAsync,
                            DeleteSessionCookie);
 
 }  // namespace net
 
 #endif  // NET_COOKIES_COOKIE_STORE_UNITTEST_H_