don't reuse the double click handled in activation event handler for toggling the item [backport of r57025 from trunk] (closes #9761)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/branches/WX_2_8_BRANCH@58755 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

docs/changes.txt

@@ -155,6 +155,8 @@ wxMSW:
 - Automatically adjust toolbar's tool size if the provided bitmaps
   don't fit into the default size.
 - Don't generate EVT_LISTBOX_DCLICK events with incorrect indices.
+- Don't reuse the double click handled by activation event handler for toggling
+  the item in wxTreeCtrl.
 
 
 wxMSW/CE:

src/msw/treectrl.cpp

@@ -2933,11 +2933,12 @@ bool wxTreeCtrl::MSWOnNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
     switch ( hdr->code )
     {
         case NM_DBLCLK:
-            // we translate NM_DBLCLK into ACTIVATED event, so don't interpret
+            // we translate NM_DBLCLK into ACTIVATED event and if the user
-            // the return code of this event handler as the return value for
+            // handled the activation of the item we shouldn't proceed with
-            // NM_DBLCLK - otherwise, double clicking the item to toggle its
+            // also using the same double click for toggling the item expanded
-            // expanded status would never work
+            // state -- but OTOH do let the user to expand/collapse the item by
-            *result = false;
+            // double clicking on it if the activation is not handled specially
+            *result = processed;
             break;
 
         case NM_RCLICK: