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

Side by Side Diff: chrome/browser/resources/omnibox/omnibox.js

Issue 415053002: Remove AutocompleteMatch::is_starred (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 /** 5 /**
6 * Javascript for omnibox.html, served from chrome://omnibox/ 6 * Javascript for omnibox.html, served from chrome://omnibox/
7 * This is used to debug omnibox ranking. The user enters some text 7 * This is used to debug omnibox ranking. The user enters some text
8 * into a box, submits it, and then sees lots of debug information 8 * into a box, submits it, and then sees lots of debug information
9 * from the autocompleter that shows what omnibox would do with that 9 * from the autocompleter that shows what omnibox would do with that
10 * input. 10 * input.
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 'The type of the result.'), 119 'The type of the result.'),
120 new PresentationInfoRecord('Relevance', '', 'relevance', true, 120 new PresentationInfoRecord('Relevance', '', 'relevance', true,
121 'The result score. Higher is more relevant.'), 121 'The result score. Higher is more relevant.'),
122 new PresentationInfoRecord('Contents', '', 'contents', true, 122 new PresentationInfoRecord('Contents', '', 'contents', true,
123 'The text that is presented identifying the result.'), 123 'The text that is presented identifying the result.'),
124 new PresentationInfoRecord( 124 new PresentationInfoRecord(
125 'Can Be Default', '', 'allowed_to_be_default_match', false, 125 'Can Be Default', '', 'allowed_to_be_default_match', false,
126 'A green checkmark indicates that the result can be the default ' + 126 'A green checkmark indicates that the result can be the default ' +
127 'match (i.e., can be the match that pressing enter in the omnibox ' + 127 'match (i.e., can be the match that pressing enter in the omnibox ' +
128 'navigates to).'), 128 'navigates to).'),
129 new PresentationInfoRecord('Starred', '', 'starred', false,
130 'A green checkmark indicates that the result has been bookmarked.'),
Peter Kasting 2014/07/24 18:51:34 This is still useful information to know. I think
hashimoto 2014/07/25 08:36:12 Added code to fill |starred| in OmniboxUIHandler.
131 new PresentationInfoRecord( 129 new PresentationInfoRecord(
132 'HWYT', '', 'is_history_what_you_typed_match', false, 130 'HWYT', '', 'is_history_what_you_typed_match', false,
133 'A green checkmark indicates that the result is an History What You ' + 131 'A green checkmark indicates that the result is an History What You ' +
134 'Typed Match'), 132 'Typed Match'),
135 new PresentationInfoRecord('Description', '', 'description', false, 133 new PresentationInfoRecord('Description', '', 'description', false,
136 'The page title of the result.'), 134 'The page title of the result.'),
137 new PresentationInfoRecord('URL', '', 'destination_url', true, 135 new PresentationInfoRecord('URL', '', 'destination_url', true,
138 'The URL for the result.'), 136 'The URL for the result.'),
139 new PresentationInfoRecord('Fill Into Edit', '', 'fill_into_edit', false, 137 new PresentationInfoRecord('Fill Into Edit', '', 'fill_into_edit', false,
140 'The text shown in the omnibox when the result is selected.'), 138 'The text shown in the omnibox when the result is selected.'),
(...skipping 288 matching lines...) Expand 10 before | Expand all | Expand 10 after
429 OmniboxPageImpl.prototype.handleNewAutocompleteResult = function(result) { 427 OmniboxPageImpl.prototype.handleNewAutocompleteResult = function(result) {
430 progressiveAutocompleteResults.push(result); 428 progressiveAutocompleteResults.push(result);
431 refresh(); 429 refresh();
432 }; 430 };
433 431
434 return function(handle) { 432 return function(handle) {
435 connection = new connector.Connection(handle, OmniboxPageImpl, 433 connection = new connector.Connection(handle, OmniboxPageImpl,
436 browser.OmniboxUIHandlerMojoProxy); 434 browser.OmniboxUIHandlerMojoProxy);
437 }; 435 };
438 }); 436 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698