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

Unified Diff: ui/app_list/app_list_folder_item.cc

Issue 27777002: Implement app list folder management page UI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix nits. Created 7 years, 2 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/app_list_folder_item.cc
diff --git a/ui/app_list/app_list_folder_item.cc b/ui/app_list/app_list_folder_item.cc
index 0868d623738a2e4808cfc209f465723255585bb5..95e5530b9895f31a2d491cd908a86d484d341a78 100644
--- a/ui/app_list/app_list_folder_item.cc
+++ b/ui/app_list/app_list_folder_item.cc
@@ -15,7 +15,7 @@ const int kIconDimension = 48;
const size_t kNumTopApps = 4;
const int kItemIconDimension = 16;
-// Genearats the folder icon with the top 4 child item icons laid in 2x2 tile.
+// Generates the folder icon with the top 4 child item icons laid in 2x2 tile.
class FolderImageSource : public gfx::CanvasImageSource {
public:
typedef std::vector<gfx::ImageSkia> Icons;
@@ -45,11 +45,11 @@ class FolderImageSource : public gfx::CanvasImageSource {
virtual void Draw(gfx::Canvas* canvas) OVERRIDE {
// Draw folder circle.
gfx::Point center = gfx::Point(size().width() / 2 , size().height() / 2);
- const SkColor kCirclColor = SkColorSetRGB(0xE1, 0xE1, 0xE1);
+ const SkColor kCircleColor = SkColorSetRGB(0xE1, 0xE1, 0xE1);
SkPaint paint;
paint.setStyle(SkPaint::kFill_Style);
paint.setAntiAlias(true);
- paint.setColor(kCirclColor);
+ paint.setColor(kCircleColor);
canvas->DrawCircle(center, size().width() / 2, paint);
if (icons_.size() == 0)
@@ -144,8 +144,7 @@ std::string AppListFolderItem::GetSortOrder() const {
}
void AppListFolderItem::Activate(int event_flags) {
- // TODO(stevenjb/jennyz): Implement.
- VLOG(1) << "AppListFolderItem::Activate";
+ // Folder handling is implemented by the View, so do nothing.
}
// static

Powered by Google App Engine
This is Rietveld 408576698