From d3cdf1e8e38bd7c7db5e79f0ab66587d6e2fe8cd Mon Sep 17 00:00:00 2001
From: "reed%reedloden.com"
 <reed%reedloden.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Tue, 13 Nov 2007 06:48:20 +0000
Subject: [PATCH] Bug 392376 - "Double-clicking on the selected application
 within the file handling dialog should work as "OK" and launch the
 application" [p=florian r=Mano a=blocking-firefox3+]

git-svn-id: svn://10.0.0.236/trunk@239277 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/toolkit/mozapps/handling/content/dialog.js  | 11 +++++++++++
 mozilla/toolkit/mozapps/handling/content/dialog.xul |  1 +
 2 files changed, 12 insertions(+)

diff --git a/mozilla/toolkit/mozapps/handling/content/dialog.js b/mozilla/toolkit/mozapps/handling/content/dialog.js
index 2bb1b2d7c1f..de4271a17d4 100644
--- a/mozilla/toolkit/mozapps/handling/content/dialog.js
+++ b/mozilla/toolkit/mozapps/handling/content/dialog.js
@@ -233,6 +233,17 @@ var dialog = {
       document.getElementById("remember-text").removeAttribute("visible");
   },
 
+  /**
+   * Function called when the user double clicks on an item of the list
+   */
+  onDblClick: function onDblClick()
+  {
+    if (this.selectedItem == this._itemChoose)
+      this.chooseApplication();
+    else
+      document.documentElement.acceptDialog();
+  },
+
   /////////////////////////////////////////////////////////////////////////////
   //// Getters / Setters
 
diff --git a/mozilla/toolkit/mozapps/handling/content/dialog.xul b/mozilla/toolkit/mozapps/handling/content/dialog.xul
index 34ccf3b90cb..01122019666 100644
--- a/mozilla/toolkit/mozapps/handling/content/dialog.xul
+++ b/mozilla/toolkit/mozapps/handling/content/dialog.xul
@@ -63,6 +63,7 @@
   <vbox flex="1">
     <description id="item-action-text"/>
     <richlistbox id="items" flex="1"
+                 ondblclick="dialog.onDblClick();"
                  onselect="dialog.updateOKButton();">
       <richlistitem id="item-choose" orient="horizontal" selected="true">
         <label value="&ChooseApp.description;" flex="1"/>