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

Unified Diff: ash/test/shelf_view_test_api.h

Issue 2575613002: Remove ShelfItemDelegate::IsDraggable; check for app list button. (Closed)
Patch Set: Created 4 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: ash/test/shelf_view_test_api.h
diff --git a/ash/test/shelf_view_test_api.h b/ash/test/shelf_view_test_api.h
index 52aa27a031570456727fba57a47f727d78f54159..e91c220b86643aee57544b35cbc105b15c57d334 100644
--- a/ash/test/shelf_view_test_api.h
+++ b/ash/test/shelf_view_test_api.h
@@ -5,7 +5,6 @@
#ifndef ASH_TEST_SHELF_VIEW_TEST_API_H_
#define ASH_TEST_SHELF_VIEW_TEST_API_H_
-#include "ash/common/shelf/shelf_item_delegate.h"
#include "ash/common/shelf/shelf_item_types.h"
#include "base/macros.h"
@@ -21,6 +20,7 @@ class Event;
namespace views {
class Button;
class InkDrop;
+class View;
}
namespace ash {
@@ -43,9 +43,12 @@ class ShelfViewTestAPI {
// Number of icons displayed.
int GetButtonCount();
- // Retrieve the button at |index|.
+ // Retrieve the button at |index|, doesn't support the app list button.
James Cook 2016/12/14 17:41:54 nit: add // because the app list button is not a S
msw 2016/12/14 19:46:50 Done (but hopefully I'll change that in the future
ShelfButton* GetButton(int index);
+ // Retrieve the view at |index|.
+ views::View* GetViewAt(int index);
+
// First visible button index.
int GetFirstVisibleIndex();

Powered by Google App Engine
This is Rietveld 408576698