Don't delegate context menu to Selection Tool.

Our superclass can handle it more correctly (including the Cancel
item at the top).

pcbnew/tools/footprint_editor_tools.cpp

@@ -252,10 +252,9 @@ int MODULE_EDITOR_TOOLS::EnumeratePads( const TOOL_EVENT& aEvent )
             break;
         }
 
-        else
+        else if( evt->IsClick( BUT_RIGHT ) )
         {
-            // Delegate BUT_RIGHT, etc. to SELECTION_TOOL
-            m_toolMgr->PassEvent();
+            m_menu.ShowContextMenu();
         }
 
         // Prepare the next loop by updating the old cursor mouse position