summaryrefslogtreecommitdiffstats
path: root/chrome/browser/views/bookmark_menu_button.cc
diff options
context:
space:
mode:
authorsky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-03-12 20:31:13 +0000
committersky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-03-12 20:31:13 +0000
commit2719e7e6d7021754ea63cdf3565d0276908a94cb (patch)
tree19b1e24ad2e44d1cf8832dccc0fbc47f026ed75d /chrome/browser/views/bookmark_menu_button.cc
parent179484d80ce158dabeaebe11fcb4dd6ce5280487 (diff)
downloadchromium_src-2719e7e6d7021754ea63cdf3565d0276908a94cb.zip
chromium_src-2719e7e6d7021754ea63cdf3565d0276908a94cb.tar.gz
chromium_src-2719e7e6d7021754ea63cdf3565d0276908a94cb.tar.bz2
Lands http://codereview.chromium.org/669132 for Thiago:
windows: Add UMA metrics for each time that the bookmarks toolbar menu button it's clicked. Since it isn't fully supported we need this to know how many people are using it. BUG=36191 TEST=none Review URL: http://codereview.chromium.org/885005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41466 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/bookmark_menu_button.cc')
-rw-r--r--chrome/browser/views/bookmark_menu_button.cc8
1 files changed, 6 insertions, 2 deletions
diff --git a/chrome/browser/views/bookmark_menu_button.cc b/chrome/browser/views/bookmark_menu_button.cc
index 02eae58..fc4fd85 100644
--- a/chrome/browser/views/bookmark_menu_button.cc
+++ b/chrome/browser/views/bookmark_menu_button.cc
@@ -17,6 +17,7 @@
#include "chrome/browser/bookmarks/bookmark_utils.h"
#include "chrome/browser/browser.h"
#include "chrome/browser/browser_theme_provider.h"
+#include "chrome/browser/metrics/user_metrics.h"
#include "chrome/browser/profile.h"
#include "chrome/browser/tab_contents/tab_contents.h"
#include "chrome/browser/view_ids.h"
@@ -28,12 +29,12 @@ BookmarkMenuButton::BookmarkMenuButton(Browser* browser)
browser_(browser),
bookmark_drop_menu_(NULL),
drop_operation_(0) {
+ // TODO(sky): if we keep this code, we need real icons, a11y support, and a
+ // tooltip.
set_menu_delegate(this);
SetID(VIEW_ID_BOOKMARK_MENU);
ThemeProvider* tp = browser_->profile()->GetThemeProvider();
- // TODO(sky): if we keep this code, we need real icons, a11y support, and a
- // tooltip.
SetIcon(*tp->GetBitmapNamed(IDR_MENU_BOOKMARK));
}
@@ -127,6 +128,9 @@ void BookmarkMenuButton::RunMenu(views::View* source,
gfx::NativeWindow hwnd,
bool for_drop) {
Profile* profile = browser_->profile();
+
+ UserMetrics::RecordAction("BookmarkMenu_clicked", profile);
+
BookmarkMenuController* menu = new BookmarkMenuController(
browser_, profile, browser_->GetSelectedTabContents(), hwnd,
GetBookmarkModel()->GetBookmarkBarNode(), 0, true);