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

Side by Side Diff: chrome/browser/autofill/autofill_external_delegate_unittest.cc

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 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) 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 #include <vector> 5 #include <vector>
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/string16.h" 9 #include "base/string16.h"
10 #include "chrome/browser/autofill/autofill_manager.h" 10 #include "chrome/browser/autofill/autofill_manager.h"
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 48
49 MOCK_METHOD0(ClearPreviewedForm, void()); 49 MOCK_METHOD0(ClearPreviewedForm, void());
50 50
51 MOCK_METHOD1(EnsurePopupForElement, void(const gfx::Rect& element_bounds)); 51 MOCK_METHOD1(EnsurePopupForElement, void(const gfx::Rect& element_bounds));
52 52
53 MOCK_METHOD0(HideAutofillPopup, void()); 53 MOCK_METHOD0(HideAutofillPopup, void());
54 }; 54 };
55 55
56 class MockAutofillManager : public AutofillManager { 56 class MockAutofillManager : public AutofillManager {
57 public: 57 public:
58 explicit MockAutofillManager(content::WebContents* web_contents, 58 MockAutofillManager(content::WebContents* web_contents,
59 autofill::AutofillManagerDelegate* delegate) 59 autofill::AutofillManagerDelegate* delegate)
60 // Force to use the constructor designated for unit test, but we don't 60 // Force to use the constructor designated for unit test, but we don't
61 // really need personal_data in this test so we pass a NULL pointer. 61 // really need personal_data in this test so we pass a NULL pointer.
62 : AutofillManager(web_contents, delegate, NULL) { 62 : AutofillManager(web_contents, delegate, NULL) {
63 } 63 }
64 64
65 MOCK_METHOD4(OnFillAutofillFormData, 65 MOCK_METHOD4(OnFillAutofillFormData,
66 void(int query_id, 66 void(int query_id,
67 const FormData& form, 67 const FormData& form,
68 const FormFieldData& field, 68 const FormFieldData& field,
69 int unique_id)); 69 int unique_id));
(...skipping 214 matching lines...) Expand 10 before | Expand all | Expand 10 after
284 EXPECT_CALL(*autofill_manager_, OnFillAutofillFormData(_, _, _, _)); 284 EXPECT_CALL(*autofill_manager_, OnFillAutofillFormData(_, _, _, _));
285 285
286 EXPECT_CALL(*external_delegate_, HideAutofillPopup()); 286 EXPECT_CALL(*external_delegate_, HideAutofillPopup());
287 287
288 // This should trigger a call to hide the popup since 288 // This should trigger a call to hide the popup since
289 // we've selected an option. 289 // we've selected an option.
290 external_delegate_->DidAcceptSuggestion( 290 external_delegate_->DidAcceptSuggestion(
291 suggestions[0], 291 suggestions[0],
292 WebAutofillClient::MenuItemIDPasswordEntry); 292 WebAutofillClient::MenuItemIDPasswordEntry);
293 } 293 }
OLDNEW
« no previous file with comments | « chrome/browser/autofill/autocomplete_history_manager_unittest.cc ('k') | chrome/browser/autofill/autofill_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698