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

Side by Side Diff: net/cookies/cookie_monster_store_test.h

Issue 1825273002: Add more out of line copy ctors for complex classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
« no previous file with comments | « net/cert/signed_tree_head.cc ('k') | net/cookies/cookie_monster_store_test.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // This file contains test infrastructure for multiple files 5 // This file contains test infrastructure for multiple files
6 // (current cookie_monster_unittest.cc and cookie_monster_perftest.cc) 6 // (current cookie_monster_unittest.cc and cookie_monster_perftest.cc)
7 // that need to test out CookieMonster interactions with the backing store. 7 // that need to test out CookieMonster interactions with the backing store.
8 // It should only be included by test code. 8 // It should only be included by test code.
9 9
10 #ifndef NET_COOKIES_COOKIE_MONSTER_STORE_TEST_H_ 10 #ifndef NET_COOKIES_COOKIE_MONSTER_STORE_TEST_H_
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 // Constructor for LOAD and LOAD_COOKIES_FOR_KEY calls. |key| should be empty 66 // Constructor for LOAD and LOAD_COOKIES_FOR_KEY calls. |key| should be empty
67 // for LOAD_COOKIES_FOR_KEY. 67 // for LOAD_COOKIES_FOR_KEY.
68 CookieStoreCommand(Type type, 68 CookieStoreCommand(Type type,
69 const CookieMonster::PersistentCookieStore::LoadedCallback& 69 const CookieMonster::PersistentCookieStore::LoadedCallback&
70 loaded_callback, 70 loaded_callback,
71 const std::string& key); 71 const std::string& key);
72 72
73 // Constructor for ADD, UPDATE_ACCESS_TIME, and REMOVE calls. 73 // Constructor for ADD, UPDATE_ACCESS_TIME, and REMOVE calls.
74 CookieStoreCommand(Type type, const CanonicalCookie& cookie); 74 CookieStoreCommand(Type type, const CanonicalCookie& cookie);
75 75
76 CookieStoreCommand(const CookieStoreCommand& other);
77
76 ~CookieStoreCommand(); 78 ~CookieStoreCommand();
77 79
78 Type type; 80 Type type;
79 81
80 // Only non-null for LOAD and LOAD_COOKIES_FOR_KEY. 82 // Only non-null for LOAD and LOAD_COOKIES_FOR_KEY.
81 CookieMonster::PersistentCookieStore::LoadedCallback loaded_callback; 83 CookieMonster::PersistentCookieStore::LoadedCallback loaded_callback;
82 84
83 // Only non-empty for LOAD_COOKIES_FOR_KEY. 85 // Only non-empty for LOAD_COOKIES_FOR_KEY.
84 std::string key; 86 std::string key;
85 87
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 scoped_ptr<CookieMonster> CreateMonsterFromStoreForGC( 223 scoped_ptr<CookieMonster> CreateMonsterFromStoreForGC(
222 int num_secure_cookies, 224 int num_secure_cookies,
223 int num_old_secure_cookies, 225 int num_old_secure_cookies,
224 int num_non_secure_cookies, 226 int num_non_secure_cookies,
225 int num_old_non_secure_cookies, 227 int num_old_non_secure_cookies,
226 int days_old); 228 int days_old);
227 229
228 } // namespace net 230 } // namespace net
229 231
230 #endif // NET_COOKIES_COOKIE_MONSTER_STORE_TEST_H_ 232 #endif // NET_COOKIES_COOKIE_MONSTER_STORE_TEST_H_
OLDNEW
« no previous file with comments | « net/cert/signed_tree_head.cc ('k') | net/cookies/cookie_monster_store_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698