requestAutocomplete (desktop): don't get stuck in infinite loop

chrome/browser/ui/views/autofill/autofill_dialog_views_unittest.cc

 // Copyright 2013 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/views/autofill/autofill_dialog_views.h"
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/ui/autofill/mock_autofill_dialog_view_delegate.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/views/autofill/decorated_textfield.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/frame/test_with_browser_view.h"
+#include "components/autofill/content/browser/wallet/wallet_service_url.h"
 #include "components/web_modal/test_web_contents_modal_dialog_host.h"
 #include "components/web_modal/test_web_contents_modal_dialog_manager_delegate.h"
 #include "components/web_modal/web_contents_modal_dialog_manager.h"
 #include "content/public/common/url_constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/views/controls/webview/webview.h"
 #include "ui/views/widget/widget.h"
 
 namespace autofill {
@@ skipping 92 matching lines @@
 }
 
 TEST_F(AutofillDialogViewsTest, SignInFocus) {
   SetSectionsFocusable();
 
   views::View* loading_shield = dialog()->GetLoadingShieldForTesting();
   views::View* sign_in_web_view = dialog()->GetSignInWebViewForTesting();
   views::View* notification_area = dialog()->GetNotificationAreaForTesting();
   views::View* scrollable_area = dialog()->GetScrollableAreaForTesting();
 
-  dialog()->ShowSignIn();
+  dialog()->ShowSignIn(wallet::GetAddAccountUrl());
 
   // The sign in view should be the only showing and focusable view.
   EXPECT_TRUE(sign_in_web_view->IsFocusable());
   EXPECT_FALSE(loading_shield->IsFocusable());
   EXPECT_FALSE(notification_area->IsFocusable());
   EXPECT_FALSE(scrollable_area->IsFocusable());
 
   EXPECT_CALL(*delegate(), ShouldShowSpinner()).WillRepeatedly(Return(false));
   dialog()->HideSignIn();
 
   // Hide sign in while not loading Wallet items as if the user clicked "Back".
   EXPECT_TRUE(notification_area->IsFocusable());
   EXPECT_TRUE(scrollable_area->IsFocusable());
   EXPECT_FALSE(loading_shield->IsFocusable());
   EXPECT_FALSE(sign_in_web_view->IsFocusable());
 
-  dialog()->ShowSignIn();
+  dialog()->ShowSignIn(wallet::GetAddAccountUrl());
 
   EXPECT_TRUE(sign_in_web_view->IsFocusable());
   EXPECT_FALSE(loading_shield->IsFocusable());
   EXPECT_FALSE(notification_area->IsFocusable());
   EXPECT_FALSE(scrollable_area->IsFocusable());
 
   EXPECT_CALL(*delegate(), ShouldShowSpinner()).WillRepeatedly(Return(true));
   dialog()->HideSignIn();
 
   // Hide sign in while pretending to load Wallet data.
@@ skipping 35 matching lines @@
   views::FocusManager* focus_manager = dialog()->GetWidget()->GetFocusManager();
   views::View* focused_view = focus_manager->GetFocusedView();
   ASSERT_STREQ(DecoratedTextfield::kViewClassName,
                focused_view->GetClassName());
   EXPECT_FALSE(dialog()->HandleKeyEvent(
       static_cast<views::Textfield*>(focused_view),
       ui::KeyEvent(ui::ET_KEY_PRESSED, ui::VKEY_A, ui::EF_NONE)));
 }
 
 }  // namespace autofill