Strip special characters in extension omnibox suggestions.

chrome/browser/autocomplete/extension_app_provider_unittest.cc

 // Copyright (c) 2011 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/basictypes.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "chrome/browser/autocomplete/extension_app_provider.h"
 #include "chrome/browser/history/history.h"
@@ skipping 87 matching lines @@
                                            GURL("http://fdsa/") }},
     // The string 'co' appears in one extension app.
     {ASCIIToUTF16("co"),              1, { GURL("http://asdf/") }},
     // Try with URL matching.
     {ASCIIToUTF16("http://asdf/"),    1, { GURL("http://asdf/") }},
     {ASCIIToUTF16("http://fdsa/"),    1, { GURL("http://fdsa/") }},
   };
 
   RunTest(edit_cases, ARRAYSIZE_UNSAFE(edit_cases));
 }
+
+TEST_F(ExtensionAppProviderTest, CreateMatchSanitize) {
+  struct TestData {
+    const char* name;
+    const char* match_contents;
+  } cases[] = {
+    { "Test", "Test" },
+    { "Test \n Test", "Test  Test" },
+    { "Test\r\t\nTest", "TestTest" },
+  };
+
+  AutocompleteInput input(ASCIIToUTF16("Test"), ASCIIToUTF16(""),

[Peter Kasting, 2011/10/25 20:37:07]

Nit: ASCIIToUTF16("") -> string16()

[Alexei Svitkine (slow), 2011/10/25 21:37:16]

Done.

+                          true, true, true, AutocompleteInput::BEST_MATCH);
+  string16 url(ASCIIToUTF16("http://example.com"));
+  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) {
+    AutocompleteMatch match =
+        app_provider_->CreateAutocompleteMatch(input,
+                                               ASCIIToUTF16(cases[i].name),
+                                               url, 0, string16::npos);
+    EXPECT_EQ(ASCIIToUTF16(cases[i].match_contents), match.contents);
+  }
+}
+