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

Unified Diff: sky/engine/core/fetch/ResourceFetcher.cpp

Issue 1107853003: Fix up strings Sky passes to base/trace_event (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 8 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: sky/engine/core/fetch/ResourceFetcher.cpp
diff --git a/sky/engine/core/fetch/ResourceFetcher.cpp b/sky/engine/core/fetch/ResourceFetcher.cpp
index 913482074e1aba1578c43757ed2cf7d4523f4a32..ec4ab7aac8b4127f077c41a8b46221ae4be1b19b 100644
--- a/sky/engine/core/fetch/ResourceFetcher.cpp
+++ b/sky/engine/core/fetch/ResourceFetcher.cpp
@@ -694,7 +694,10 @@ void ResourceFetcher::willTerminateResourceLoader(ResourceLoader* loader)
void ResourceFetcher::willStartLoadingResource(Resource* resource, ResourceRequest& request)
{
- TRACE_EVENT_ASYNC_BEGIN2("net", "Resource", resource, "url", resource->url().string().ascii().data(), "priority", resource->resourceRequest().priority());
+ TRACE_EVENT_ASYNC_BEGIN2(
+ "net", "Resource", resource, "url",
+ TRACE_STR_COPY(resource->url().string().ascii().data()), "priority",
jamesr 2015/04/29 00:12:46 same here
+ resource->resourceRequest().priority());
}
void ResourceFetcher::stopFetching()

Powered by Google App Engine
This is Rietveld 408576698