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

Side by Side Diff: chrome/browser/ui/webui/ntp/ntp_user_data_logger.h

Issue 2435683003: Cleanup desktop NTP metrics recording, part 2 (Closed)
Patch Set: review Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_
6 #define CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_ 6 #define CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <bitset> 10 #include <bitset>
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 // only the impressions for the first source will be logged, leaving the 77 // only the impressions for the first source will be logged, leaving the
78 // number of impressions for a source slightly out-of-sync with navigations. 78 // number of impressions for a source slightly out-of-sync with navigations.
79 std::bitset<kNumMostVisited> impression_was_logged_; 79 std::bitset<kNumMostVisited> impression_was_logged_;
80 80
81 // True if at least one iframe came from a server-side suggestion. 81 // True if at least one iframe came from a server-side suggestion.
82 bool has_server_side_suggestions_; 82 bool has_server_side_suggestions_;
83 83
84 // True if at least one iframe came from a client-side suggestion. 84 // True if at least one iframe came from a client-side suggestion.
85 bool has_client_side_suggestions_; 85 bool has_client_side_suggestions_;
86 86
87 // Total number of tiles rendered, no matter if it's a thumbnail, a gray tile
88 // or an external tile.
89 size_t number_of_tiles_;
90
91 // Whether we have already emitted NTP stats for this web contents. 87 // Whether we have already emitted NTP stats for this web contents.
92 bool has_emitted_; 88 bool has_emitted_;
93 89
94 // Are stats being logged during Chrome startup? 90 // Are stats being logged during Chrome startup?
95 bool during_startup_; 91 bool during_startup_;
96 92
97 // The URL of this New Tab Page - varies based on NTP version. 93 // The URL of this New Tab Page - varies based on NTP version.
98 GURL ntp_url_; 94 GURL ntp_url_;
99 95
100 DISALLOW_COPY_AND_ASSIGN(NTPUserDataLogger); 96 DISALLOW_COPY_AND_ASSIGN(NTPUserDataLogger);
101 }; 97 };
102 98
103 #endif // CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_ 99 #endif // CHROME_BROWSER_UI_WEBUI_NTP_NTP_USER_DATA_LOGGER_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/search/search_ipc_router_unittest.cc ('k') | chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698