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

Unified Diff: chrome/browser/tab_contents/render_view_context_menu.cc

Issue 7601021: Add UMA for register protocol handler. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix compile error Created 9 years, 4 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
Index: chrome/browser/tab_contents/render_view_context_menu.cc
diff --git a/chrome/browser/tab_contents/render_view_context_menu.cc b/chrome/browser/tab_contents/render_view_context_menu.cc
index d02f69d183ef73e409e3620ebde1947b0faf16fb..cf38b67e0cd3ad50c8986bb79ed735c00531b591 100644
--- a/chrome/browser/tab_contents/render_view_context_menu.cc
+++ b/chrome/browser/tab_contents/render_view_context_menu.cc
@@ -1356,6 +1356,8 @@ void RenderViewContextMenu::ExecuteCommand(int id) {
if (handlers.empty()) {
return;
}
+ UserMetrics::RecordAction(UserMetricsAction(
+ "RegisterProtocolHandler.ContextMenu_Open"));
int handlerIndex = id - IDC_CONTENT_CONTEXT_PROTOCOL_HANDLER_FIRST;
OpenURL(
handlers[handlerIndex].TranslateUrl(params_.link_url),
@@ -1678,6 +1680,8 @@ void RenderViewContextMenu::ExecuteCommand(int id) {
break;
#endif // OS_MACOSX
case IDC_CONTENT_CONTEXT_PROTOCOL_HANDLER_SETTINGS: {
+ UserMetrics::RecordAction(UserMetricsAction(
+ "RegisterProtocolHandler.ContextMenu_Settings"));
std::string url = std::string(chrome::kChromeUISettingsURL) +
chrome::kHandlerSettingsSubPage;
OpenURL(GURL(url), GURL(), 0, NEW_FOREGROUND_TAB, PageTransition::LINK);

Powered by Google App Engine
This is Rietveld 408576698