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

Unified Diff: chrome/browser/ui/webui/ntp/new_tab_page_handler.h

Issue 8086002: Add metrics to help measure effects of installing default apps in a profile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Change events to histograms Created 9 years, 2 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/ui/webui/ntp/new_tab_page_handler.h
===================================================================
--- chrome/browser/ui/webui/ntp/new_tab_page_handler.h (revision 105055)
+++ chrome/browser/ui/webui/ntp/new_tab_page_handler.h (working copy)
@@ -19,6 +19,7 @@
virtual ~NewTabPageHandler() {}
// WebUIMessageHandler implementation.
+ virtual WebUIMessageHandler* Attach(WebUI* web_ui) OVERRIDE;
virtual void RegisterMessages() OVERRIDE;
// Callback for "closeNotificationPromo".
@@ -53,10 +54,11 @@
// group, and the rest of the bits are used for the page group ID (defined
// here).
enum {
- INDEX_MASK = (1 << 10) - 1,
- MOST_VISITED_PAGE_ID = 1 << 10,
- APPS_PAGE_ID = 2 << 10,
- BOOKMARKS_PAGE_ID = 3 << 10,
+ PAGE_ID_OFFSET = 10,
Evan Stade 2011/10/13 00:07:51 do not add this as an enum value, it doesn't make
Roger Tawa OOO till Jul 10th 2011/10/13 00:39:43 Done.
+ INDEX_MASK = (1 << PAGE_ID_OFFSET) - 1,
+ MOST_VISITED_PAGE_ID = 1 << PAGE_ID_OFFSET,
+ APPS_PAGE_ID = 2 << PAGE_ID_OFFSET,
+ BOOKMARKS_PAGE_ID = 3 << PAGE_ID_OFFSET,
Evan Stade 2011/10/13 00:07:51 LAST_PAGE_ID = BOOKMARKS_PAGE_ID
Roger Tawa OOO till Jul 10th 2011/10/13 00:39:43 Done.
jar (doing other things) 2011/10/13 22:16:53 This looks like you're going for a histogram with
jar (doing other things) 2011/10/13 22:22:07 I retract my comment. Sorry about that.... the hi
};
// Helper to send out promo resource change notification.

Powered by Google App Engine
This is Rietveld 408576698