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

Unified Diff: chrome/browser/extensions/extensions_service.cc

Issue 5098001: Revert some old debugging code for tracking down a (now-closed) crash bug. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years, 1 month 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
« no previous file with comments | « chrome/browser/extensions/extension_host.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extensions_service.cc
diff --git a/chrome/browser/extensions/extensions_service.cc b/chrome/browser/extensions/extensions_service.cc
index 05c6768a9d6e3ac6ed15d2559bb06511fa6a36b3..03bed1a25c08583714faa41f937f4b77d208b4a7 100644
--- a/chrome/browser/extensions/extensions_service.cc
+++ b/chrome/browser/extensions/extensions_service.cc
@@ -1874,21 +1874,6 @@ void ExtensionsService::Observe(NotificationType type,
if (profile_ != Source<Profile>(source).ptr()->GetOriginalProfile())
break;
- ExtensionHost* host = Details<ExtensionHost>(details).ptr();
-
- // TODO(rafaelw): Remove this check and ExtensionHost::recently_deleted().
- // This is only here to help track down crbug.com/49114.
- ExtensionHost::HostPointerList::iterator iter =
- ExtensionHost::recently_deleted()->begin();
- for (; iter != ExtensionHost::recently_deleted()->end(); ++iter) {
- if (*iter == host) {
- CHECK(host->GetURL().spec().size() + 2 != 0);
- break;
- }
- }
- if (iter == ExtensionHost::recently_deleted()->end())
- CHECK(host->GetURL().spec().size() + 1 != 0);
-
// Unload the entire extension. We want it to be in a consistent state:
// either fully working or not loaded at all, but never half-crashed.
// We do it in a PostTask so that other handlers of this notification will
« no previous file with comments | « chrome/browser/extensions/extension_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698