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

Unified Diff: ash/system/web_notification/popup_alignment_delegate.h

Issue 369573004: Separate the logic of popup alignment and workarea handling as delegate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: polish Created 6 years, 6 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/system/web_notification/popup_alignment_delegate.h
diff --git a/ash/system/web_notification/popup_alignment_delegate.h b/ash/system/web_notification/popup_alignment_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..becaf1aba30c3cd47506f7f0e05aa4aca1985f7f
--- /dev/null
+++ b/ash/system/web_notification/popup_alignment_delegate.h
@@ -0,0 +1,92 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef ASH_SYSTEM_WEB_NOTIFICATION_POPUP_ALIGNMENT_DELEGATE_H_
+#define ASH_SYSTEM_WEB_NOTIFICATION_POPUP_ALIGNMENT_DELEGATE_H_
+
+#include "ash/ash_export.h"
+#include "ash/shelf/shelf_layout_manager_observer.h"
+#include "ash/shelf/shelf_types.h"
+#include "ash/shell_observer.h"
+#include "base/macros.h"
+#include "ui/gfx/display_observer.h"
+#include "ui/gfx/geometry/rect.h"
+#include "ui/message_center/views/popup_alignment_delegate.h"
+
+namespace aura {
+class Window;
+}
+
+namespace gfx {
+class Screen;
+}
+
+namespace ash {
+
+class PopupAlignmentDelegateTest;
+class ShelfLayoutManager;
+class WebNotificationTrayTest;
+
+// The PopupAlignmentDelegate for Ash. It needs to care of alignment of the
stevenjb 2014/07/02 21:59:23 s/care of/handle/ (or 'take care of')
Jun Mukai 2014/07/07 18:12:35 Done.
+// shelf and its autohide state.
+class ASH_EXPORT PopupAlignmentDelegate
stevenjb 2014/07/02 21:59:23 Can we name this AshPopupAlignmentDelegate just to
Jun Mukai 2014/07/07 18:12:36 Done.
+ : public message_center::PopupAlignmentDelegate,
+ public ShelfLayoutManagerObserver,
+ public ShellObserver,
+ public gfx::DisplayObserver {
+ public:
+ PopupAlignmentDelegate();
+ virtual ~PopupAlignmentDelegate();
+
+ // Start observing the system.
+ void StartObserving(gfx::Screen* screen, const gfx::Display& display);
+
+ // Sets the current height of the system tray so that the notification toasts
+ // can avoid it.
+ void SetSystemTrayHeight(int height);
+
+ // Overridden from message_center::PopupAlignmentDelegate:
+ virtual int GetToastOriginX(const gfx::Rect& toast_bounds) const OVERRIDE;
+ virtual int GetBaseLine() const OVERRIDE;
+ virtual int GetWorkAreaBottom() const OVERRIDE;
+ virtual bool IsTopDown() const OVERRIDE;
+ virtual bool IsFromLeft() const OVERRIDE;
+
+ private:
+ friend class PopupAlignmentDelegateTest;
+ friend class WebNotificationTrayTest;
+
+ // Get the current alignment of the shelf.
+ ShelfAlignment GetAlignment() const;
+
+ // Update |shelf_| and start watching when it's set first. This should not
stevenjb 2014/07/02 21:59:23 s/set first/first set/
Jun Mukai 2014/07/07 18:12:36 Done.
+ // be done in the constructor because shelf might not be initialized at that
stevenjb 2014/07/02 21:59:23 s/shelf/the shelf/
Jun Mukai 2014/07/07 18:12:35 Done.
+ // point.
+ void UpdateShelf();
+
+ // Overridden from ShellObserver:
+ virtual void OnDisplayWorkAreaInsetsChanged() OVERRIDE;
+
+ // Overridden from ShelfLayoutManagerObserver:
+ virtual void OnAutoHideStateChanged(ShelfAutoHideState new_state) OVERRIDE;
+
+ // Overridden from gfx::DisplayObserver:
+ virtual void OnDisplayAdded(const gfx::Display& new_display) OVERRIDE;
+ virtual void OnDisplayRemoved(const gfx::Display& old_display) OVERRIDE;
+ virtual void OnDisplayMetricsChanged(const gfx::Display& display,
+ uint32_t metrics) OVERRIDE;
+
+ int64_t display_id_;
+ gfx::Screen* screen_;
+ gfx::Rect work_area_;
+ aura::Window* root_window_;
+ ShelfLayoutManager* shelf_;
+ int system_tray_height_;
+
+ DISALLOW_COPY_AND_ASSIGN(PopupAlignmentDelegate);
+};
+
+} // namespace message_center
+
+#endif // ASH_SYSTEM_WEB_NOTIFICATION_ASH_POPUP_ALIGNMENT_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698