Fail autocheckout if there is no forms in the page.

chrome/browser/ui/autofill/tab_autofill_manager_delegate.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 "chrome/browser/ui/autofill/tab_autofill_manager_delegate.h"
 
 #include "base/logging.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/autofill/autocheckout_whitelist_manager_factory.h"
 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h"
@@ skipping 80 matching lines @@
   sync_state_changed_callback_ = callback;
 
   // Invariant: Either sync_state_changed_callback_.is_null() is true
   // and this object is not subscribed as a
   // ProfileSyncServiceObserver, or
   // sync_state_changed_callback_.is_null() is false and this object
   // is subscribed as a ProfileSyncServiceObserver.
 }
 
 void TabAutofillManagerDelegate::OnAutocheckoutError() {
-  dialog_controller_->OnAutocheckoutError();
+  if (dialog_controller_)

[ahutter, 2013/04/10 22:38:06]

I still don't think this change should be here.

[benquan, 2013/04/11 02:17:56]

replaced the null check with a comment

+    dialog_controller_->OnAutocheckoutError();
 }
 
 void TabAutofillManagerDelegate::ShowAutofillSettings() {
 #if defined(OS_ANDROID)
   NOTIMPLEMENTED();
 #else
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents_);
   if (browser)
     chrome::ShowSettingsSubPage(browser, chrome::kAutofillSubPage);
 #endif  // #if defined(OS_ANDROID)
@@ skipping 107 matching lines @@
       (dialog_controller_->dialog_type() ==
           autofill::DIALOG_TYPE_REQUEST_AUTOCOMPLETE ||
        !dialog_controller_->AutocheckoutIsRunning())) {
     HideRequestAutocompleteDialog();
   }
 
   HideAutocheckoutBubble();
 }
 
 }  // namespace autofill