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

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

Issue 2228713003: mash: Convert ShelfView to wm common types, clean up Shelf access (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix control reaches end of non-void function warnings 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
« no previous file with comments | « ash/common/shelf/shelf_tooltip_manager_unittest.cc ('k') | ash/common/shelf/shelf_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/shelf/shelf_view.h
diff --git a/ash/shelf/shelf_view.h b/ash/common/shelf/shelf_view.h
similarity index 98%
rename from ash/shelf/shelf_view.h
rename to ash/common/shelf/shelf_view.h
index dffd22782f840a7b93f731fdf58784f89158598b..e5f0deccc63facb1725cd21d852addcf377e3cc0 100644
--- a/ash/shelf/shelf_view.h
+++ b/ash/common/shelf/shelf_view.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef ASH_SHELF_SHELF_VIEW_H_
-#define ASH_SHELF_SHELF_VIEW_H_
+#ifndef ASH_COMMON_SHELF_SHELF_VIEW_H_
+#define ASH_COMMON_SHELF_SHELF_VIEW_H_
#include <memory>
#include <string>
@@ -42,12 +42,12 @@ class DragImageView;
class OverflowBubble;
class OverflowButton;
class ScopedRootWindowForNewWindows;
-class Shelf;
class ShelfButton;
class ShelfDelegate;
class ShelfIconObserver;
class ShelfModel;
struct ShelfItem;
+class ShelfWidget;
class WmShelf;
namespace test {
@@ -70,10 +70,9 @@ class ASH_EXPORT ShelfView : public views::View,
ShelfView(ShelfModel* model,
ShelfDelegate* delegate,
WmShelf* wm_shelf,
- Shelf* shelf);
+ ShelfWidget* shelf_widget);
~ShelfView() override;
- Shelf* shelf() const { return shelf_; }
WmShelf* wm_shelf() const { return wm_shelf_; }
ShelfModel* model() const { return model_; }
@@ -352,8 +351,9 @@ class ASH_EXPORT ShelfView : public views::View,
// The shelf controller; owned by RootWindowController.
WmShelf* wm_shelf_;
- // The shelf; owned by ShelfWidget.
- Shelf* shelf_;
+ // The shelf widget for this view. For overflow bubbles, this is the widget
+ // for the shelf, not for the bubble.
+ ShelfWidget* shelf_widget_;
// Used to manage the set of active launcher buttons. There is a view per
// item in |model_|.
@@ -474,4 +474,4 @@ class ASH_EXPORT ShelfView : public views::View,
} // namespace ash
-#endif // ASH_SHELF_SHELF_VIEW_H_
+#endif // ASH_COMMON_SHELF_SHELF_VIEW_H_
« no previous file with comments | « ash/common/shelf/shelf_tooltip_manager_unittest.cc ('k') | ash/common/shelf/shelf_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698