OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2013 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_DRIVER_H_ | |
6 #define COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_DRIVER_H_ | |
7 | |
8 namespace content { | |
9 class WebContents; | |
10 } | |
11 | |
12 namespace autofill { | |
13 | |
14 // Interface that provides access to the driver-level context in which autofill | |
Ilya Sherman
2013/06/12 00:07:46
nit: "autofill" -> "Autofill" (feature name)
blundell
2013/06/12 16:29:37
Done.
| |
15 // is operating. A concrete implementation must be provided by the driver. | |
16 class AutofillDriver { | |
blundell
2013/06/11 15:35:47
I envision this interface being the one that will
Ilya Sherman
2013/06/12 00:07:46
AutofillDriver sounds fine to me. If the interfac
blundell
2013/06/12 16:29:37
SGTM.
On 2013/06/12 00:07:46, Ilya Sherman wrote:
| |
17 public: | |
18 // TODO(blundell): Remove this method once shared code no longer needs to | |
19 // know about WebContents. | |
20 virtual content::WebContents* GetWebContents() = 0; | |
21 virtual ~AutofillDriver() {} | |
Ilya Sherman
2013/06/12 00:07:46
nit: Destructor belongs above other methods.
blundell
2013/06/12 16:29:37
Done.
| |
22 }; | |
23 | |
24 } // namespace autofill | |
25 | |
26 #endif // COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_DRIVER_H_ | |
OLD | NEW |