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

Side by Side Diff: chrome/browser/ui/views/omnibox/omnibox_result_view.h

Issue 1023993003: [AiS] Added option to put a space between answer values (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed default param Created 5 years, 9 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
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_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 // Returns the offset at which the contents of the |match| should be displayed 153 // Returns the offset at which the contents of the |match| should be displayed
154 // within the text bounds. The directionality of UI and match contents is used 154 // within the text bounds. The directionality of UI and match contents is used
155 // to determine the offset relative to the correct edge. 155 // to determine the offset relative to the correct edge.
156 int GetDisplayOffset(const AutocompleteMatch& match, 156 int GetDisplayOffset(const AutocompleteMatch& match,
157 bool is_ui_rtl, 157 bool is_ui_rtl,
158 bool is_match_contents_rtl) const; 158 bool is_match_contents_rtl) const;
159 159
160 int GetAnswerLineHeight() const; 160 int GetAnswerLineHeight() const;
161 int GetContentLineHeight() const; 161 int GetContentLineHeight() const;
162 162
163 void AppendAnswerText(const SuggestionAnswer::TextField& text_field); 163 void AppendAnswerText(const SuggestionAnswer::TextField& text_field,
Peter Kasting 2015/03/23 20:19:21 While here: This function needs a comment about wh
dschuyler 2015/03/24 00:52:41 Done.
164 bool prefix_space);
164 165
165 static int default_icon_size_; 166 static int default_icon_size_;
166 167
167 // Default values cached here, may be overridden using the setters above. 168 // Default values cached here, may be overridden using the setters above.
168 int edge_item_padding_; 169 int edge_item_padding_;
169 int item_padding_; 170 int item_padding_;
170 171
171 // This row's model and model index. 172 // This row's model and model index.
172 OmniboxPopupContentsView* model_; 173 OmniboxPopupContentsView* model_;
173 size_t model_index_; 174 size_t model_index_;
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 mutable scoped_ptr<gfx::RenderText> keyword_description_rendertext_; 208 mutable scoped_ptr<gfx::RenderText> keyword_description_rendertext_;
208 209
209 mutable int separator_width_; 210 mutable int separator_width_;
210 211
211 base::WeakPtrFactory<OmniboxResultView> weak_ptr_factory_; 212 base::WeakPtrFactory<OmniboxResultView> weak_ptr_factory_;
212 213
213 DISALLOW_COPY_AND_ASSIGN(OmniboxResultView); 214 DISALLOW_COPY_AND_ASSIGN(OmniboxResultView);
214 }; 215 };
215 216
216 #endif // CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_ 217 #endif // CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_RESULT_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698