Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(131)

Side by Side Diff: content/browser/net/quota_policy_cookie_store_unittest.cc

Issue 1773133002: SameSite: Implement 'Strict'/'Lax' attribute parsing. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mmenke@ Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/files/file_util.h" 6 #include "base/files/file_util.h"
7 #include "base/files/scoped_temp_dir.h" 7 #include "base/files/scoped_temp_dir.h"
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "base/stl_util.h" 10 #include "base/stl_util.h"
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 store_ = new QuotaPolicyCookieStore(sqlite_store.get(), storage_policy); 89 store_ = new QuotaPolicyCookieStore(sqlite_store.get(), storage_policy);
90 Load(cookies); 90 Load(cookies);
91 } 91 }
92 92
93 // Adds a persistent cookie to store_. 93 // Adds a persistent cookie to store_.
94 void AddCookie(const std::string& name, 94 void AddCookie(const std::string& name,
95 const std::string& value, 95 const std::string& value,
96 const std::string& domain, 96 const std::string& domain,
97 const std::string& path, 97 const std::string& path,
98 const base::Time& creation) { 98 const base::Time& creation) {
99 store_->AddCookie( 99 store_->AddCookie(net::CanonicalCookie(
100 net::CanonicalCookie( 100 GURL(), name, value, domain, path, creation, creation, creation, false,
101 GURL(), name, value, domain, path, creation, creation, creation, 101 false, net::CookieSameSite::DEFAULT_MODE,
102 false, false, false, net::COOKIE_PRIORITY_DEFAULT)); 102 net::COOKIE_PRIORITY_DEFAULT));
103 } 103 }
104 104
105 void DestroyStore() { 105 void DestroyStore() {
106 store_ = nullptr; 106 store_ = nullptr;
107 // Ensure that |store_|'s destructor has run by shutting down the pool and 107 // Ensure that |store_|'s destructor has run by shutting down the pool and
108 // then forcing the pool to be destructed. This will ensure that all the 108 // then forcing the pool to be destructed. This will ensure that all the
109 // tasks that block pool shutdown (e.g. |store_|'s cleanup) have run before 109 // tasks that block pool shutdown (e.g. |store_|'s cleanup) have run before
110 // yielding control. 110 // yielding control.
111 pool_owner_->pool()->FlushForTesting(); 111 pool_owner_->pool()->FlushForTesting();
112 pool_owner_.reset(new base::SequencedWorkerPoolOwner(3, "Background Pool")); 112 pool_owner_.reset(new base::SequencedWorkerPoolOwner(3, "Background Pool"));
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 // Reload and test for persistence. 280 // Reload and test for persistence.
281 STLDeleteElements(&cookies); 281 STLDeleteElements(&cookies);
282 CreateAndLoad(storage_policy.get(), &cookies); 282 CreateAndLoad(storage_policy.get(), &cookies);
283 EXPECT_EQ(0U, cookies.size()); 283 EXPECT_EQ(0U, cookies.size());
284 284
285 STLDeleteElements(&cookies); 285 STLDeleteElements(&cookies);
286 } 286 }
287 287
288 } // namespace 288 } // namespace
289 } // namespace content 289 } // namespace content
OLDNEW
« no previous file with comments | « components/signin/core/browser/gaia_cookie_manager_service.cc ('k') | ios/net/cookies/cookie_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698