[CookieStore] Upgrading host-based deleting to predicate-based deleting.

content/browser/storage_partition_impl.cc

Index: content/browser/storage_partition_impl.cc
diff --git a/content/browser/storage_partition_impl.cc b/content/browser/storage_partition_impl.cc
index e102445330aa80d3d0e18e50d1cca68e3514e013..a96770b1a3b07312ea1abe880e00742f86bcf66e 100644
--- a/content/browser/storage_partition_impl.cc
+++ b/content/browser/storage_partition_impl.cc
@@ -66,10 +66,10 @@ void ClearCookiesOnIOThread(
         end,
         base::Bind(&OnClearedCookies, callback));
   } else {
-    cookie_store->DeleteAllCreatedBetweenForHostAsync(
-        begin,
-        end,
-        storage_origin, base::Bind(&OnClearedCookies, callback));
+    cookie_store->DeleteAllCreatedBetweenWithPredicateAsync(

[Mike West, 2016/04/01 07:40:52]

Why do we only clear host cookies here? I understand that it shouldn't be a
behavioral change, but I don't understand the current behavior.

[dmurph, 2016/04/01 23:16:45]

I have no idea. I suspect that we might be able to get rid of the host-specific
code path all together, but it's a little murky.

+        begin, end,
+        net::CookieStore::CreatePredicateForHostCookies(storage_origin),
+        base::Bind(&OnClearedCookies, callback));
   }
 }