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

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

Issue 185873003: Task Manager: overhaul & re-enable task manager browser tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Retry upload Created 6 years, 10 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/notification_resource_provider.cc
diff --git a/chrome/browser/task_manager/notification_resource_provider.cc b/chrome/browser/task_manager/notification_resource_provider.cc
index 0b317c5e4e95bf8b8eabb262f03cba330d2e7442..7aaad2c3739a8ec658e5e8db3f36ad084783cc61 100644
--- a/chrome/browser/task_manager/notification_resource_provider.cc
+++ b/chrome/browser/task_manager/notification_resource_provider.cc
@@ -205,8 +205,11 @@ void NotificationResourceProvider::Observe(
void NotificationResourceProvider::AddToTaskManager(
BalloonHost* balloon_host) {
+ // The resource may already be tracked, if the task manager was opened
+ // while the BalloonHost was waiting to connect.
+ if (resources_.count(balloon_host))
+ return;
NotificationResource* resource = new NotificationResource(balloon_host);
- DCHECK(resources_.find(balloon_host) == resources_.end());
resources_[balloon_host] = resource;
task_manager_->AddResource(resource);
}
« no previous file with comments | « chrome/browser/task_manager/extension_process_resource_provider.cc ('k') | chrome/browser/task_manager/task_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698