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

Unified Diff: ui/app_list/search_result.h

Issue 796293002: App list voice searches now prioritize exact-match web results. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed MixerTest and add MixerTest.VoiceQuery. Created 6 years 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 side-by-side diff with in-line comments
Download patch
Index: ui/app_list/search_result.h
diff --git a/ui/app_list/search_result.h b/ui/app_list/search_result.h
index dd955a6c831ca5148c867af9b25f01b0441ce948..5abefd71806d5baff45a2c3cac2c0214f9d34da5 100644
--- a/ui/app_list/search_result.h
+++ b/ui/app_list/search_result.h
@@ -104,6 +104,11 @@ class APP_LIST_EXPORT SearchResult {
}
void SetActions(const Actions& sets);
+ // Whether the result can be automatically selected by a voice query.
+ // (Non-voice results can still appear in the results list to be manually
+ // selected.)
+ bool voice_result() const { return voice_result_; }
calamity 2014/12/16 01:50:03 So do we want to change this as discussed yesterda
Matt Giuca 2014/12/16 02:04:09 As discussed, I think after the changes that the p
+
bool is_installing() const { return is_installing_; }
void SetIsInstalling(bool is_installing);
@@ -140,6 +145,7 @@ class APP_LIST_EXPORT SearchResult {
void set_display_type(DisplayType display_type) {
display_type_ = display_type;
}
+ void set_voice_result(bool voice_result) { voice_result_ = voice_result; }
private:
gfx::ImageSkia icon_;
@@ -155,6 +161,7 @@ class APP_LIST_EXPORT SearchResult {
DisplayType display_type_;
Actions actions_;
+ bool voice_result_;
bool is_installing_;
int percent_downloaded_;

Powered by Google App Engine
This is Rietveld 408576698