Add SearchProvider's most relevant NavigationResult to |matches_|.

chrome/browser/autocomplete/search_provider_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 "chrome/browser/autocomplete/search_provider.h"
 
 #include "base/string_util.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "build/build_config.h"
@@ skipping 623 matching lines @@
 
   EXPECT_FALSE(result.match_at(0).keyword.empty());
   EXPECT_FALSE(result.match_at(1).keyword.empty());
   EXPECT_NE(result.match_at(0).keyword, result.match_at(1).keyword);
 
   EXPECT_FALSE(result.match_at(0).description.empty());
   EXPECT_FALSE(result.match_at(1).description.empty());
   EXPECT_NE(result.match_at(0).description, result.match_at(1).description);
 }
 
-// Verifies Navsuggest results don't set a TemplateURL (which instant relies
-// on).
-TEST_F(SearchProviderTest, NoTemplateURLForNavsuggest) {
+// Verifies Navsuggest results don't set a TemplateURL, which instant relies on.
+// Also verifies that just the *first* navigational result is listed as a match.
+TEST_F(SearchProviderTest, Navsuggest) {
   QueryForInput(ASCIIToUTF16("a.c"), false);
 
   // Make sure the default providers suggest service was queried.
   TestURLFetcher* fetcher = test_factory_.GetFetcherByID(
       SearchProvider::kDefaultProviderURLFetcherID);
   ASSERT_TRUE(fetcher);
 
   // Tell the SearchProvider the suggest query is done.
   fetcher->set_response_code(200);
   fetcher->SetResponseString(
-      "[\"a.c\",[\"a.com\"],[\"\"],[],"
-      "{\"google:suggesttype\":[\"NAVIGATION\"]}]");
+      "[\"a.c\",[\"a.com\", \"a.com/b\"],[\"\"],[],"
+      "{\"google:suggesttype\":[\"NAVIGATION\", \"NAVIGATION\"]}]");
   fetcher->delegate()->OnURLFetchComplete(fetcher);
   fetcher = NULL;
 
   // Run till the history results complete.
   RunTillProviderDone();
 
-  // Make sure there is a match for 'a.com' and it doesn't have a template_url.
+  // Make sure the only match is 'a.com' and it doesn't have a template_url.
   AutocompleteMatch nav_match;
   EXPECT_TRUE(FindMatchWithDestination(GURL("http://a.com"), &nav_match));
+  EXPECT_FALSE(FindMatchWithDestination(GURL("http://a.com/b"), &nav_match));
   EXPECT_TRUE(nav_match.keyword.empty());
 }