Fail autocheckout if there is no forms in the page.

components/autofill/browser/autofill_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/browser/autofill_manager.h"
 
 #include <stddef.h>
 
 #include <limits>
 #include <map>
@@ skipping 448 matching lines @@
     host->Send(
         new AutofillMsg_WhitelistedForAutocheckout(host->GetRoutingID()));
     // If whitelisted URL, fetch all the forms.
     if (has_more_forms) {
       host->Send(new AutofillMsg_GetAllForms(host->GetRoutingID()));
       return;
     }
   }
 
   autocheckout_manager_.OnFormsSeen();
+  if (forms.empty()) {
+    // Call OnLoadedPageMetaData with no page metadata immediately if there is
+    // no form in the page. This give |autocheckout_manager| a chance to
+    // terminate Autocheckout and send Autocheckout status.
+    autocheckout_manager_.OnLoadedPageMetaData(
+        scoped_ptr<autofill::AutocheckoutPageMetaData>(NULL));
+  }
+
   bool enabled = IsAutofillEnabled();
   if (!has_logged_autofill_enabled_) {
     metric_logger_->LogIsAutofillEnabledAtPageLoad(enabled);
     has_logged_autofill_enabled_ = true;
   }
 
   if (!enabled)
     return;
 
   forms_loaded_timestamp_ = timestamp;
   ParseForms(forms);

[Ilya Sherman, 2013/04/10 20:52:18]

What if |forms| is nonempty, but only contains forms that we end up ignoring?

[benquan, 2013/04/10 22:08:15]

Good catch. I moved the logic into |ParseForms|.

 }
 
 void AutofillManager::OnTextFieldDidChange(const FormData& form,
                                            const FormFieldData& field,
                                            const TimeTicks& timestamp) {
   FormStructure* form_structure = NULL;
   AutofillField* autofill_field = NULL;
   if (!GetCachedFormAndField(form, field, &form_structure, &autofill_field))
     return;
 
@@ skipping 847 matching lines @@
   *profile_guid = IDToGUID(profile_id);
 }
 
 void AutofillManager::UpdateInitialInteractionTimestamp(
     const TimeTicks& interaction_timestamp) {
   if (initial_interaction_timestamp_.is_null() ||
       interaction_timestamp < initial_interaction_timestamp_) {
     initial_interaction_timestamp_ = interaction_timestamp;
   }
 }