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

Unified Diff: ui/arc/notification/arc_custom_notification_view.cc

Issue 2870283002: Move message_center::CustomNotificationView to arc::ArcNotificationView (Closed)
Patch Set: Addressed comment Created 3 years, 7 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: ui/arc/notification/arc_custom_notification_view.cc
diff --git a/ui/arc/notification/arc_custom_notification_view.cc b/ui/arc/notification/arc_custom_notification_view.cc
index 3a8d99ee08a8bb26c39da6a620514703eb059f5a..06fd9465307e4edfb83ee749f35c77056ac985b8 100644
--- a/ui/arc/notification/arc_custom_notification_view.cc
+++ b/ui/arc/notification/arc_custom_notification_view.cc
@@ -10,6 +10,7 @@
#include "components/exo/notification_surface.h"
#include "components/exo/surface.h"
#include "ui/accessibility/ax_action_data.h"
+#include "ui/arc/notification/arc_notification_view.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/compositor/layer_animation_observer.h"
@@ -19,7 +20,6 @@
#include "ui/gfx/canvas.h"
#include "ui/gfx/transform.h"
#include "ui/message_center/message_center_style.h"
-#include "ui/message_center/views/custom_notification_view.h"
#include "ui/message_center/views/toast_contents_view.h"
#include "ui/strings/grit/ui_strings.h"
#include "ui/views/background.h"
@@ -175,7 +175,7 @@ class ArcCustomNotificationView::SlideHelper
};
class ArcCustomNotificationView::ContentViewDelegate
- : public message_center::CustomNotificationContentViewDelegate {
+ : public ArcNotificationContentViewDelegate {
public:
explicit ContentViewDelegate(ArcCustomNotificationView* owner)
: owner_(owner) {}
@@ -264,7 +264,7 @@ ArcCustomNotificationView::~ArcCustomNotificationView() {
}
}
-std::unique_ptr<message_center::CustomNotificationContentViewDelegate>
+std::unique_ptr<ArcNotificationContentViewDelegate>
ArcCustomNotificationView::CreateContentViewDelegate() {
return base::MakeUnique<ArcCustomNotificationView::ContentViewDelegate>(this);
}
@@ -580,12 +580,10 @@ void ArcCustomNotificationView::OnMouseExited(const ui::MouseEvent&) {
}
void ArcCustomNotificationView::OnFocus() {
- CHECK_EQ(message_center::CustomNotificationView::kViewClassName,
- parent()->GetClassName());
+ CHECK_EQ(ArcNotificationView::kViewClassName, parent()->GetClassName());
NativeViewHost::OnFocus();
- static_cast<message_center::CustomNotificationView*>(parent())
- ->OnContentFocused();
+ static_cast<ArcNotificationView*>(parent())->OnContentFocused();
}
void ArcCustomNotificationView::OnBlur() {
@@ -594,12 +592,10 @@ void ArcCustomNotificationView::OnBlur() {
return;
}
- CHECK_EQ(message_center::CustomNotificationView::kViewClassName,
- parent()->GetClassName());
+ CHECK_EQ(ArcNotificationView::kViewClassName, parent()->GetClassName());
NativeViewHost::OnBlur();
- static_cast<message_center::CustomNotificationView*>(parent())
- ->OnContentBlured();
+ static_cast<ArcNotificationView*>(parent())->OnContentBlured();
}
void ArcCustomNotificationView::ActivateToast() {
@@ -629,10 +625,8 @@ bool ArcCustomNotificationView::HandleAccessibleAction(
void ArcCustomNotificationView::ButtonPressed(views::Button* sender,
const ui::Event& event) {
if (item_ && !item_->GetPinned() && sender == close_button_.get()) {
- CHECK_EQ(message_center::CustomNotificationView::kViewClassName,
- parent()->GetClassName());
- static_cast<message_center::CustomNotificationView*>(parent())
- ->OnCloseButtonPressed();
+ CHECK_EQ(ArcNotificationView::kViewClassName, parent()->GetClassName());
+ static_cast<ArcNotificationView*>(parent())->OnCloseButtonPressed();
}
if (item_ && settings_button_ && sender == settings_button_) {
item_->OpenSettings();
« no previous file with comments | « ui/arc/notification/arc_custom_notification_view.h ('k') | ui/arc/notification/arc_notification_content_view_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698