GTK Keyboard Support for New Autofill

chrome/browser/autofill/autofill_popup_view_browsertest.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/autofill/autofill_popup_view.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/autofill/test_autofill_external_delegate.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/url_constants.h"
@@ skipping 17 matching lines @@
  public:
   MockAutofillExternalDelegate() : TestAutofillExternalDelegate(NULL, NULL) {}
   ~MockAutofillExternalDelegate() {}
 
   virtual void SelectAutofillSuggestionAtIndex(int unique_id, int list_index)
       OVERRIDE {}
 };
 
 class TestAutofillPopupView : public AutofillPopupView {
  public:
-  explicit TestAutofillPopupView(
+  TestAutofillPopupView(
       content::WebContents* web_contents,
       AutofillExternalDelegate* autofill_external_delegate)
       : AutofillPopupView(web_contents, autofill_external_delegate) {}
   virtual ~TestAutofillPopupView() {}
 
   MOCK_METHOD0(Hide, void());
 
-  MOCK_METHOD1(InvalidateRow, void(size_t));
-
-  void SetSelectedLine(size_t selected_line) {
-    AutofillPopupView::SetSelectedLine(selected_line);
-  }
-
  protected:
   virtual void ShowInternal() OVERRIDE {}
 
   virtual void HideInternal() OVERRIDE {}
+
+  virtual void InvalidateRow(size_t row) OVERRIDE {}
 };
 
 }  // namespace
 
 class AutofillPopupViewBrowserTest : public InProcessBrowserTest {
  public:
   AutofillPopupViewBrowserTest() {}
   virtual ~AutofillPopupViewBrowserTest() {}
 
   virtual void SetUpOnMainThread() OVERRIDE {
@@ skipping 36 matching lines @@
   browser()->OpenURL(content::OpenURLParams(
       GURL(chrome::kAboutBlankURL), content::Referrer(),
       CURRENT_TAB, content::PAGE_TRANSITION_TYPED, false));
   browser()->OpenURL(content::OpenURLParams(
       GURL(chrome::kChromeUIAboutURL), content::Referrer(),
       CURRENT_TAB, content::PAGE_TRANSITION_TYPED, false));
   observer.Wait();
 
   // The mock verifies that the call was made.
 }
-
-IN_PROC_BROWSER_TEST_F(AutofillPopupViewBrowserTest,
-                       SetSelectedAutofillLineAndCallInvalidate) {
-  std::vector<string16> autofill_values;
-  autofill_values.push_back(string16());
-  std::vector<int> autofill_ids;
-  autofill_ids.push_back(0);
-  autofill_popup_view_->Show(
-      autofill_values, autofill_values, autofill_values, autofill_ids, 0);
-
-  // Make sure that when a new line is selected, it is invalidated so it can
-  // be updated to show it is selected.
-  int selected_line = 0;
-  EXPECT_CALL(*autofill_popup_view_, InvalidateRow(selected_line));
-  autofill_popup_view_->SetSelectedLine(selected_line);
-
-  // Ensure that the row isn't invalidated if it didn't change.
-  EXPECT_CALL(*autofill_popup_view_, InvalidateRow(selected_line)).Times(0);
-  autofill_popup_view_->SetSelectedLine(selected_line);
-
-  // Change back to no selection.
-  EXPECT_CALL(*autofill_popup_view_, InvalidateRow(selected_line));
-  autofill_popup_view_->SetSelectedLine(-1);
-}