| Index: chrome/browser/push_messaging/push_messaging_browsertest.cc
|
| diff --git a/chrome/browser/push_messaging/push_messaging_browsertest.cc b/chrome/browser/push_messaging/push_messaging_browsertest.cc
|
| index 1efd1a38214daecd24f67ba080833570b7bd6189..7208af99a36eba5db15ec9d29b84aadbcfd88bc0 100644
|
| --- a/chrome/browser/push_messaging/push_messaging_browsertest.cc
|
| +++ b/chrome/browser/push_messaging/push_messaging_browsertest.cc
|
| @@ -111,9 +111,7 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
| https_server_->ServeFilesFromSourceDirectory("chrome/test/data");
|
| ASSERT_TRUE(https_server_->Start());
|
|
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| notification_manager_.reset(new StubNotificationUIManager);
|
| -#endif
|
|
|
| InProcessBrowserTest::SetUp();
|
| }
|
| @@ -135,12 +133,10 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
|
|
| push_service_ =
|
| PushMessagingServiceFactory::GetForProfile(GetBrowser()->profile());
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| display_service_.reset(new MessageCenterDisplayService(
|
| GetBrowser()->profile(), notification_manager_.get()));
|
| notification_service()->SetNotificationDisplayServiceForTesting(
|
| display_service_.get());
|
| -#endif
|
|
|
| LoadTestPage();
|
| InProcessBrowserTest::SetUpOnMainThread();
|
| @@ -174,10 +170,7 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
|
|
| // InProcessBrowserTest:
|
| void TearDown() override {
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| notification_service()->SetNotificationDisplayServiceForTesting(nullptr);
|
| -#endif
|
| -
|
| InProcessBrowserTest::TearDown();
|
| }
|
|
|
| @@ -240,7 +233,6 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
|
|
| net::EmbeddedTestServer* https_server() const { return https_server_.get(); }
|
|
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| // To be called when delivery of a push message has finished. The |run_loop|
|
| // will be told to quit after |messages_required| messages were received.
|
| void OnDeliveryFinished(std::vector<size_t>* number_of_notifications_shown,
|
| @@ -260,7 +252,6 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
| PlatformNotificationServiceImpl* notification_service() const {
|
| return PlatformNotificationServiceImpl::GetInstance();
|
| }
|
| -#endif
|
|
|
| PushMessagingServiceImpl* push_service() const { return push_service_; }
|
|
|
| @@ -287,10 +278,8 @@ class PushMessagingBrowserTest : public InProcessBrowserTest {
|
| std::unique_ptr<net::EmbeddedTestServer> https_server_;
|
| PushMessagingServiceImpl* push_service_;
|
|
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| std::unique_ptr<StubNotificationUIManager> notification_manager_;
|
| std::unique_ptr<MessageCenterDisplayService> display_service_;
|
| -#endif
|
|
|
| DISALLOW_COPY_AND_ASSIGN(PushMessagingBrowserTest);
|
| };
|
| @@ -1204,7 +1193,6 @@ IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest, PushEventWithoutPermission) {
|
| content::PUSH_UNREGISTRATION_REASON_DELIVERY_PERMISSION_DENIED, 1);
|
| }
|
|
|
| -#if defined(ENABLE_NOTIFICATIONS)
|
| IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest,
|
| PushEventEnforcesUserVisibleNotification) {
|
| std::string script_result;
|
| @@ -1469,7 +1457,6 @@ IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest,
|
| ASSERT_TRUE(RunScript("permissionState()", &script_result));
|
| EXPECT_EQ("permission status - granted", script_result);
|
| }
|
| -#endif
|
|
|
| IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest, PermissionStateSaysPrompt) {
|
| std::string script_result;
|
|
|