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

Unified Diff: ash/test/shelf_view_test_api.cc

Issue 2178163002: Add ink drop ripple to shelf overflow button (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed OnClickCanceled Created 4 years, 4 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: ash/test/shelf_view_test_api.cc
diff --git a/ash/test/shelf_view_test_api.cc b/ash/test/shelf_view_test_api.cc
index a14432c1f958ecc65d3c30f7f3d96a4e4cb8a0ab..700ed2e9d83e5a9b4ac0b5d5113b6c788113346b 100644
--- a/ash/test/shelf_view_test_api.cc
+++ b/ash/test/shelf_view_test_api.cc
@@ -72,6 +72,11 @@ void ShelfViewTestAPI::ShowOverflowBubble() {
shelf_view_->ToggleOverflowBubble();
}
+void ShelfViewTestAPI::HideOverflowBubble() {
+ if (shelf_view_->IsShowingOverflowBubble())
bruthig 2016/09/01 16:12:54 Can you make this a DCHECK instead of an if? It i
mohsen 2016/09/10 03:03:23 Done. Also, for the ShowOverflowBubble().
+ shelf_view_->ToggleOverflowBubble();
+}
+
const gfx::Rect& ShelfViewTestAPI::GetBoundsByIndex(int index) {
return shelf_view_->view_model_->view_at(index)->bounds();
}
@@ -110,6 +115,10 @@ OverflowBubble* ShelfViewTestAPI::overflow_bubble() {
return shelf_view_->overflow_bubble_.get();
}
+OverflowButton* ShelfViewTestAPI::overflow_button() const {
+ return shelf_view_->overflow_button_;
+}
+
ShelfTooltipManager* ShelfViewTestAPI::tooltip_manager() {
return &shelf_view_->tooltip_;
}

Powered by Google App Engine
This is Rietveld 408576698