| OLD | NEW |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include <memory> | 5 #include <memory> |
| 6 #include <string> | 6 #include <string> |
| 7 #include <vector> | 7 #include <vector> |
| 8 | 8 |
| 9 #include "base/command_line.h" | 9 #include "base/command_line.h" |
| 10 #include "base/feature_list.h" | 10 #include "base/feature_list.h" |
| 11 #include "base/files/file_path.h" | 11 #include "base/files/file_path.h" |
| 12 #include "base/path_service.h" | 12 #include "base/path_service.h" |
| 13 #include "base/strings/string_piece.h" | 13 #include "base/strings/string_piece.h" |
| 14 #include "base/strings/string_split.h" | 14 #include "base/strings/string_split.h" |
| 15 #include "base/strings/utf_string_conversions.h" | 15 #include "base/strings/utf_string_conversions.h" |
| 16 #include "base/test/scoped_feature_list.h" | 16 #include "base/test/scoped_feature_list.h" |
| 17 #include "base/time/time.h" | 17 #include "base/time/time.h" |
| 18 #include "chrome/browser/notifications/desktop_notification_profile_util.h" | 18 #include "chrome/browser/notifications/desktop_notification_profile_util.h" |
| 19 #include "chrome/browser/notifications/message_center_display_service.h" | 19 #include "chrome/browser/notifications/message_center_display_service.h" |
| 20 #include "chrome/browser/notifications/notification.h" | 20 #include "chrome/browser/notifications/notification.h" |
| 21 #include "chrome/browser/notifications/notification_test_util.h" | 21 #include "chrome/browser/notifications/notification_test_util.h" |
| 22 #include "chrome/browser/notifications/platform_notification_service_impl.h" | 22 #include "chrome/browser/notifications/platform_notification_service_impl.h" |
| 23 #include "chrome/browser/notifications/web_notification_delegate.h" | 23 #include "chrome/browser/notifications/web_notification_delegate.h" |
| 24 #include "chrome/browser/permissions/permission_manager.h" | 24 #include "chrome/browser/permissions/permission_manager.h" |
| 25 #include "chrome/browser/permissions/permission_request_manager.h" | 25 #include "chrome/browser/permissions/permission_request_manager.h" |
| 26 #include "chrome/browser/permissions/permission_result.h" |
| 26 #include "chrome/browser/ui/browser.h" | 27 #include "chrome/browser/ui/browser.h" |
| 27 #include "chrome/browser/ui/browser_window.h" | 28 #include "chrome/browser/ui/browser_window.h" |
| 28 #include "chrome/browser/ui/exclusive_access/exclusive_access_context.h" | 29 #include "chrome/browser/ui/exclusive_access/exclusive_access_context.h" |
| 29 #include "chrome/browser/ui/tabs/tab_strip_model.h" | 30 #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| 30 #include "chrome/common/chrome_features.h" | 31 #include "chrome/common/chrome_features.h" |
| 31 #include "chrome/test/base/in_process_browser_test.h" | 32 #include "chrome/test/base/in_process_browser_test.h" |
| 32 #include "chrome/test/base/interactive_test_utils.h" | 33 #include "chrome/test/base/interactive_test_utils.h" |
| 33 #include "chrome/test/base/ui_test_utils.h" | 34 #include "chrome/test/base/ui_test_utils.h" |
| 34 #include "components/content_settings/core/common/content_settings_types.h" | 35 #include "components/content_settings/core/common/content_settings_types.h" |
| 35 #include "content/public/common/content_features.h" | 36 #include "content/public/common/content_features.h" |
| (...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 155 | 156 |
| 156 void PlatformNotificationServiceBrowserTest::TearDown() { | 157 void PlatformNotificationServiceBrowserTest::TearDown() { |
| 157 service()->SetNotificationDisplayServiceForTesting(nullptr); | 158 service()->SetNotificationDisplayServiceForTesting(nullptr); |
| 158 } | 159 } |
| 159 | 160 |
| 160 void PlatformNotificationServiceBrowserTest:: | 161 void PlatformNotificationServiceBrowserTest:: |
| 161 GrantNotificationPermissionForTest() const { | 162 GrantNotificationPermissionForTest() const { |
| 162 GURL origin = TestPageUrl().GetOrigin(); | 163 GURL origin = TestPageUrl().GetOrigin(); |
| 163 | 164 |
| 164 DesktopNotificationProfileUtil::GrantPermission(browser()->profile(), origin); | 165 DesktopNotificationProfileUtil::GrantPermission(browser()->profile(), origin); |
| 165 ASSERT_EQ(blink::mojom::PermissionStatus::GRANTED, | 166 ASSERT_EQ(CONTENT_SETTING_ALLOW, |
| 166 PermissionManager::Get(browser()->profile()) | 167 PermissionManager::Get(browser()->profile()) |
| 167 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, | 168 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, |
| 168 origin, origin)); | 169 origin, origin) |
| 170 .content_setting); |
| 169 } | 171 } |
| 170 | 172 |
| 171 void PlatformNotificationServiceBrowserTest::NavigateToTestPage( | 173 void PlatformNotificationServiceBrowserTest::NavigateToTestPage( |
| 172 const std::string& path) const { | 174 const std::string& path) const { |
| 173 ui_test_utils::NavigateToURL(browser(), https_server_->GetURL(path)); | 175 ui_test_utils::NavigateToURL(browser(), https_server_->GetURL(path)); |
| 174 } | 176 } |
| 175 | 177 |
| 176 bool PlatformNotificationServiceBrowserTest::RunScript( | 178 bool PlatformNotificationServiceBrowserTest::RunScript( |
| 177 const std::string& script, std::string* result) const { | 179 const std::string& script, std::string* result) const { |
| 178 return content::ExecuteScriptAndExtractString( | 180 return content::ExecuteScriptAndExtractString( |
| (...skipping 283 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 462 // TODO(felt): This DCHECKs when bubbles are enabled, when the file_url is | 464 // TODO(felt): This DCHECKs when bubbles are enabled, when the file_url is |
| 463 // persisted. crbug.com/502057 | 465 // persisted. crbug.com/502057 |
| 464 IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceBrowserTest, | 466 IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceBrowserTest, |
| 465 DISABLED_CheckFilePermissionNotGranted) { | 467 DISABLED_CheckFilePermissionNotGranted) { |
| 466 // This case should succeed because a normal page URL is used. | 468 // This case should succeed because a normal page URL is used. |
| 467 std::string script_result; | 469 std::string script_result; |
| 468 | 470 |
| 469 PermissionManager* permission_manager = | 471 PermissionManager* permission_manager = |
| 470 PermissionManager::Get(browser()->profile()); | 472 PermissionManager::Get(browser()->profile()); |
| 471 | 473 |
| 472 EXPECT_EQ(blink::mojom::PermissionStatus::ASK, | 474 EXPECT_EQ(CONTENT_SETTING_ASK, |
| 473 permission_manager->GetPermissionStatus( | 475 permission_manager |
| 474 CONTENT_SETTINGS_TYPE_NOTIFICATIONS, TestPageUrl(), | 476 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, |
| 475 TestPageUrl())); | 477 TestPageUrl(), TestPageUrl()) |
| 478 .content_setting); |
| 476 | 479 |
| 477 RequestAndAcceptPermission(); | 480 RequestAndAcceptPermission(); |
| 478 EXPECT_EQ(blink::mojom::PermissionStatus::GRANTED, | 481 EXPECT_EQ(CONTENT_SETTING_ALLOW, |
| 479 permission_manager->GetPermissionStatus( | 482 permission_manager |
| 480 CONTENT_SETTINGS_TYPE_NOTIFICATIONS, TestPageUrl(), | 483 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, |
| 481 TestPageUrl())); | 484 TestPageUrl(), TestPageUrl()) |
| 485 .content_setting); |
| 482 | 486 |
| 483 // This case should fail because a file URL is used. | 487 // This case should fail because a file URL is used. |
| 484 base::FilePath dir_source_root; | 488 base::FilePath dir_source_root; |
| 485 EXPECT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &dir_source_root)); | 489 EXPECT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &dir_source_root)); |
| 486 base::FilePath full_file_path = | 490 base::FilePath full_file_path = |
| 487 dir_source_root.Append(server_root()).AppendASCII(kTestFileName); | 491 dir_source_root.Append(server_root()).AppendASCII(kTestFileName); |
| 488 GURL file_url(net::FilePathToFileURL(full_file_path)); | 492 GURL file_url(net::FilePathToFileURL(full_file_path)); |
| 489 | 493 |
| 490 ui_test_utils::NavigateToURL(browser(), file_url); | 494 ui_test_utils::NavigateToURL(browser(), file_url); |
| 491 | 495 |
| 492 EXPECT_EQ(blink::mojom::PermissionStatus::ASK, | 496 EXPECT_EQ(CONTENT_SETTING_ASK, |
| 493 permission_manager->GetPermissionStatus( | 497 permission_manager |
| 494 CONTENT_SETTINGS_TYPE_NOTIFICATIONS, file_url, file_url)); | 498 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, |
| 499 file_url, file_url) |
| 500 .content_setting); |
| 495 | 501 |
| 496 RequestAndAcceptPermission(); | 502 RequestAndAcceptPermission(); |
| 497 EXPECT_EQ(blink::mojom::PermissionStatus::ASK, | 503 EXPECT_EQ(CONTENT_SETTING_ASK, |
| 498 permission_manager->GetPermissionStatus( | 504 permission_manager |
| 499 CONTENT_SETTINGS_TYPE_NOTIFICATIONS, file_url, file_url)) | 505 ->GetPermissionStatus(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, |
| 506 file_url, file_url) |
| 507 .content_setting) |
| 500 << "If this test fails, you may have fixed a bug preventing file origins " | 508 << "If this test fails, you may have fixed a bug preventing file origins " |
| 501 << "from sending their origin from Blink; if so you need to update the " | 509 << "from sending their origin from Blink; if so you need to update the " |
| 502 << "display function for notification origins to show the file path."; | 510 << "display function for notification origins to show the file path."; |
| 503 } | 511 } |
| 504 | 512 |
| 505 IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceBrowserTest, | 513 IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceBrowserTest, |
| 506 DataUrlAsNotificationImage) { | 514 DataUrlAsNotificationImage) { |
| 507 ASSERT_NO_FATAL_FAILURE(GrantNotificationPermissionForTest()); | 515 ASSERT_NO_FATAL_FAILURE(GrantNotificationPermissionForTest()); |
| 508 | 516 |
| 509 std::string script_result; | 517 std::string script_result; |
| (...skipping 271 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 781 RunScript("DisplayPersistentAllOptionsNotification()", &script_result)); | 789 RunScript("DisplayPersistentAllOptionsNotification()", &script_result)); |
| 782 EXPECT_EQ("ok", script_result); | 790 EXPECT_EQ("ok", script_result); |
| 783 | 791 |
| 784 ASSERT_EQ(1u, ui_manager()->GetNotificationCount()); | 792 ASSERT_EQ(1u, ui_manager()->GetNotificationCount()); |
| 785 const Notification& notification = ui_manager()->GetNotificationAt(0); | 793 const Notification& notification = ui_manager()->GetNotificationAt(0); |
| 786 | 794 |
| 787 // Since the kNotificationContentImage kill switch has disabled images, the | 795 // Since the kNotificationContentImage kill switch has disabled images, the |
| 788 // notification should be shown without an image. | 796 // notification should be shown without an image. |
| 789 EXPECT_TRUE(notification.image().IsEmpty()); | 797 EXPECT_TRUE(notification.image().IsEmpty()); |
| 790 } | 798 } |
| OLD | NEW |