diff options
| author | erikchen <erikchen@chromium.org> | 2015-03-11 18:10:46 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-12 01:11:21 +0000 |
| commit | 22f04ffa54f12fdb75e90202def5199759054eab (patch) | |
| tree | 30fada2ff6d7e580c20e9f8dc07c77060217d3bc /components/webdata | |
| parent | 51e4d20a8699a245daa4a6548834751aa1339aff (diff) | |
| download | chromium_src-22f04ffa54f12fdb75e90202def5199759054eab.zip chromium_src-22f04ffa54f12fdb75e90202def5199759054eab.tar.gz chromium_src-22f04ffa54f12fdb75e90202def5199759054eab.tar.bz2 | |
Add profiling to investigate jank in WebDataRequestManager::RequestCompletedOnThread.
BUG=466312
TBR=atwilson@chromium.org
Review URL: https://codereview.chromium.org/1002543003
Cr-Commit-Position: refs/heads/master@{#320197}
Diffstat (limited to 'components/webdata')
| -rw-r--r-- | components/webdata/common/web_data_request_manager.cc | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/components/webdata/common/web_data_request_manager.cc b/components/webdata/common/web_data_request_manager.cc index 9aba721..103f10d 100644 --- a/components/webdata/common/web_data_request_manager.cc +++ b/components/webdata/common/web_data_request_manager.cc @@ -6,6 +6,7 @@ #include "base/bind.h" #include "base/message_loop/message_loop.h" +#include "base/profiler/scoped_tracker.h" #include "base/stl_util.h" //////////////////////////////////////////////////////////////////////////////// @@ -120,6 +121,11 @@ void WebDataRequestManager::RequestCompletedOnThread( scoped_ptr<WebDataRequest> request) { if (request->IsCancelled()) return; + // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466312 + // is fixed. + tracked_objects::ScopedTracker tracking_profile1( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "466312 WebDataRequestManager::RequestCompletedOnThread::UpdateMap")); { base::AutoLock l(pending_lock_); RequestMap::iterator i = pending_requests_.find(request->GetHandle()); @@ -132,6 +138,12 @@ void WebDataRequestManager::RequestCompletedOnThread( pending_requests_.erase(i); } + // TODO(erikchen): Remove ScopedTracker below once http://crbug.com/466312 + // is fixed. + tracked_objects::ScopedTracker tracking_profile2( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "466312 " + "WebDataRequestManager::RequestCompletedOnThread::NotifyConsumer")); // Notify the consumer if needed. if (!request->IsCancelled()) { WebDataServiceConsumer* consumer = request->GetConsumer(); |