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

Side by Side Diff: components/autofill/core/browser/test_autofill_driver.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_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_ 5 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_
6 #define COMPONENTS_AUTOFILL_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_ 6 #define COMPONENTS_AUTOFILL_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "components/autofill/core/browser/autofill_driver.h" 10 #include "components/autofill/core/browser/autofill_driver.h"
11 #include "content/public/browser/web_contents_observer.h" 11 #include "content/public/browser/web_contents_observer.h"
12 #include "testing/gmock/include/gmock/gmock.h"
12 13
13 namespace autofill { 14 namespace autofill {
14 15
15 // This class is only for easier writing of tests. All pure virtual functions 16 // This class is only for easier writing of tests. All pure virtual functions
16 // have been given empty methods. 17 // have been given empty methods.
17 // TODO(blundell): Eliminate this class being a WebContentsObserver once 18 // TODO(blundell): Eliminate this class being a WebContentsObserver once
18 // autofill shared code no longer needs knowledge of WebContents. 19 // autofill shared code no longer needs knowledge of WebContents.
19 class TestAutofillDriver : public AutofillDriver, 20 class TestAutofillDriver : public AutofillDriver,
20 public content::WebContentsObserver { 21 public content::WebContentsObserver {
21 public: 22 public:
22 explicit TestAutofillDriver(content::WebContents* web_contents); 23 explicit TestAutofillDriver(content::WebContents* web_contents);
23 virtual ~TestAutofillDriver(); 24 virtual ~TestAutofillDriver();
24 25
25 // AutofillDriver implementation. 26 // AutofillDriver implementation.
26 virtual content::WebContents* GetWebContents() OVERRIDE; 27 virtual content::WebContents* GetWebContents() OVERRIDE;
28 virtual bool RendererIsAvailable() OVERRIDE;
29
30 // Mock methods to enable testability.
31 MOCK_METHOD2(SendFormDataToRenderer, void(int query_id,
32 const FormData& data));
Ilya Sherman 2013/06/26 22:54:58 Please stub out the method rather than mocking it
blundell 2013/06/27 21:58:27 Done.
27 33
28 private: 34 private:
29 DISALLOW_COPY_AND_ASSIGN(TestAutofillDriver); 35 DISALLOW_COPY_AND_ASSIGN(TestAutofillDriver);
30 }; 36 };
31 37
32 } // namespace autofill 38 } // namespace autofill
33 39
34 #endif // COMPONENTS_AUTOFILL_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_ 40 #endif // COMPONENTS_AUTOFILL_CORE_BROWSER_TEST_AUTOFILL_DRIVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698