Cleanup FWL Event and Message code.

xfa/fwl/core/cfwl_event.h

Index: xfa/fwl/core/cfwl_event.h
diff --git a/xfa/fwl/core/cfwl_event.h b/xfa/fwl/core/cfwl_event.h
index fe8e1b37f0eaf1d50857734d995a3798329b1974..f1cc4addf4879fbfca03060ab52b1975b457895e 100644
--- a/xfa/fwl/core/cfwl_event.h
+++ b/xfa/fwl/core/cfwl_event.h
@@ -15,23 +15,16 @@
 #include "xfa/fwl/core/fwl_error.h"
 
 enum class CFWL_EventType {

[Tom Sepez, 2016/12/07 18:48:26]

nit: If I were really compulsive, I'd nest this as CFWL_Event::Type.

[dsinclair, 2016/12/07 20:02:01]

Done.

-  None = 0,
-
   CheckStateChanged,
   CheckWord,
   Click,
   Close,
   EditChanged,
-  Key,
-  KillFocus,
   Mouse,
-  MouseWheel,
   PostDropDown,
   PreDropDown,
   Scroll,
   SelectChanged,
-  SetFocus,
-  SizeChanged,
   TextChanged,
   TextFull,
   Validate
@@ -42,13 +35,16 @@ class CFWL_Widget;
 
 class CFWL_Event {
  public:
-  CFWL_Event();
+  explicit CFWL_Event(CFWL_EventType type);

[Tom Sepez, 2016/12/07 18:48:26]

nit: let's add two more ctors:

CFWL_Event(CFWL_EventType type, CFWL_Widget* pSrcTarget);
CFWL_Event(CFWL_EventType type, CFWL_Widget* pSrcTarget, CFWL_Widget*
pDstTarget);

since we nearly always set one or both these immediately after making one.

[dsinclair, 2016/12/07 20:02:01]

Done.

   virtual ~CFWL_Event();
 
-  virtual CFWL_EventType GetClassID() const;
+  CFWL_EventType GetClassID() const { return m_type; }

[Tom Sepez, 2016/12/07 18:48:26]

nit: bad name, maybe GetEventType() ?

[dsinclair, 2016/12/07 20:02:01]

Went with GetType()

 
   CFWL_Widget* m_pSrcTarget;
   CFWL_Widget* m_pDstTarget;
+
+ private:
+  CFWL_EventType m_type;
 };
 
 #endif  // XFA_FWL_CORE_CFWL_EVENT_H_