diff options
| author | mmenke <mmenke@chromium.org> | 2015-02-27 14:01:27 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-27 22:02:20 +0000 |
| commit | 0156831a6f278576422bbd6e87105d506e5f5224 (patch) | |
| tree | 887f040f9ca018bf7ba7da2dfcde3cdbe17ada6f /content/browser/loader/resource_dispatcher_host_impl.cc | |
| parent | ff66006e62449071ab87ad01b025dac4a9029d06 (diff) | |
| download | chromium_src-0156831a6f278576422bbd6e87105d506e5f5224.zip chromium_src-0156831a6f278576422bbd6e87105d506e5f5224.tar.gz chromium_src-0156831a6f278576422bbd6e87105d506e5f5224.tar.bz2 | |
RDH: More instrumentation for jank in GetLoadInfoForAllRoutes.
Getting and updating the load state of requests is janky.
TBR=eroman@chromium.org
BUG=455952
Review URL: https://codereview.chromium.org/966813002
Cr-Commit-Position: refs/heads/master@{#318525}
Diffstat (limited to 'content/browser/loader/resource_dispatcher_host_impl.cc')
| -rw-r--r-- | content/browser/loader/resource_dispatcher_host_impl.cc | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/content/browser/loader/resource_dispatcher_host_impl.cc b/content/browser/loader/resource_dispatcher_host_impl.cc index 6c4da44..a80d6f3 100644 --- a/content/browser/loader/resource_dispatcher_host_impl.cc +++ b/content/browser/loader/resource_dispatcher_host_impl.cc @@ -2184,6 +2184,11 @@ ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes() { "455952 ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes2")); upload_progress = request->GetUploadProgress(); } + + tracked_objects::ScopedTracker tracking_profile4( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "455952 ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes4")); + LoadInfo load_info; load_info.url = request->url(); { @@ -2197,8 +2202,17 @@ ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes() { load_info.upload_size = upload_progress.size(); load_info.upload_position = upload_progress.position(); + tracked_objects::ScopedTracker tracking_profile5( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "455952 ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes5")); + GlobalRoutingID id(loader.second->GetRequestInfo()->GetGlobalRoutingID()); LoadInfoMap::iterator existing = info_map->find(id); + + tracked_objects::ScopedTracker tracking_profile6( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "455952 ResourceDispatcherHostImpl::GetLoadInfoForAllRoutes6")); + if (existing == info_map->end() || LoadInfoIsMoreInteresting(load_info, existing->second)) { (*info_map)[id] = load_info; |