Make GlobalErrorService's ownership model slightly less insane.

chrome/browser/extensions/warning_badge_service.cc

Index: chrome/browser/extensions/warning_badge_service.cc
diff --git a/chrome/browser/extensions/warning_badge_service.cc b/chrome/browser/extensions/warning_badge_service.cc
index cebe7a67f1c7e52ed98aa072528c717124fb3411..b485975be5ccb16a1022e1d177a450dafd6697df 100644
--- a/chrome/browser/extensions/warning_badge_service.cc
+++ b/chrome/browser/extensions/warning_badge_service.cc
@@ -7,6 +7,7 @@
 #include <stddef.h>
 
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/stl_util.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/extensions/warning_badge_service_factory.h"
@@ -154,12 +155,10 @@ void WarningBadgeService::ShowBadge(bool show) {
       ErrorBadge::GetMenuItemCommandID());
 
   // Activate or hide the warning badge in case the current state is incorrect.
-  if (error && !show) {
-    service->RemoveGlobalError(error);
-    delete error;
-  } else if (!error && show) {
-    service->AddGlobalError(new ErrorBadge(this));
-  }
+  if (error && !show)
+    service->RemoveOwnedGlobalError(error);
+  else if (!error && show)
+    service->AddOwnedGlobalError(base::MakeUnique<ErrorBadge>(this));
 }
 
 }  // namespace extensions