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

Unified Diff: chrome/browser/ui/views/download/download_item_view_md.h

Issue 1832963002: MD - add ripples to DL shelf items (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix another TODO Created 4 years, 8 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
« no previous file with comments | « no previous file | chrome/browser/ui/views/download/download_item_view_md.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/download/download_item_view_md.h
diff --git a/chrome/browser/ui/views/download/download_item_view_md.h b/chrome/browser/ui/views/download/download_item_view_md.h
index e4357eaa57f2d222cfd6c26f073b87cfd709380c..26836d7c370ace289115f86445fca7cf3367a8c1 100644
--- a/chrome/browser/ui/views/download/download_item_view_md.h
+++ b/chrome/browser/ui/views/download/download_item_view_md.h
@@ -31,9 +31,10 @@
#include "content/public/browser/download_manager.h"
#include "ui/gfx/animation/animation_delegate.h"
#include "ui/gfx/font_list.h"
+#include "ui/views/animation/button_ink_drop_delegate.h"
+#include "ui/views/animation/ink_drop_host_view.h"
#include "ui/views/context_menu_controller.h"
#include "ui/views/controls/button/button.h"
-#include "ui/views/view.h"
class BarControlButton;
class DownloadShelfView;
@@ -61,8 +62,8 @@ class LabelButton;
// The DownloadItemView in MD style. This is copied from DownloadItemView,
// which it should eventually replace.
-class DownloadItemViewMd : public views::ButtonListener,
- public views::View,
+class DownloadItemViewMd : public views::InkDropHostView,
+ public views::ButtonListener,
public views::ContextMenuController,
public content::DownloadItem::Observer,
public gfx::AnimationDelegate {
@@ -103,6 +104,11 @@ class DownloadItemViewMd : public views::ButtonListener,
void GetAccessibleState(ui::AXViewState* state) override;
void OnThemeChanged() override;
+ // Overridden from view::InkDropHostView:
+ void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
+ scoped_ptr<views::InkDropAnimation> CreateInkDropAnimation() const override;
+ scoped_ptr<views::InkDropHover> CreateInkDropHover() const override;
+
// Overridden from ui::EventHandler:
void OnGestureEvent(ui::GestureEvent* event) override;
@@ -220,10 +226,10 @@ class DownloadItemViewMd : public views::ButtonListener,
void ProgressTimerFired();
// Returns the base text color.
- SkColor GetTextColor();
+ SkColor GetTextColor() const;
// Returns a slightly dimmed version of the base text color.
- SkColor GetDimmedTextColor();
+ SkColor GetDimmedTextColor() const;
// The download shelf that owns us.
DownloadShelfView* shelf_;
@@ -269,13 +275,11 @@ class DownloadItemViewMd : public views::ButtonListener,
// A model class to control the status text we display.
DownloadItemModel model_;
- // Hover animations for our body and drop buttons.
- scoped_ptr<gfx::SlideAnimation> body_hover_animation_;
- scoped_ptr<gfx::SlideAnimation> drop_hover_animation_;
-
// Animation for download complete.
scoped_ptr<gfx::SlideAnimation> complete_animation_;
+ views::ButtonInkDropDelegate ink_drop_delegate_;
+
// Progress animation
base::RepeatingTimer progress_timer_;
« no previous file with comments | « no previous file | chrome/browser/ui/views/download/download_item_view_md.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698