Skip Unacceptable Items in new Autofill UI when Navigating with Keys.

chrome/browser/autofill/autofill_popup_unittest.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 "base/memory/scoped_ptr.h"
 #include "chrome/browser/autofill/autofill_popup_view.h"
 #include "chrome/browser/autofill/test_autofill_external_delegate.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAutofillClient.h"
@@ skipping 135 matching lines @@
       autofill_popup_view_->autofill_values().size() - 1);
   EXPECT_FALSE(autofill_popup_view_->RemoveSelectedLine());
   EXPECT_LE(0, autofill_popup_view_->selected_line());
 
   // Remove the first (and only) entry. The popup should then be hidden since
   // there are no Autofill entries left.
   EXPECT_CALL(*autofill_popup_view_, HideInternal());
   autofill_popup_view_->SetSelectedLine(0);
   EXPECT_TRUE(autofill_popup_view_->RemoveSelectedLine());
 }
+
+TEST_F(AutofillPopupViewUnitTest, SkipSeparator) {
+  // Set up the popup.
+  std::vector<string16> autofill_values(3, string16());
+  std::vector<int> autofill_ids;
+  autofill_ids.push_back(1);
+  autofill_ids.push_back(WebAutofillClient::MenuItemIDSeparator);
+  autofill_ids.push_back(WebAutofillClient::MenuItemIDAutofillOptions);
+  autofill_popup_view_->Show(autofill_values, autofill_values, autofill_values,
+                             autofill_ids);
+
+  // To remove warnings.
+  EXPECT_CALL(*autofill_popup_view_, InvalidateRow(_)).Times(AtLeast(0));
+
+  autofill_popup_view_->SetSelectedLine(0);
+
+  // Make sure next skips the unselectable separator.
+  autofill_popup_view_->SelectNextLine();
+  EXPECT_EQ(2, autofill_popup_view_->selected_line());
+
+  // Make sure previous skips the unselectable separator.
+  autofill_popup_view_->SelectPreviousLine();
+  EXPECT_EQ(0, autofill_popup_view_->selected_line());
+}