Autocomplete suggestions for bookmark TitleMatch's does not order matching bo...

chrome/browser/autocomplete/history_contents_provider_unittest.cc

 // Copyright (c) 2006-2008 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/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "chrome/browser/autocomplete/autocomplete.h"
 #include "chrome/browser/autocomplete/history_contents_provider.h"
 #include "chrome/browser/history/history.h"
@@ skipping 83 matching lines @@
   scoped_refptr<HistoryContentsProvider> provider_;
 };
 
 TEST_F(HistoryContentsProviderTest, Body) {
   AutocompleteInput input(L"FOO", std::wstring(), true, false, false);
   RunQuery(input, false);
 
   // The results should be the first two pages, in decreasing order.
   const ACMatches& m = matches();
   ASSERT_EQ(2U, m.size());
-  EXPECT_EQ(test_entries[1].url, m[0].destination_url.spec());
-  EXPECT_STREQ(test_entries[1].title, m[0].description.c_str());
-  EXPECT_EQ(test_entries[0].url, m[1].destination_url.spec());
-  EXPECT_STREQ(test_entries[0].title, m[1].description.c_str());
+  EXPECT_EQ(test_entries[0].url, m[0].destination_url.spec());
+  EXPECT_STREQ(test_entries[0].title, m[0].description.c_str());
+  EXPECT_EQ(test_entries[1].url, m[1].destination_url.spec());
+  EXPECT_STREQ(test_entries[1].title, m[1].description.c_str());
 }
 
 TEST_F(HistoryContentsProviderTest, Title) {
   AutocompleteInput input(L"PAGEONE", std::wstring(), true, false, false);
   RunQuery(input, false);
 
   // The results should be the first two pages.
   const ACMatches& m = matches();
   ASSERT_EQ(2U, m.size());
-  EXPECT_EQ(test_entries[1].url, m[0].destination_url.spec());
-  EXPECT_STREQ(test_entries[1].title, m[0].description.c_str());
-  EXPECT_EQ(test_entries[0].url, m[1].destination_url.spec());
-  EXPECT_STREQ(test_entries[0].title, m[1].description.c_str());
+  EXPECT_EQ(test_entries[0].url, m[0].destination_url.spec());
+  EXPECT_STREQ(test_entries[0].title, m[0].description.c_str());
+  EXPECT_EQ(test_entries[1].url, m[1].destination_url.spec());
+  EXPECT_STREQ(test_entries[1].title, m[1].description.c_str());
 }
 
 // The "minimal changes" flag should mean that we don't re-query the DB.
 TEST_F(HistoryContentsProviderTest, MinimalChanges) {
   // A minimal changes request when there have been no real queries should
   // give us no results.
   AutocompleteInput sync_input(L"PAGEONE", std::wstring(), true, false, true);
   RunQuery(sync_input, true);
   const ACMatches& m1 = matches();
   EXPECT_EQ(0U, m1.size());
@@ skipping 44 matching lines @@
   ASSERT_EQ(2U, m3.size());
   if (bookmark_url == m3[0].destination_url) {
     EXPECT_EQ("http://www.google.com/3", m3[1].destination_url.spec());
   } else {
     EXPECT_EQ(bookmark_url, m3[1].destination_url);
     EXPECT_EQ("http://www.google.com/3", m3[0].destination_url.spec());
   }
 }
 
 }  // namespace