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

Side by Side Diff: components/autofill/browser/autofill_common_test.h

Issue 17465003: Fix most tests in //components/autofill/content to not depend on //chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove IO thread, seems not needed. Created 7 years, 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #ifndef COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_ 5 #ifndef COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_
6 #define COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_ 6 #define COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_
7 7
8 class Profile; 8 namespace content {
9 class BrowserContext;
10 }
9 11
10 namespace autofill { 12 namespace autofill {
11 13
12 class AutofillProfile; 14 class AutofillProfile;
13 class CreditCard; 15 class CreditCard;
14 struct FormFieldData; 16 struct FormFieldData;
15 17
16 // Common utilities shared amongst Autofill tests. 18 // Common utilities shared amongst Autofill tests.
17 namespace test { 19 namespace test {
18 20
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 // A unit testing utility that is common to a number of the Autofill unit 63 // A unit testing utility that is common to a number of the Autofill unit
62 // tests. |SetCreditCardInfo| provides a quick way to populate a credit card 64 // tests. |SetCreditCardInfo| provides a quick way to populate a credit card
63 // with c-strings. 65 // with c-strings.
64 void SetCreditCardInfo(CreditCard* credit_card, 66 void SetCreditCardInfo(CreditCard* credit_card,
65 const char* name_on_card, const char* card_number, 67 const char* name_on_card, const char* card_number,
66 const char* expiration_month, const char* expiration_year); 68 const char* expiration_month, const char* expiration_year);
67 69
68 // TODO(isherman): We should do this automatically for all tests, not manually 70 // TODO(isherman): We should do this automatically for all tests, not manually
69 // on a per-test basis: http://crbug.com/57221 71 // on a per-test basis: http://crbug.com/57221
70 // Disables or mocks out code that would otherwise reach out to system services. 72 // Disables or mocks out code that would otherwise reach out to system services.
71 void DisableSystemServices(Profile* profile); 73 void DisableSystemServices(content::BrowserContext* profile);
Ilya Sherman 2013/06/19 23:06:14 nit: Please update the variable name here.
Jói 2013/06/21 13:13:23 Done.
72 74
73 } // namespace test 75 } // namespace test
74 } // namespace autofill 76 } // namespace autofill
75 77
76 #endif // COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_ 78 #endif // COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_COMMON_TEST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698