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

Side by Side Diff: components/autofill/content/browser/autofill_driver_impl.h

Issue 17572015: Begin abstracting sending of IPC from autofill core code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Response to review, add tests Created 7 years, 5 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 2013 The Chromium Authors. All rights reserved. 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 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_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_ 5 #ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_
6 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_ 6 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 25 matching lines...) Expand all
36 public: 36 public:
37 static void CreateForWebContentsAndDelegate( 37 static void CreateForWebContentsAndDelegate(
38 content::WebContents* contents, 38 content::WebContents* contents,
39 autofill::AutofillManagerDelegate* delegate, 39 autofill::AutofillManagerDelegate* delegate,
40 const std::string& app_locale, 40 const std::string& app_locale,
41 AutofillManager::AutofillDownloadManagerState enable_download_manager); 41 AutofillManager::AutofillDownloadManagerState enable_download_manager);
42 static AutofillDriverImpl* FromWebContents(content::WebContents* contents); 42 static AutofillDriverImpl* FromWebContents(content::WebContents* contents);
43 43
44 // AutofillDriver: 44 // AutofillDriver:
45 virtual content::WebContents* GetWebContents() OVERRIDE; 45 virtual content::WebContents* GetWebContents() OVERRIDE;
46 virtual bool RendererIsAvailable() OVERRIDE;
47 virtual void SendFormDataToRenderer(int query_id, const FormData& data)
48 OVERRIDE;
Ilya Sherman 2013/06/26 22:54:58 nit: IMO, it would be slightly cleaner to wrap the
blundell 2013/06/27 21:58:27 Done.
46 49
47 AutofillExternalDelegate* autofill_external_delegate() { 50 AutofillExternalDelegate* autofill_external_delegate() {
48 return autofill_external_delegate_.get(); 51 return autofill_external_delegate_.get();
49 } 52 }
50 53
51 // Sets the external delegate to |delegate| both within this class and in the 54 // Sets the external delegate to |delegate| both within this class and in the
52 // shared Autofill code. Takes ownership of |delegate|. 55 // shared Autofill code. Takes ownership of |delegate|.
53 void SetAutofillExternalDelegate( 56 void SetAutofillExternalDelegate(
54 scoped_ptr<AutofillExternalDelegate> delegate); 57 scoped_ptr<AutofillExternalDelegate> delegate);
55 58
(...skipping 23 matching lines...) Expand all
79 scoped_ptr<AutofillExternalDelegate> autofill_external_delegate_; 82 scoped_ptr<AutofillExternalDelegate> autofill_external_delegate_;
80 83
81 // AutofillManager instance via which this object drives the shared Autofill 84 // AutofillManager instance via which this object drives the shared Autofill
82 // code. 85 // code.
83 scoped_ptr<AutofillManager> autofill_manager_; 86 scoped_ptr<AutofillManager> autofill_manager_;
84 }; 87 };
85 88
86 } // namespace autofill 89 } // namespace autofill
87 90
88 #endif // COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_ 91 #endif // COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698