OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2016 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef ASH_SHARED_APP_TYPES_H_ | |
6 #define ASH_SHARED_APP_TYPES_H_ | |
7 | |
8 namespace ash { | |
9 | |
10 // App type of the window. This enum is used to back | |
11 // an UMA histogram and should be treated as append-only. | |
12 enum class AppType { | |
13 OTHERS = 0, | |
14 BROWSER, | |
15 CHROME_APP, | |
16 ARC_APP, | |
17 DEFAULT_NOTE_TAKING_APP, | |
sky
2016/09/27 22:08:59
Why the DEFAULT_ here instead of NOTE_TAKING_APP?
Daniel Erat
2016/09/27 23:34:09
i'm fine with dropping the DEFAULT_ here. (as back
xiaoyinh(OOO Sep 11-29)
2016/09/29 20:37:43
Thanks for the explanation.
| |
18 APP_COUNT, | |
sky
2016/09/27 22:08:59
I would prefer you define a constant outside the e
xiaoyinh(OOO Sep 11-29)
2016/09/29 20:37:43
Done.
| |
19 }; | |
20 | |
21 } // namespace ash | |
22 | |
23 #endif // ASH_SHARED_APP_TYPES_H_ | |
OLD | NEW |