diff options
| author | rdevlin.cronin <rdevlin.cronin@chromium.org> | 2016-03-14 17:30:14 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-03-15 00:32:11 +0000 |
| commit | 79a377a9f9a3955e1acbb61640e6ecd11a83581f (patch) | |
| tree | 0bae28f119de1e819e832804ebecb16d53105194 /extensions | |
| parent | ba7fcd6928dbcb974e28fff928e34818ad40bed6 (diff) | |
| download | chromium_src-79a377a9f9a3955e1acbb61640e6ecd11a83581f.zip chromium_src-79a377a9f9a3955e1acbb61640e6ecd11a83581f.tar.gz chromium_src-79a377a9f9a3955e1acbb61640e6ecd11a83581f.tar.bz2 | |
[Extensions] Gather more webRequest render frame look up metrics
BUG=455776
Review URL: https://codereview.chromium.org/1802683002
Cr-Commit-Position: refs/heads/master@{#381133}
Diffstat (limited to 'extensions')
| -rw-r--r-- | extensions/browser/api/web_request/web_request_api.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/extensions/browser/api/web_request/web_request_api.cc b/extensions/browser/api/web_request/web_request_api.cc index aa799bb..2982d27 100644 --- a/extensions/browser/api/web_request/web_request_api.cc +++ b/extensions/browser/api/web_request/web_request_api.cc @@ -1349,8 +1349,12 @@ void ExtensionWebRequestEventRouter::GetMatchingListenersImpl( ExtensionApiFrameIdMap::FrameData frame_data; // TODO(devlin): Figure out when one/both of these can fail, and if we // need to address it. - if (content::ResourceRequestInfo::GetRenderFrameForRequest( - request, &render_process_id, &render_frame_id) && + bool found_render_frame = + content::ResourceRequestInfo::GetRenderFrameForRequest( + request, &render_process_id, &render_frame_id); + UMA_HISTOGRAM_BOOLEAN("Extensions.WebRequestEventFoundFrame", + found_render_frame); + if (found_render_frame && ExtensionApiFrameIdMap::Get()->GetCachedFrameDataOnIO( render_process_id, render_frame_id, &frame_data)) { tab_id = frame_data.tab_id; |