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

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

Issue 451005: Merge 33269 - Revert 33255 Report active extensions in crash reports. This o... (Closed) Base URL: svn://svn.chromium.org/chrome/branches/249/src/
Patch Set: Created 11 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
Index: chrome/browser/extensions/extensions_service.cc
===================================================================
--- chrome/browser/extensions/extensions_service.cc (revision 33272)
+++ chrome/browser/extensions/extensions_service.cc (working copy)
@@ -24,7 +24,6 @@
#include "chrome/browser/extensions/external_pref_extension_provider.h"
#include "chrome/browser/profile.h"
#include "chrome/browser/net/chrome_url_request_context.h"
-#include "chrome/common/child_process_logging.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/extension.h"
#include "chrome/common/extensions/extension_constants.h"
@@ -410,8 +409,6 @@
void ExtensionsService::NotifyExtensionUnloaded(Extension* extension) {
LOG(INFO) << "Sending EXTENSION_UNLOADED";
- UpdateActiveExtensionsInCrashReporter();
-
NotificationService::current()->Notify(
NotificationType::EXTENSION_UNLOADED,
Source<Profile>(profile_),
@@ -619,18 +616,8 @@
break;
}
}
-
- UpdateActiveExtensionsInCrashReporter();
}
-void ExtensionsService::UpdateActiveExtensionsInCrashReporter() {
- std::vector<std::string> extension_ids;
- for (size_t i = 0; i < extensions_.size(); ++i)
- extension_ids.push_back(extensions_[i]->id());
-
- child_process_logging::SetActiveExtensions(extension_ids);
-}
-
void ExtensionsService::OnExtensionInstalled(Extension* extension,
bool allow_privilege_increase) {
extension_prefs_->OnExtensionInstalled(extension);
« no previous file with comments | « chrome/browser/extensions/extensions_service.h ('k') | chrome/browser/renderer_host/browser_render_process_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698