Cleanup FWL Event and Message code.

xfa/fwl/core/cfwl_event.h

 // Copyright 2016 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #ifndef XFA_FWL_CORE_CFWL_EVENT_H_
 #define XFA_FWL_CORE_CFWL_EVENT_H_
 
 #include "core/fxcrt/fx_coordinates.h"
 #include "core/fxcrt/fx_string.h"
 #include "core/fxcrt/fx_system.h"
 #include "xfa/fwl/core/cfwl_msgkey.h"
 #include "xfa/fwl/core/cfwl_msgmouse.h"
 #include "xfa/fwl/core/fwl_error.h"
 
 enum class CFWL_EventType {
-  None = 0,
-
   CheckStateChanged,

[Tom Sepez, 2016/11/28 18:26:33]

Maybe want None = 0, worry about implicit cast to bools?

[dsinclair, 2016/12/05 20:14:05]

There should be no implicit cast as it's an enum class. At least, that's my
understanding, is that incorrect?

   CheckWord,
   Click,
   Close,
   EditChanged,
-  Key,
-  KillFocus,
   Mouse,
-  MouseWheel,
   PostDropDown,
   PreDropDown,
   Scroll,
   SelectChanged,
-  SetFocus,
-  SizeChanged,
   TextChanged,
   TextFull,
   Validate
 };
 
 class CFX_Graphics;
 class CFWL_Widget;
 
 class CFWL_Event {
  public:
-  CFWL_Event();
+  explicit CFWL_Event(CFWL_EventType type);
   virtual ~CFWL_Event();
 
-  virtual CFWL_EventType GetClassID() const;
+  CFWL_EventType GetClassID() const { return m_type; }
 
   CFWL_Widget* m_pSrcTarget;
   CFWL_Widget* m_pDstTarget;
+
+ private:
+  CFWL_EventType m_type;
 };
 
 #endif  // XFA_FWL_CORE_CFWL_EVENT_H_