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

Side by Side Diff: rlz/lib/rlz_enums.h

Issue 2857343004: Add rlz_utils as a new build target to rlz/ (Closed)
Patch Set: Move RLZ_LIB_API to shared rlz_enums.h Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 RLZ_LIB_RLZ_ENUMS_H_ 5 #ifndef RLZ_LIB_RLZ_ENUMS_H_
6 #define RLZ_LIB_RLZ_ENUMS_H_ 6 #define RLZ_LIB_RLZ_ENUMS_H_
7 7
8 #if defined(OS_WIN)
9 #define RLZ_LIB_API __cdecl
10 #else
11 #define RLZ_LIB_API
12 #endif
13
Roger Tawa OOO till Jul 10th 2017/05/05 15:31:29 I see why you did this, but it is sub optimal. Th
Yusuf 2017/05/05 17:14:58 Got it. I had them duplicated, then win_clang bui
8 namespace rlz_lib { 14 namespace rlz_lib {
9 15
10 // An Access Point offers a way to search using Google. 16 // An Access Point offers a way to search using Google.
11 enum AccessPoint { 17 enum AccessPoint {
12 NO_ACCESS_POINT = 0, 18 NO_ACCESS_POINT = 0,
13 19
14 // Access points on Windows PCs. 20 // Access points on Windows PCs.
15 IE_DEFAULT_SEARCH, // The IE7+ chrome search box next to the address bar. 21 IE_DEFAULT_SEARCH, // The IE7+ chrome search box next to the address bar.
16 IE_HOME_PAGE, // Search box on IE 5+ primary home page when Google. 22 IE_HOME_PAGE, // Search box on IE 5+ primary home page when Google.
17 IETB_SEARCH_BOX, // IE Toolbar v4+ search box. 23 IETB_SEARCH_BOX, // IE Toolbar v4+ search box.
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 // Events that note Product and Access Point modifications. 129 // Events that note Product and Access Point modifications.
124 enum Event { 130 enum Event {
125 INVALID_EVENT = 0, 131 INVALID_EVENT = 0,
126 INSTALL = 1, // Access Point added to the system. 132 INSTALL = 1, // Access Point added to the system.
127 SET_TO_GOOGLE, // Point set from non-Google provider to Google. 133 SET_TO_GOOGLE, // Point set from non-Google provider to Google.
128 FIRST_SEARCH, // First search from point since INSTALL 134 FIRST_SEARCH, // First search from point since INSTALL
129 REPORT_RLS, // Report old system "RLS" financial value for this point. 135 REPORT_RLS, // Report old system "RLS" financial value for this point.
130 // New Events should be added here without changing existing enums, 136 // New Events should be added here without changing existing enums,
131 // before LAST_EVENT. 137 // before LAST_EVENT.
132 ACTIVATE, // Product being used for a period of time. 138 ACTIVATE, // Product being used for a period of time.
133 LAST_EVENT 139 LAST_EVENT
Roger Tawa OOO till Jul 10th 2017/05/05 15:31:29 Do you plan on adding new events in a follow up CL
Yusuf 2017/05/05 17:14:58 I may. Not sure on that part since they will be gi
134 }; 140 };
135 141
136 } // namespace rlz_lib 142 } // namespace rlz_lib
137 143
138 #endif // RLZ_LIB_RLZ_ENUMS_H_ 144 #endif // RLZ_LIB_RLZ_ENUMS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698