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

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

Issue 2357143004: mash: Support ShelfWindowWatcher via ShelfItem properties. (Closed)
Patch Set: Address comments. Created 4 years, 3 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_item_types.cc ('k') | ash/common/shelf/shelf_window_watcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/shelf/shelf_window_watcher.h
diff --git a/ash/common/shelf/shelf_window_watcher.h b/ash/common/shelf/shelf_window_watcher.h
index 1fa112878371ee3301e7f8c894233270781b9a13..45565e8f4b67f375d0109445f62bf5a321277683 100644
--- a/ash/common/shelf/shelf_window_watcher.h
+++ b/ash/common/shelf/shelf_window_watcher.h
@@ -17,7 +17,7 @@ class ShelfModel;
class WmWindow;
// ShelfWindowWatcher creates and handles a ShelfItem for windows in the default
-// container that have a ShelfItemDetails property (e.g. the task manager
+// container that have a valid ShelfItemType property (e.g. the task manager
// dialog or the OS settings window). It adds the ShelfItem when the window is
// added to the default container and maintains it until the window is closed,
// even if the window is transiently reparented (e.g. during a drag).
@@ -46,7 +46,7 @@ class ShelfWindowWatcher : public WmActivationObserver,
};
// Observes individual user windows to detect when they are closed or when
- // they have ShelfItemDetails changed.
+ // their shelf item properties have changed.
class UserWindowObserver : public WmWindowObserver {
public:
explicit UserWindowObserver(ShelfWindowWatcher* window_watcher);
@@ -63,7 +63,7 @@ class ShelfWindowWatcher : public WmActivationObserver,
DISALLOW_COPY_AND_ASSIGN(UserWindowObserver);
};
- // Creates a ShelfItem for |window| that has ShelfItemDetails.
+ // Creates a ShelfItem for |window|.
void AddShelfItem(WmWindow* window);
// Removes a ShelfItem for |window|.
@@ -79,11 +79,11 @@ class ShelfWindowWatcher : public WmActivationObserver,
void OnContainerWindowDestroying(WmWindow* container);
// Adds a shelf item for new windows added to the default container that have
- // a ShelfItemDetails property.
+ // a valid ShelfItemType property value.
void OnUserWindowAdded(WmWindow* window);
// Adds, updates or removes the shelf item based on a property change.
- void OnUserWindowShelfItemDetailsChanged(WmWindow* window);
+ void OnUserWindowPropertyChanged(WmWindow* window);
// Removes the shelf item when a window closes.
void OnUserWindowDestroying(WmWindow* window);
« no previous file with comments | « ash/common/shelf/shelf_item_types.cc ('k') | ash/common/shelf/shelf_window_watcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698