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

Unified Diff: ash/common/shelf/app_list_button.h

Issue 2710053005: Remove non-MD code from AppListButton (Closed)
Patch Set: Rebased Created 3 years, 10 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 | ash/common/shelf/app_list_button.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/shelf/app_list_button.h
diff --git a/ash/common/shelf/app_list_button.h b/ash/common/shelf/app_list_button.h
index e6d55bf182ebe03d9b56d1c4a6e2e144c14aa9d6..d4a0fbeeb7f5160d5ea739a1496b12612526af2e 100644
--- a/ash/common/shelf/app_list_button.h
+++ b/ash/common/shelf/app_list_button.h
@@ -28,8 +28,6 @@ class ASH_EXPORT AppListButton : public views::ImageButton {
bool is_showing_app_list() const { return is_showing_app_list_; }
- bool draw_background_as_active() const { return draw_background_as_active_; }
-
// Updates background and schedules a paint.
void UpdateShelfItemBackground(SkColor color);
@@ -51,17 +49,6 @@ class ASH_EXPORT AppListButton : public views::ImageButton {
void OnGestureEvent(ui::GestureEvent* event) override;
private:
- // Toggles the active state for painting the background and schedules a paint.
- void SetDrawBackgroundAsActive(bool draw_background_as_active);
-
- // Helper function to paint the background and foreground of the AppList
- // button in Chrome OS MD.
- void PaintMd(gfx::Canvas* canvas);
-
- // Helper function to paint the AppList button in Chrome OS non-MD.
- void PaintAppListButton(gfx::Canvas* canvas,
- const gfx::ImageSkia& foreground_image);
-
// Get the center point of the app list button used to draw its background and
// ink drops.
gfx::Point GetCenterPoint() const;
@@ -70,10 +57,6 @@ class ASH_EXPORT AppListButton : public views::ImageButton {
// This is useful because other IsApplistVisible functions aren't per-display.
bool is_showing_app_list_;
- // True if the background should render as active, regardless of the state of
- // the application list.
- bool draw_background_as_active_;
-
// Color used to paint the background.
SkColor background_color_;
« no previous file with comments | « no previous file | ash/common/shelf/app_list_button.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698