diff options
author | Kristian Monsen <kristianm@google.com> | 2011-06-02 17:51:19 +0100 |
---|---|---|
committer | Kristian Monsen <kristianm@google.com> | 2011-06-14 20:31:43 -0700 |
commit | 8d015ec3759624dc88b69752de43dc2007329518 (patch) | |
tree | 481ab321b45115299d04f3e4adf4f6e7faee2afe /chrome | |
parent | 137f702fd336e45a37a8feccbaf74756ff5bfa03 (diff) | |
download | external_chromium-8d015ec3759624dc88b69752de43dc2007329518.zip external_chromium-8d015ec3759624dc88b69752de43dc2007329518.tar.gz external_chromium-8d015ec3759624dc88b69752de43dc2007329518.tar.bz2 |
Merge Chromium at r11.0.672.0: Fix conflicts in autofill_manager.*
New header added close to #ifndef ANDROID, we also don't want it:
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/autofill/autofill_manager.cc?view=diff&pathrev=74924&r1=70913&r2=70914
Also exluding chrome/browser/ui/browser_list.h from same CL.
Major refactoring:
http://codereview.chromium.org/5958021
Now inherits from TabContentsObserver (but not in Android):
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/autofill/autofill_manager.h?view=diff&pathrev=74924&r1=72915&r2=72916
upload_form_structure_ was removed close to local edit:
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/autofill/autofill_manager.h?r1=71282&r2=72004&pathrev=74924
Change-Id: Ifd158f1aca8b5de661ab6279d5f8ce3fce742806
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/autofill/autofill_manager.cc | 76 | ||||
-rw-r--r-- | chrome/browser/autofill/autofill_manager.h | 19 |
2 files changed, 11 insertions, 84 deletions
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc index 93215eb..d2a1d62 100644 --- a/chrome/browser/autofill/autofill_manager.cc +++ b/chrome/browser/autofill/autofill_manager.cc @@ -13,11 +13,8 @@ #include "base/command_line.h" #include "base/string16.h" #include "base/utf_string_conversions.h" -<<<<<<< HEAD #ifndef ANDROID -======= #include "chrome/browser/autocomplete_history_manager.h" ->>>>>>> chromium.org at r11.0.672.0 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h" #endif #include "chrome/browser/autofill/autofill_dialog.h" @@ -31,7 +28,9 @@ #include "chrome/browser/renderer_host/render_view_host.h" #endif #include "chrome/browser/tab_contents/tab_contents.h" +#ifndef ANDROID #include "chrome/browser/ui/browser_list.h" +#endif #include "chrome/common/autofill_messages.h" #include "chrome/common/chrome_switches.h" #include "chrome/common/guid.h" @@ -325,42 +324,20 @@ void AutoFillManager::OnFormsSeen(const std::vector<FormData>& forms) { ParseForms(forms); } -<<<<<<< HEAD -bool AutoFillManager::GetAutoFillSuggestions(const FormData& form, - const FormField& field) { -#ifdef ANDROID - AutoFillHost* host = NULL; -#else - RenderViewHost* host = NULL; -#endif - FormStructure* form_structure = NULL; - AutoFillField* autofill_field = NULL; - if (!GetHost(personal_data_->profiles(), - personal_data_->credit_cards(), - &host) || - !FindCachedFormAndField(form, field, &form_structure, &autofill_field)) - return false; - - DCHECK(host); - DCHECK(form_structure); - DCHECK(autofill_field); - - // Don't send suggestions for forms that aren't auto-fillable. - if (!form_structure->IsAutoFillable(false)) - return false; - -======= void AutoFillManager::OnQueryFormFieldAutoFill( int query_id, const webkit_glue::FormData& form, const webkit_glue::FormField& field) { ->>>>>>> chromium.org at r11.0.672.0 std::vector<string16> values; std::vector<string16> labels; std::vector<string16> icons; std::vector<int> unique_ids; +#ifdef ANDROID + AutoFillHost* host = NULL; +#else RenderViewHost* host = NULL; +#endif FormStructure* form_structure = NULL; AutoFillField* autofill_field = NULL; if (GetHost( @@ -378,26 +355,7 @@ void AutoFillManager::OnQueryFormFieldAutoFill( form_structure, field, type, &values, &labels, &icons, &unique_ids); } -<<<<<<< HEAD #ifndef ANDROID - // Don't provide AutoFill suggestions when AutoFill is disabled, and don't - // provide credit card suggestions for non-HTTPS pages. However, provide a - // warning to the user in these cases. - int warning = 0; - if (!form_structure->IsAutoFillable(true)) - warning = IDS_AUTOFILL_WARNING_FORM_DISABLED; - else if (is_filling_credit_card && !FormIsHTTPS(form_structure)) - warning = IDS_AUTOFILL_WARNING_INSECURE_CONNECTION; - if (warning) { - values.assign(1, l10n_util::GetStringUTF16(warning)); - labels.assign(1, string16()); - icons.assign(1, string16()); - unique_ids.assign(1, -1); - host->AutoFillSuggestionsReturned(values, labels, icons, unique_ids); - return true; - } -#endif -======= DCHECK_EQ(values.size(), labels.size()); DCHECK_EQ(values.size(), icons.size()); DCHECK_EQ(values.size(), unique_ids.size()); @@ -429,10 +387,10 @@ void AutoFillManager::OnQueryFormFieldAutoFill( labels.assign(labels.size(), string16()); icons.assign(icons.size(), string16()); } ->>>>>>> chromium.org at r11.0.672.0 RemoveDuplicateSuggestions(&values, &labels, &icons, &unique_ids); } +#endif } } @@ -585,20 +543,13 @@ void AutoFillManager::OnShowAutoFillDialog() { tab_contents_->profile()->GetOriginalProfile()); } -<<<<<<< HEAD -void AutoFillManager::ShowAutoFillDialog() { -#ifndef ANDROID - ::ShowAutoFillDialog(tab_contents_->GetContentNativeView(), - personal_data_, - tab_contents_->profile()->GetOriginalProfile()); -#endif -======= void AutoFillManager::OnDidFillAutoFillFormData() { +#ifndef ANDROID NotificationService::current()->Notify( NotificationType::AUTOFILL_DID_FILL_FORM_DATA, Source<RenderViewHost>(tab_contents_->render_view_host()), NotificationService::NoDetails()); ->>>>>>> chromium.org at r11.0.672.0 +#endif } void AutoFillManager::OnDidShowAutoFillSuggestions() { @@ -748,16 +699,11 @@ void AutoFillManager::ImportFormData(const FormStructure& submitted_form) { if (!personal_data_->ImportFormData(import, &imported_credit_card)) return; -<<<<<<< HEAD #ifndef ANDROID - // Show an infobar to offer to save the credit card info. - if (tab_contents_) { -======= // If credit card information was submitted, show an infobar to offer to save // it. if (imported_credit_card && tab_contents_) { imported_credit_card_.reset(imported_credit_card); ->>>>>>> chromium.org at r11.0.672.0 tab_contents_->AddInfoBar(new AutoFillCCInfoBarDelegate(tab_contents_, this)); } @@ -838,12 +784,8 @@ bool AutoFillManager::GetHost(const std::vector<AutoFillProfile*>& profiles, *host = tab_contents_->autofill_host(); #else *host = tab_contents_->render_view_host(); -<<<<<<< HEAD #endif - if (!(*host)) -======= if (!*host) ->>>>>>> chromium.org at r11.0.672.0 return false; return true; diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h index 933927b..30cc6a5 100644 --- a/chrome/browser/autofill/autofill_manager.h +++ b/chrome/browser/autofill/autofill_manager.h @@ -16,13 +16,9 @@ #include "chrome/browser/autofill/autofill_dialog.h" #include "chrome/browser/autofill/autofill_download.h" #include "chrome/browser/autofill/personal_data_manager.h" -<<<<<<< HEAD #ifndef ANDROID -#include "chrome/browser/renderer_host/render_view_host_delegate.h" -#endif -======= #include "chrome/browser/tab_contents/tab_contents_observer.h" ->>>>>>> chromium.org at r11.0.672.0 +#endif #ifndef ANDROID class AutoFillCCInfoBarDelegate; @@ -45,16 +41,11 @@ class FormField; // Manages saving and restoring the user's personal information entered into web // forms. -<<<<<<< HEAD class AutoFillManager : #ifndef ANDROID - public RenderViewHostDelegate::AutoFill, + public TabContentsObserver, #endif - public AutoFillDownloadManager::Observer { -======= -class AutoFillManager : public TabContentsObserver, public AutoFillDownloadManager::Observer { ->>>>>>> chromium.org at r11.0.672.0 public: explicit AutoFillManager(TabContents* tab_contents); virtual ~AutoFillManager(); @@ -238,16 +229,10 @@ class AutoFillManager : public TabContentsObserver, // Our copy of the form data. ScopedVector<FormStructure> form_structures_; -<<<<<<< HEAD - // The form data the user has submitted. - scoped_ptr<FormStructure> upload_form_structure_; - #ifdef ANDROID // To minimize merge conflicts, we keep this pointer around, but never use it. void* cc_infobar_; #else -======= ->>>>>>> chromium.org at r11.0.672.0 // The InfoBar that asks for permission to store credit card information. // Deletes itself when closed. AutoFillCCInfoBarDelegate* cc_infobar_; |