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

Unified Diff: shell/application_manager/application_manager.cc

Issue 1024723002: Add some trace points to mojo_shell for app lifetime (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 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: shell/application_manager/application_manager.cc
diff --git a/shell/application_manager/application_manager.cc b/shell/application_manager/application_manager.cc
index 8ffe80860d6f97e3011363cf8e6b59330a17b108..cb9ed293dfab1ebb26dbb9d7f02999293ef95181 100644
--- a/shell/application_manager/application_manager.cc
+++ b/shell/application_manager/application_manager.cc
@@ -10,6 +10,7 @@
#include "base/macros.h"
#include "base/stl_util.h"
#include "base/strings/string_util.h"
+#include "base/trace_event/trace_event.h"
#include "mojo/public/cpp/bindings/binding.h"
#include "mojo/public/cpp/bindings/error_handler.h"
#include "mojo/services/content_handler/public/interfaces/content_handler.mojom.h"
@@ -111,6 +112,9 @@ void ApplicationManager::ConnectToApplication(
const GURL& requestor_url,
InterfaceRequest<ServiceProvider> services,
ServiceProviderPtr exposed_services) {
+ TRACE_EVENT_INSTANT1("mojo_shell", "ApplicationManager::ConnectToApplication",
+ TRACE_EVENT_SCOPE_THREAD, "requested_url",
+ requested_url.spec());
DCHECK(requested_url.is_valid());
// We check both the mapped and resolved urls for existing shell_impls because
@@ -330,6 +334,8 @@ void ApplicationManager::RunNativeApplication(
return;
}
+ TRACE_EVENT1("mojo_shell", "ApplicationManager::RunNativeApplication", "path",
+ path.AsUTF8Unsafe());
NativeRunner* runner = native_runner_factory_->Create(options).release();
native_runners_.push_back(runner);
runner->Start(path, cleanup_behavior, application_request.Pass(),
« no previous file with comments | « no previous file | shell/application_manager/local_fetcher.cc » ('j') | shell/application_manager/local_fetcher.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698