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

Side by Side Diff: ash/common/system/chromeos/palette/palette_utils.h

Issue 2303963002: cros/ash: If the partial magnifier is active, do not consume input events if over palette views. (Closed)
Patch Set: Address comments Created 4 years, 3 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 2016 The Chromium Authors. All rights reserved. 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 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 ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_ 5 #ifndef ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_
6 #define ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_ 6 #define ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_
7 7
8 #include "ash/ash_export.h" 8 #include "ash/ash_export.h"
9 9
10 namespace gfx {
11 class Point;
12 }
13
10 namespace ash { 14 namespace ash {
11 15
12 // Returns true if the palette feature is enabled. The palette itself may have 16 // Returns true if the palette feature is enabled. The palette itself may have
13 // been disabled by the user. 17 // been disabled by the user.
14 ASH_EXPORT bool IsPaletteFeatureEnabled(); 18 ASH_EXPORT bool IsPaletteFeatureEnabled();
15 19
16 // Are experimental palette features enabled? 20 // Are experimental palette features enabled?
17 ASH_EXPORT bool ArePaletteExperimentalFeaturesEnabled(); 21 ASH_EXPORT bool ArePaletteExperimentalFeaturesEnabled();
18 22
23 // Returns true if either the palette icon or the palette widget contain the
24 // given point (in screen space).
25 ASH_EXPORT bool PaletteContainsPointInScreen(const gfx::Point& point);
James Cook 2016/09/06 16:16:32 optional nit: Do you anticipate using this functio
jdufault 2016/09/07 22:39:06 Yep, it will be used by the laser pointer.
26
19 } // namespace ash 27 } // namespace ash
20 28
21 #endif // ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_ 29 #endif // ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_PALETTE_UTILS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698