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

Unified Diff: ui/app_list/views/search_result_tile_item_view.h

Issue 2949733002: Show Play Store rating and price in app list (Closed)
Patch Set: Rebase Created 3 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 side-by-side diff with in-line comments
Download patch
Index: ui/app_list/views/search_result_tile_item_view.h
diff --git a/ui/app_list/views/search_result_tile_item_view.h b/ui/app_list/views/search_result_tile_item_view.h
index 3f5ff19fe4083c32a2f2186e685041435dc2b60c..6d0525a8bb330f7039541e4dde4d3fba7510d456 100644
--- a/ui/app_list/views/search_result_tile_item_view.h
+++ b/ui/app_list/views/search_result_tile_item_view.h
@@ -40,6 +40,8 @@ class APP_LIST_EXPORT SearchResultTileItemView
// Overridden from SearchResultObserver:
void OnIconChanged() override;
void OnBadgeIconChanged() override;
+ void OnRatingChanged() override;
+ void OnPriceChanged() override;
void OnResultDestroying() override;
// views::ContextMenuController overrides:
@@ -48,6 +50,13 @@ class APP_LIST_EXPORT SearchResultTileItemView
ui::MenuSourceType source_type) override;
private:
+ // Shows rating in proper format if |rating| is not negative. Otherwise, hides
+ // the rating label.
+ void SetRating(float rating);
+
+ // Shows price if |price| is not empty. Otherwise, hides the price label.
+ void SetPrice(const base::string16& price);
+
// Overridden from views::View:
void Layout() override;
gfx::Size CalculatePreferredSize() const override;
@@ -57,6 +66,9 @@ class APP_LIST_EXPORT SearchResultTileItemView
// Owned by the model provided by the AppListViewDelegate.
SearchResult* item_;
+ views::Label* rating_; // Owned by views hierarchy.
+ views::Label* price_; // Owned by views hierarchy.
xiyuan 2017/06/22 17:07:34 nit: forward declare views::Label. Slightly prefe
weidongg 2017/06/22 17:58:30 Done.
+
AppListViewDelegate* view_delegate_;
const bool is_fullscreen_app_list_enabled_;

Powered by Google App Engine
This is Rietveld 408576698