Commit 5b4239fa authored by Anita Woodruff's avatar Anita Woodruff Committed by Commit Bot

[Cleanup] Fix TODOs in PlatformNotificationService Interactive UI Test

Change-Id: I6b8d69e82bc99b1a01ca053afb31b044362048ae
Reviewed-on: https://chromium-review.googlesource.com/888621
Commit-Queue: Anita Woodruff <awdf@chromium.org>
Reviewed-by: default avatarPeter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532397}
parent ef82c674
......@@ -214,7 +214,7 @@ PlatformNotificationServiceBrowserTest::PlatformNotificationServiceBrowserTest()
: server_root_(FILE_PATH_LITERAL("chrome/test/data")) {}
// TODO(peter): Move PlatformNotificationService-related tests over from
// notification_browsertest.cc to this file.
// notification_interactive_uitest.cc to this file.
IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceBrowserTest,
DisplayPersistentNotificationWithoutPermission) {
......@@ -1004,9 +1004,6 @@ IN_PROC_BROWSER_TEST_F(PlatformNotificationServiceMojoEnabledBrowserTest,
// First, test the default values.
std::string script_result;
// TODO(crbug.com/595685): Can simply call
// 'RunScript("DisplayNonPersistentNotification(...' once the show event is
// implemented via mojo, here and elsewhere in this test.
RunScriptAndWaitForNotificationAdded(
"DisplayNonPersistentNotification('Title')", &script_result);
EXPECT_EQ("ok", script_result);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment