Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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_ |
| OLD | NEW |