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

Side by Side Diff: third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

Issue 1815383003: Attach requestId to resource fetch trace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move trace call to FrameFetchContext Created 4 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 unified diff | Download patch
« no previous file with comments | « third_party/WebKit/Source/core/loader/FrameFetchContext.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 25 matching lines...) Expand all
36 #include "core/fetch/ResourceLoader.h" 36 #include "core/fetch/ResourceLoader.h"
37 #include "core/fetch/UniqueIdentifier.h" 37 #include "core/fetch/UniqueIdentifier.h"
38 #include "core/frame/FrameConsole.h" 38 #include "core/frame/FrameConsole.h"
39 #include "core/frame/FrameHost.h" 39 #include "core/frame/FrameHost.h"
40 #include "core/frame/FrameView.h" 40 #include "core/frame/FrameView.h"
41 #include "core/frame/LocalFrame.h" 41 #include "core/frame/LocalFrame.h"
42 #include "core/frame/Settings.h" 42 #include "core/frame/Settings.h"
43 #include "core/html/HTMLFrameOwnerElement.h" 43 #include "core/html/HTMLFrameOwnerElement.h"
44 #include "core/html/imports/HTMLImportsController.h" 44 #include "core/html/imports/HTMLImportsController.h"
45 #include "core/inspector/ConsoleMessage.h" 45 #include "core/inspector/ConsoleMessage.h"
46 #include "core/inspector/IdentifiersFactory.h"
46 #include "core/inspector/InspectorInstrumentation.h" 47 #include "core/inspector/InspectorInstrumentation.h"
47 #include "core/inspector/InspectorResourceAgent.h" 48 #include "core/inspector/InspectorResourceAgent.h"
48 #include "core/inspector/InspectorTraceEvents.h" 49 #include "core/inspector/InspectorTraceEvents.h"
49 #include "core/inspector/InstrumentingAgents.h" 50 #include "core/inspector/InstrumentingAgents.h"
50 #include "core/loader/DocumentLoader.h" 51 #include "core/loader/DocumentLoader.h"
51 #include "core/loader/FrameLoader.h" 52 #include "core/loader/FrameLoader.h"
52 #include "core/loader/FrameLoaderClient.h" 53 #include "core/loader/FrameLoaderClient.h"
53 #include "core/loader/LinkLoader.h" 54 #include "core/loader/LinkLoader.h"
54 #include "core/loader/MixedContentChecker.h" 55 #include "core/loader/MixedContentChecker.h"
55 #include "core/loader/NetworkHintsInterface.h" 56 #include "core/loader/NetworkHintsInterface.h"
56 #include "core/loader/PingLoader.h" 57 #include "core/loader/PingLoader.h"
57 #include "core/loader/ProgressTracker.h" 58 #include "core/loader/ProgressTracker.h"
58 #include "core/loader/appcache/ApplicationCacheHost.h" 59 #include "core/loader/appcache/ApplicationCacheHost.h"
59 #include "core/page/Page.h" 60 #include "core/page/Page.h"
60 #include "core/svg/graphics/SVGImageChromeClient.h" 61 #include "core/svg/graphics/SVGImageChromeClient.h"
61 #include "core/timing/DOMWindowPerformance.h" 62 #include "core/timing/DOMWindowPerformance.h"
62 #include "core/timing/Performance.h" 63 #include "core/timing/Performance.h"
63 #include "platform/Logging.h" 64 #include "platform/Logging.h"
65 #include "platform/TracedValue.h"
64 #include "platform/mhtml/MHTMLArchive.h" 66 #include "platform/mhtml/MHTMLArchive.h"
65 #include "platform/network/ResourceTimingInfo.h" 67 #include "platform/network/ResourceTimingInfo.h"
66 #include "platform/weborigin/SchemeRegistry.h" 68 #include "platform/weborigin/SchemeRegistry.h"
67 #include "platform/weborigin/SecurityPolicy.h" 69 #include "platform/weborigin/SecurityPolicy.h"
68 #include "public/platform/WebFrameScheduler.h" 70 #include "public/platform/WebFrameScheduler.h"
69 71
70 #include <algorithm> 72 #include <algorithm>
71 73
72 namespace blink { 74 namespace blink {
73 75
(...skipping 257 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 333
332 bool FrameFetchContext::shouldLoadNewResource(Resource::Type type) const 334 bool FrameFetchContext::shouldLoadNewResource(Resource::Type type) const
333 { 335 {
334 if (!m_documentLoader) 336 if (!m_documentLoader)
335 return true; 337 return true;
336 if (type == Resource::MainResource) 338 if (type == Resource::MainResource)
337 return m_documentLoader == frame()->loader().provisionalDocumentLoader() ; 339 return m_documentLoader == frame()->loader().provisionalDocumentLoader() ;
338 return m_documentLoader == frame()->loader().documentLoader(); 340 return m_documentLoader == frame()->loader().documentLoader();
339 } 341 }
340 342
341 void FrameFetchContext::willStartLoadingResource(ResourceRequest& request) 343 static PassOwnPtr<TracedValue> loadResourceTraceData(unsigned long identifier, c onst KURL& url, int priority)
342 { 344 {
345 String requestId = IdentifiersFactory::requestId(identifier);
346
347 OwnPtr<TracedValue> value = TracedValue::create();
348 value->setString("requestId", requestId);
349 value->setString("url", url.getString());
350 value->setInteger("priority", priority);
351 return value.release();
352 }
353
354 void FrameFetchContext::willStartLoadingResource(Resource* resource, ResourceReq uest& request)
355 {
356 TRACE_EVENT_ASYNC_BEGIN1("blink.net", "Resource", resource, "data", loadReso urceTraceData(resource->identifier(), resource->url(), resource->resourceRequest ().priority()));
357
343 if (m_documentLoader) 358 if (m_documentLoader)
344 m_documentLoader->applicationCacheHost()->willStartLoadingResource(reque st); 359 m_documentLoader->applicationCacheHost()->willStartLoadingResource(reque st);
345 } 360 }
346 361
347 void FrameFetchContext::didLoadResource(Resource* resource) 362 void FrameFetchContext::didLoadResource(Resource* resource)
348 { 363 {
349 if (resource->isLoadEventBlockingResourceType()) 364 if (resource->isLoadEventBlockingResourceType())
350 frame()->loader().checkCompleted(); 365 frame()->loader().checkCompleted();
351 } 366 }
352 367
(...skipping 452 matching lines...) Expand 10 before | Expand all | Expand 10 after
805 } 820 }
806 821
807 DEFINE_TRACE(FrameFetchContext) 822 DEFINE_TRACE(FrameFetchContext)
808 { 823 {
809 visitor->trace(m_document); 824 visitor->trace(m_document);
810 visitor->trace(m_documentLoader); 825 visitor->trace(m_documentLoader);
811 FetchContext::trace(visitor); 826 FetchContext::trace(visitor);
812 } 827 }
813 828
814 } // namespace blink 829 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/loader/FrameFetchContext.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698