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

Unified Diff: chrome/browser/ui/browser.cc

Issue 10347002: Add a simple UMA for counting web intents dispatches. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index e08b21057b6c710083abdfeb07e9019f5d0dcaae..929f3bae317b919dbf24e6471880a29beda7ab29 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -4001,6 +4001,8 @@ void Browser::WebIntentDispatch(
if (!web_intents::IsWebIntentsEnabled())
return;
+ UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch", 1);
James Hawkins 2012/05/02 20:03:00 Are you familiar with all the other requirements y
Greg Billock 2012/05/02 20:11:54 No. I looked for details, but didn't find them, so
+
TabContentsWrapper* tcw =
TabContentsWrapper::GetCurrentWrapperForContents(tab);
tcw->web_intent_picker_controller()->SetIntentsDispatcher(intents_dispatcher);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698