Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1191)

Side by Side Diff: chrome/browser/autocomplete/autocomplete.h

Issue 10274023: Omnibox SearchProvider Experiment Client Implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes and cleanup. Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_ 5 #ifndef CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_
6 #define CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_ 6 #define CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_
7 #pragma once 7 #pragma once
8 8
9 #include <map> 9 #include <map>
10 #include <string> 10 #include <string>
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 // HistoryContents (any match in body of nonstarred page) | 500++ 166 // HistoryContents (any match in body of nonstarred page) | 500++
167 // 167 //
168 // (A search keyword is a keyword with a replacement string; a bookmark keyword 168 // (A search keyword is a keyword with a replacement string; a bookmark keyword
169 // is a keyword with no replacement string, that is, a shortcut for a URL.) 169 // is a keyword with no replacement string, that is, a shortcut for a URL.)
170 // 170 //
171 // There are two possible providers for search suggestions. If the user has 171 // There are two possible providers for search suggestions. If the user has
172 // typed a keyword, then the primary provider is the keyword provider and the 172 // typed a keyword, then the primary provider is the keyword provider and the
173 // secondary provider is the default provider. If the user has not typed a 173 // secondary provider is the default provider. If the user has not typed a
174 // keyword, then the primary provider corresponds to the default provider. 174 // keyword, then the primary provider corresponds to the default provider.
175 // 175 //
176 // Search providers may supply relevance values along with their results to be
177 // used in place of client-side calculated values. See http://crbug.com/125871
Peter Kasting 2012/06/05 00:47:51 Nit: I'd just omit the bug reference
msw 2012/06/05 01:49:41 Done.
178 //
176 // The value column gives the ranking returned from the various providers. 179 // The value column gives the ranking returned from the various providers.
177 // ++: a series of matches with relevance from n up to (n + max_matches). 180 // ++: a series of matches with relevance from n up to (n + max_matches).
178 // --: relevance score falls off over time (discounted 50 points @ 15 minutes, 181 // --: relevance score falls off over time (discounted 50 points @ 15 minutes,
179 // 450 points @ two weeks) 182 // 450 points @ two weeks)
180 // **: relevance score falls off over two days (discounted 99 points after two 183 // **: relevance score falls off over two days (discounted 99 points after two
181 // days). 184 // days).
182 // *~: Partial matches get a score on a sliding scale from about 575-1125 based 185 // *~: Partial matches get a score on a sliding scale from about 575-1125 based
183 // on how many times the URL for the Extension App has been typed and how 186 // on how many times the URL for the Extension App has been typed and how
184 // many of the letters match. 187 // many of the letters match.
185 188
(...skipping 644 matching lines...) Expand 10 before | Expand all | Expand 10 after
830 size_t inline_autocompleted_length; 833 size_t inline_autocompleted_length;
831 // Result set. 834 // Result set.
832 const AutocompleteResult& result; 835 const AutocompleteResult& result;
833 // Diagnostic information from providers. See 836 // Diagnostic information from providers. See
834 // AutocompleteController::AddProvidersInfo() and 837 // AutocompleteController::AddProvidersInfo() and
835 // AutocompleteProvider::AddProviderInfo() above. 838 // AutocompleteProvider::AddProviderInfo() above.
836 ProvidersInfo providers_info; 839 ProvidersInfo providers_info;
837 }; 840 };
838 841
839 #endif // CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_ 842 #endif // CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/autocomplete/search_provider.h » ('j') | chrome/browser/autocomplete/search_provider.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698