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

Side by Side Diff: components/arc/arc_util.h

Issue 2877883002: Consolidate two IsArcWindow functions to reduce code duplication (Closed)
Patch Set: review 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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 COMPONENTS_ARC_ARC_UTIL_H_ 5 #ifndef COMPONENTS_ARC_ARC_UTIL_H_
6 #define COMPONENTS_ARC_ARC_UTIL_H_ 6 #define COMPONENTS_ARC_ARC_UTIL_H_
7 7
8 // This file contains utility to see ARC functionality status controlled by 8 // This file contains utility to see ARC functionality status controlled by
9 // outside of ARC, e.g. CommandLine flag, attribute of global data/state, 9 // outside of ARC, e.g. CommandLine flag, attribute of global data/state,
10 // users' preferences, and FeatureList. 10 // users' preferences, and FeatureList.
11 11
12 namespace aura {
13 class Window;
14 } // namespace aura
15
12 namespace base { 16 namespace base {
13 class CommandLine; 17 class CommandLine;
14 } // namespace base 18 } // namespace base
15 19
16 namespace arc { 20 namespace arc {
17 21
18 // Returns true if ARC is installed and the current device is officially 22 // Returns true if ARC is installed and the current device is officially
19 // supported to run ARC. 23 // supported to run ARC.
20 // Note that, to run ARC practically, it is necessary to meet more conditions, 24 // Note that, to run ARC practically, it is necessary to meet more conditions,
21 // e.g., ARC supports only on Primary User Profile. To see if ARC can actually 25 // e.g., ARC supports only on Primary User Profile. To see if ARC can actually
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 bool IsArcKioskMode(); 63 bool IsArcKioskMode();
60 64
61 // Returns true if it is allowed to use ARC with Active Directory managed 65 // Returns true if it is allowed to use ARC with Active Directory managed
62 // devices. 66 // devices.
63 bool IsArcAllowedForActiveDirectoryUsers(); 67 bool IsArcAllowedForActiveDirectoryUsers();
64 68
65 // Checks if opt-in verification was disabled by switch in command line. 69 // Checks if opt-in verification was disabled by switch in command line.
66 // In most cases, it is disabled for testing purpose. 70 // In most cases, it is disabled for testing purpose.
67 bool IsArcOptInVerificationDisabled(); 71 bool IsArcOptInVerificationDisabled();
68 72
73 // Returns true if |window| is associated with an ash::WmWindow whose type is
Luis Héctor Chávez 2017/05/12 20:44:19 nit: s/ash::WmWindow/aura::Window/
Yusuke Sato 2017/05/12 21:06:13 Rephrased the comment. I think the original one wa
74 // ARC_APP. When |window| is nullptr, returns false.
75 bool IsArcAppWindow(aura::Window* window);
76
69 } // namespace arc 77 } // namespace arc
70 78
71 #endif // COMPONENTS_ARC_ARC_UTIL_H_ 79 #endif // COMPONENTS_ARC_ARC_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698