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

Unified Diff: chrome/browser/push_messaging/push_messaging_service_impl.h

Issue 2555833002: Remove enable_notifications build flag and define (Closed)
Patch Set: Merge Created 4 years 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: chrome/browser/push_messaging/push_messaging_service_impl.h
diff --git a/chrome/browser/push_messaging/push_messaging_service_impl.h b/chrome/browser/push_messaging/push_messaging_service_impl.h
index 59dd061e4eb00ebf2931bea6689135ec1fcb3f35..66c769496bdd1c3965ffdfd03b1fa43f4981f927 100644
--- a/chrome/browser/push_messaging/push_messaging_service_impl.h
+++ b/chrome/browser/push_messaging/push_messaging_service_impl.h
@@ -16,6 +16,7 @@
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
#include "chrome/browser/background/background_trigger.h"
+#include "chrome/browser/push_messaging/push_messaging_notification_manager.h"
#include "chrome/common/features.h"
#include "components/content_settings/core/browser/content_settings_observer.h"
#include "components/content_settings/core/common/content_settings.h"
@@ -30,10 +31,6 @@
#include "third_party/WebKit/public/platform/modules/permissions/permission_status.mojom.h"
#include "third_party/WebKit/public/platform/modules/push_messaging/WebPushPermissionStatus.h"
-#if defined(ENABLE_NOTIFICATIONS)
-#include "chrome/browser/push_messaging/push_messaging_notification_manager.h"
-#endif
-
class Profile;
class PushMessagingAppIdentifier;
class PushMessagingServiceObserver;
@@ -252,9 +249,7 @@ class PushMessagingServiceImpl : public content::PushMessagingService,
base::Closure message_callback_for_testing_;
base::Closure content_setting_changed_callback_for_testing_;
-#if defined(ENABLE_NOTIFICATIONS)
PushMessagingNotificationManager notification_manager_;
-#endif
// A multiset containing one entry for each in-flight push message delivery,
// keyed by the receiver's app id.

Powered by Google App Engine
This is Rietveld 408576698