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

Side by Side Diff: components/autofill/browser/autofill_manager_unittest.cc

Issue 15097004: Enable Autocomplete feature for chromium webview (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@setSaveFormData2
Patch Set: added a unit test 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
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 <algorithm> 5 #include <algorithm>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 2687 matching lines...) Expand 10 before | Expand all | Expand 10 after
2698 FormData results; 2698 FormData results;
2699 EXPECT_TRUE(GetAutofillFormDataFilledMessage(&page_id, &results)); 2699 EXPECT_TRUE(GetAutofillFormDataFilledMessage(&page_id, &results));
2700 ExpectFilledAddressFormElvis(page_id, results, kDefaultPageID, false); 2700 ExpectFilledAddressFormElvis(page_id, results, kDefaultPageID, false);
2701 2701
2702 // Simulate form submission. We should call into the PDM to try to save the 2702 // Simulate form submission. We should call into the PDM to try to save the
2703 // filled data. 2703 // filled data.
2704 EXPECT_CALL(personal_data_, SaveImportedProfile(::testing::_)).Times(1); 2704 EXPECT_CALL(personal_data_, SaveImportedProfile(::testing::_)).Times(1);
2705 FormSubmitted(results); 2705 FormSubmitted(results);
2706 } 2706 }
2707 2707
2708 // Test that when the personal data manager is null, the form submission flow
2709 // works as expected.
Ilya Sherman 2013/06/15 00:28:09 I don't see any code to trigger a form submission
sgurun-gerrit only 2013/06/17 21:20:32 I actually looked into it. I could not find any sa
Ilya Sherman 2013/06/18 05:56:25 Presumably you could install a test AutocompleteHi
2710 TEST_F(AutofillManagerTest, FormSubmittedNoPersonalDataManager) {
2711 autofill_manager_.reset(new TestAutofillManager(
2712 web_contents(),
2713 autofill::TabAutofillManagerDelegate::FromWebContents(web_contents()),
2714 NULL));
2715
2716 // Set up our form data.
2717 FormData form;
2718 CreateTestAddressFormData(&form);
2719 form.method = ASCIIToUTF16("GET");
2720 std::vector<FormData> forms(1, form);
2721 FormsSeen(forms);
2722 const FormFieldData& field = form.fields[0];
2723 GetAutofillSuggestions(form, field);
2724
2725 // Add some Autocomplete suggestions. We should return the autocomplete
2726 // suggestions and the warning; these will be culled by the renderer.
Ilya Sherman 2013/06/15 00:28:09 What is the warning referred to in this comment?
sgurun-gerrit only 2013/06/17 21:20:32 inherited from copy/paste. does not make sense, re
2727 std::vector<base::string16> suggestions;
2728 suggestions.push_back(ASCIIToUTF16("Jay"));
2729 suggestions.push_back(ASCIIToUTF16("Jason"));
2730 AutocompleteSuggestionsReturned(suggestions);
2731
2732 int page_id = 0;
2733 std::vector<base::string16> values;
2734 std::vector<base::string16> labels;
2735 std::vector<base::string16> icons;
2736 std::vector<int> unique_ids;
2737 EXPECT_TRUE(GetAutofillSuggestionsMessage(&page_id, &values, &labels, &icons,
2738 &unique_ids));
2739
2740 base::string16 expected_values[] = {
2741 ASCIIToUTF16("Jay"),
2742 ASCIIToUTF16("Jason")
2743 };
2744 base::string16 expected_labels[] = { base::string16(), base::string16()};
2745 base::string16 expected_icons[] = { base::string16(), base::string16()};
2746 int expected_unique_ids[] = {0, 0};
2747 ExpectSuggestions(page_id, values, labels, icons, unique_ids,
2748 kDefaultPageID, arraysize(expected_values), expected_values,
2749 expected_labels, expected_icons, expected_unique_ids);
2750 }
2751
2708 // Test that we are able to save form data when forms are submitted and we only 2752 // Test that we are able to save form data when forms are submitted and we only
2709 // have server data for the field types. 2753 // have server data for the field types.
2710 TEST_F(AutofillManagerTest, FormSubmittedServerTypes) { 2754 TEST_F(AutofillManagerTest, FormSubmittedServerTypes) {
2711 // Set up our form data. 2755 // Set up our form data.
2712 FormData form; 2756 FormData form;
2713 CreateTestAddressFormData(&form); 2757 CreateTestAddressFormData(&form);
2714 2758
2715 // Simulate having seen this form on page load. 2759 // Simulate having seen this form on page load.
2716 // |form_structure| will be owned by |autofill_manager_|. 2760 // |form_structure| will be owned by |autofill_manager_|.
2717 TestFormStructure* form_structure = new TestFormStructure(form); 2761 TestFormStructure* form_structure = new TestFormStructure(form);
(...skipping 562 matching lines...) Expand 10 before | Expand all | Expand 10 after
3280 CreateTestAddressFormData(&form); 3324 CreateTestAddressFormData(&form);
3281 std::vector<FormData> forms(1, form); 3325 std::vector<FormData> forms(1, form);
3282 FormsSeen(forms); 3326 FormsSeen(forms);
3283 const FormFieldData& field = form.fields[0]; 3327 const FormFieldData& field = form.fields[0];
3284 GetAutofillSuggestions(form, field); // should call the delegate's OnQuery() 3328 GetAutofillSuggestions(form, field); // should call the delegate's OnQuery()
3285 3329
3286 autofill_manager_->SetExternalDelegate(NULL); 3330 autofill_manager_->SetExternalDelegate(NULL);
3287 } 3331 }
3288 3332
3289 } // namespace autofill 3333 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698