Fix a bug that was causing the AutoFill infobar to not show. Initially the AutoFillEnabled preference is false, so we were bailing out before giving the chance to enable AutoFill. This changes to guard to |personal_data_|, which is necessary for DeterminePossibleFieldTypes.

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/667018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40567 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 5 deletions

diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index 9263e08..ecc72d9 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -60,6 +60,12 @@ class AutoFillManager : public RenderViewHostDelegate::AutoFill,
   // PersonalDataManager::Observer implementation:
   virtual void OnPersonalDataLoaded();
 
+  // Called by the AutoFillInfoBarDelegate when the user accepts the infobar.
+  virtual void OnInfoBarAccepted();
+
+  // Resets the stored form data.
+  virtual void Reset();
+
   // Uses heuristics and existing personal data to determine the possible field
   // types.
   void DeterminePossibleFieldTypes(FormStructure* form_structure);
@@ -67,15 +73,10 @@ class AutoFillManager : public RenderViewHostDelegate::AutoFill,
   // Handles the form data submitted by the user.
   void HandleSubmit();
 
-  // Called by the AutoFillInfoBarDelegate when the user accepts the infobar.
-  void OnInfoBarAccepted();
 
   // Uploads the form data to the autofill server.
   void UploadFormData();
 
-  // Resets the stored form data.
-  void Reset();
-
   // Returns the value of the AutoFillEnabled pref.
   bool IsAutoFillEnabled();