Use MakeAllInclusive() in storage partition cookie unittests

Modify storage partition cookie helper functions to use
MakeAllInclusive() instead of the CookieOptions() in order to prevent
test breakage when cookies become SameSite-by-Default.

MakeAllInclusive() has a SameSite context of STRICT instead
of CookieOptions()'s CROSS_SITE.

Bug: 1016512
Change-Id: Id826eeaf0bea5c0bd60dc6380f57fd419d62388d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879561Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Steven Bingler <bingler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709227}

content/browser/storage_partition_impl_unittest.cc

@@ -144,7 +144,7 @@ class RemoveCookieTester {
   bool ContainsCookie() {
     get_cookie_success_ = false;
     storage_partition_->GetCookieManagerForBrowserProcess()->GetCookieList(
-        kOrigin1.GetURL(), net::CookieOptions(),
+        kOrigin1.GetURL(), net::CookieOptions::MakeAllInclusive(),
         base::BindOnce(&RemoveCookieTester::GetCookieListCallback,
                        base::Unretained(this)));
     await_completion_.BlockUntilNotified();
@@ -157,7 +157,7 @@ class RemoveCookieTester {
         kOrigin1.GetURL(), "A=1", base::Time::Now(),
         base::nullopt /* server_time */, &status));
     storage_partition_->GetCookieManagerForBrowserProcess()->SetCanonicalCookie(
-        *cc, kOrigin1.scheme(), net::CookieOptions(),
+        *cc, kOrigin1.scheme(), net::CookieOptions::MakeAllInclusive(),
         base::BindOnce(&RemoveCookieTester::SetCookieCallback,
                        base::Unretained(this)));
     await_completion_.BlockUntilNotified();