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

Unified Diff: xfa/fwl/core/fwl_noteimp.h

Issue 1938163002: More define cleanup. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: xfa/fwl/core/fwl_noteimp.h
diff --git a/xfa/fwl/core/fwl_noteimp.h b/xfa/fwl/core/fwl_noteimp.h
index 8607a334f11a5190e44dd989bc5c80e811993541..575c99f0c059b09236d8a226dcf192b1362324ea 100644
--- a/xfa/fwl/core/fwl_noteimp.h
+++ b/xfa/fwl/core/fwl_noteimp.h
@@ -13,13 +13,15 @@
#include "xfa/fwl/core/ifwl_widget.h"
#include "xfa/fxgraphics/include/cfx_graphics.h"
-#define FWL_KEYFLAG_Ctrl (1 << 0)
-#define FWL_KEYFLAG_Alt (1 << 1)
-#define FWL_KEYFLAG_Shift (1 << 2)
-#define FWL_KEYFLAG_Command (1 << 3)
-#define FWL_KEYFLAG_LButton (1 << 4)
-#define FWL_KEYFLAG_RButton (1 << 5)
-#define FWL_KEYFLAG_MButton (1 << 6)
+enum FWL_KeyFlag {
+ FWL_KEYFLAG_Ctrl = (1 << 0),
Tom Sepez 2016/05/02 21:45:39 nit: ()
dsinclair 2016/05/03 13:58:16 Done.
+ FWL_KEYFLAG_Alt = (1 << 1),
+ FWL_KEYFLAG_Shift = (1 << 2),
+ FWL_KEYFLAG_Command = (1 << 3),
+ FWL_KEYFLAG_LButton = (1 << 4),
+ FWL_KEYFLAG_RButton = (1 << 5),
+ FWL_KEYFLAG_MButton = (1 << 6)
+};
class CFWL_CoreToolTipDP;
class CFWL_MsgActivate;

Powered by Google App Engine
This is Rietveld 408576698