Fixing bug 43628. r=hyatt,pavlov.

content/events/src/nsDOMEvent.cpp

@@ -346,6 +346,7 @@ nsDOMEvent::GetDetail(PRInt32* aDetail)
       default:
         break;
       }
+      return NS_OK;
     }
 
     default:

layout/events/src/nsDOMEvent.cpp

@@ -346,6 +346,7 @@ nsDOMEvent::GetDetail(PRInt32* aDetail)
       default:
         break;
       }
+      return NS_OK;
     }
 
     default:

xpfe/components/filepicker/res/content/filepicker.js

@@ -239,6 +239,7 @@ function createTree(parentElement, dirArray)
     /* treeRow */
     var treeRow = document.createElement("treerow");
     treeRow.setAttribute("path", file.path);
+    treeRow.setAttribute("onclick", "onClick(event)");
 
     /* treeCell -- name */
     var treeCell = document.createElement("treecell");

xpfe/components/filepicker/res/content/filepicker.xul

@@ -24,7 +24,7 @@
 </box>
 
 <box align="horizontal" flex="1">
-  <tree id="directoryList" onclick="onClick(event);" flex="1">
+  <tree id="directoryList" flex="1">
     <treecolgroup>
       <treecol flex="1"/>
       <treecol flex="1"/>