From 753717a592d27b490657f4e40b188ac6391f5241 Mon Sep 17 00:00:00 2001 From: vadimt Date: Wed, 8 Oct 2014 11:23:19 -0700 Subject: Instrumenting descendants of EventRouter::Observer to find out what causes jankiness of ExtensionMessageFilter::OnExtensionAddListener task. See the bug and the previous checking in the bug for details. BUG=417106 Review URL: https://codereview.chromium.org/638763002 Cr-Commit-Position: refs/heads/master@{#298749} --- chrome/browser/extensions/api/processes/processes_api.cc | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'chrome/browser/extensions/api/processes') diff --git a/chrome/browser/extensions/api/processes/processes_api.cc b/chrome/browser/extensions/api/processes/processes_api.cc index 56a7cb7..cf4fd39 100644 --- a/chrome/browser/extensions/api/processes/processes_api.cc +++ b/chrome/browser/extensions/api/processes/processes_api.cc @@ -9,6 +9,7 @@ #include "base/lazy_instance.h" #include "base/message_loop/message_loop.h" #include "base/metrics/histogram.h" +#include "base/profiler/scoped_profile.h" #include "base/strings/string_number_conversions.h" #include "base/strings/utf_string_conversions.h" #include "base/values.h" @@ -518,6 +519,10 @@ ProcessesEventRouter* ProcessesAPI::processes_event_router() { } void ProcessesAPI::OnListenerAdded(const EventListenerInfo& details) { + // TODO(vadimt): Remove ScopedProfile below once crbug.com/417106 is fixed. + tracked_objects::ScopedProfile tracking_profile( + FROM_HERE_WITH_EXPLICIT_FUNCTION("ProcessesAPI::OnListenerAdded")); + // We lazily tell the TaskManager to start updating when listeners to the // processes.onUpdated or processes.onUpdatedWithMemory events arrive. processes_event_router()->ListenerAdded(); -- cgit v1.1