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

Unified Diff: chrome/browser/task_manager/task_manager_browsertest.cc

Issue 7053041: Add a Create method to DesktopNotificationHandler and stubs for the notification objects. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 9 years, 6 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: chrome/browser/task_manager/task_manager_browsertest.cc
diff --git a/chrome/browser/task_manager/task_manager_browsertest.cc b/chrome/browser/task_manager/task_manager_browsertest.cc
index 4bd10d4b46e842d2cc7c232a5c02fccb95392f3c..8a060d94e9a07663dad151c70e93c1bb986f3c66 100644
--- a/chrome/browser/task_manager/task_manager_browsertest.cc
+++ b/chrome/browser/task_manager/task_manager_browsertest.cc
@@ -365,39 +365,6 @@ IN_PROC_BROWSER_TEST_F(TaskManagerBrowserTest, NoticeHostedAppTabs) {
ASSERT_TRUE(StartsWith(model()->GetResourceTitle(2), tab_prefix, true));
}
-IN_PROC_BROWSER_TEST_F(TaskManagerBrowserTest, NoticeNotificationChanges) {
- EXPECT_EQ(0, model()->ResourceCount());
-
- // Show the task manager.
- browser()->window()->ShowTaskManager();
- // Expect to see the browser and the New Tab Page renderer.
- WaitForResourceChange(2);
-
- // Show a notification.
- NotificationUIManager* notifications =
- g_browser_process->notification_ui_manager();
-
- string16 content = DesktopNotificationService::CreateDataUrl(
- GURL(), ASCIIToUTF16("Hello World!"), string16(),
- WebKit::WebTextDirectionDefault);
-
- scoped_refptr<NotificationDelegate> del1(new MockNotificationDelegate("n1"));
- Notification n1(
- GURL(), GURL(content), ASCIIToUTF16("Test 1"), string16(), del1.get());
- scoped_refptr<NotificationDelegate> del2(new MockNotificationDelegate("n2"));
- Notification n2(
- GURL(), GURL(content), ASCIIToUTF16("Test 2"), string16(), del2.get());
-
- notifications->Add(n1, browser()->profile());
- WaitForResourceChange(3);
- notifications->Add(n2, browser()->profile());
- WaitForResourceChange(4);
- notifications->CancelById(n1.notification_id());
- WaitForResourceChange(3);
- notifications->CancelById(n2.notification_id());
- WaitForResourceChange(2);
-}
-
IN_PROC_BROWSER_TEST_F(TaskManagerBrowserTest, KillExtension) {
EXPECT_EQ(0, TaskManager::GetBackgroundPageCount());
// Show the task manager. This populates the model, and helps with debugging

Powered by Google App Engine
This is Rietveld 408576698