Use "= default" to denote default constructor or destructor
[WebKit-https.git] / Source / WebCore / platform / ContextMenuItem.cpp
index 0c2a6ea..2c501d0 100644 (file)
@@ -28,7 +28,7 @@
 
 #include "ContextMenu.h"
 
-#if USE(CROSS_PLATFORM_CONTEXT_MENUS)
+#if ENABLE(CONTEXT_MENUS)
 
 namespace WebCore {
 
@@ -62,8 +62,20 @@ ContextMenuItem::ContextMenuItem(ContextMenuAction action, const String& title,
 {
 }
 
-ContextMenuItem::~ContextMenuItem()
+ContextMenuItem::ContextMenuItem()
+    : m_type(SeparatorType)
+    , m_action(ContextMenuItemTagNoAction)
+    , m_enabled(false)
+    , m_checked(false)
+{
+}
+
+ContextMenuItem::~ContextMenuItem() = default;
+
+bool ContextMenuItem::isNull() const
 {
+    // FIXME: This is a bit of a hack. Cross-platform ContextMenuItem users need a concrete way to track "isNull".
+    return m_action == ContextMenuItemTagNoAction && m_title.isNull() && m_subMenuItems.isEmpty();
 }
 
 void ContextMenuItem::setSubMenu(ContextMenu* subMenu)
@@ -119,4 +131,4 @@ bool ContextMenuItem::enabled() const
 
 } // namespace WebCore
 
-#endif // USE(CROSS_PLATFORM_CONTEXT_MENUS)
+#endif // ENABLE(CONTEXT_MENUS)