From 7260d4e2be65df5af2b7eed74fca09523153e7aa Mon Sep 17 00:00:00 2001
From: "ben%netscape.com"
 <ben%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 20 Feb 2002 20:55:28 +0000
Subject: [PATCH] blocker 126704 - can't load history items. removing some
 unnecessary code.

git-svn-id: svn://10.0.0.236/trunk@115112 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../xpfe/components/history/resources/history-panel.xul  | 6 +++---
 mozilla/xpfe/components/history/resources/history.js     | 9 ++-------
 mozilla/xpfe/components/history/resources/history.xul    | 3 +--
 .../components/history/resources/historyTreeOverlay.xul  | 3 ++-
 4 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/mozilla/xpfe/components/history/resources/history-panel.xul b/mozilla/xpfe/components/history/resources/history-panel.xul
index 81a3cb9bc97..b430a980a09 100644
--- a/mozilla/xpfe/components/history/resources/history-panel.xul
+++ b/mozilla/xpfe/components/history/resources/history-panel.xul
@@ -22,9 +22,10 @@
 -->
 
 <?xml-stylesheet href="chrome://communicator/skin/" type="text/css"?>
+<?xml-stylesheet href="chrome://communicator/skin/sidebar/sidebarListView.css" type="text/css"?>
+
 <?xul-overlay href="chrome://communicator/content/history/historyTreeOverlay.xul"?>
 <?xul-overlay href="chrome://communicator/content/utilityOverlay.xul"?>
-<?xml-stylesheet href="chrome://communicator/skin/sidebar/sidebarListView.css" type="text/css"?>
 
 <!DOCTYPE window SYSTEM "chrome://communicator/locale/history/history.dtd" >
 
@@ -41,8 +42,7 @@
   </keyset>
   <popupset id="historyContextMenu"/>
   <!-- use deep merging to hide certain columns by default -->
-  <outliner id="historyOutliner" clickcount="1"
-            onclick="if (event.button == 0) OpenURL(event.ctrlKey || event.metaKey);">
+  <outliner id="historyOutliner">
     <outlinercols id="historyOutlinerCols">
       <outlinercol id="Name"/>
       <splitter id="pre-URL-splitter"/>
diff --git a/mozilla/xpfe/components/history/resources/history.js b/mozilla/xpfe/components/history/resources/history.js
index c5f73904845..9e45db9ebd1 100644
--- a/mozilla/xpfe/components/history/resources/history.js
+++ b/mozilla/xpfe/components/history/resources/history.js
@@ -254,13 +254,8 @@ function collapseExpand()
 
 function OpenURL(aInNewWindow)
 {
-    var currentIndex = gHistoryOutliner.currentIndex;
-    if (isContainer(gHistoryOutliner, currentIndex)) {
-        if (gHistoryOutliner.getAttribute("clickcount") == "1")
-            gHistoryOutliner.outlinerBoxObject.view.toggleOpenState(currentIndex);
-        return false;
-    }
-      
+    var currentIndex = gHistoryOutliner.currentIndex;     
+    var builder = gHistoryOutliner.builder.QueryInterface(Components.interfaces.nsIXULOutlinerBuilder);
     var url = builder.getResourceAtIndex(currentIndex).Value;
 
     if (aInNewWindow) {
diff --git a/mozilla/xpfe/components/history/resources/history.xul b/mozilla/xpfe/components/history/resources/history.xul
index f8866798cf1..1686d2f9387 100644
--- a/mozilla/xpfe/components/history/resources/history.xul
+++ b/mozilla/xpfe/components/history/resources/history.xul
@@ -138,8 +138,7 @@
 </menubar>
 </toolbox>
 
-  <outliner id="historyOutliner"
-            ondblclick="if (event.button == 0) OpenURL(event.ctrlKey || event.metaKey);"/>
+  <outliner id="historyOutliner"/>
   <statusbar id="status-bar" class="chromeclass-status">
     <statusbarpanel id="statusbar-display" flex="1"/>
     <statusbarpanel class="statusbarpanel-iconic" id="offline-status"/>
diff --git a/mozilla/xpfe/components/history/resources/historyTreeOverlay.xul b/mozilla/xpfe/components/history/resources/historyTreeOverlay.xul
index 1ee1a929f04..00836986cdb 100644
--- a/mozilla/xpfe/components/history/resources/historyTreeOverlay.xul
+++ b/mozilla/xpfe/components/history/resources/historyTreeOverlay.xul
@@ -78,7 +78,8 @@
             onkeypress="if (event.keyCode == 13) OpenURL(event.ctrlKey || event.metaKey);" 
             onselect="this.outlinerBoxObject.view.selectionChanged();
                       historyOnSelect();"
-            ondraggesture="if (event.originalTarget.localName == 'outlinerchildren') nsDragAndDrop.startDrag(event, historyDNDObserver);">
+            ondraggesture="if (event.originalTarget.localName == 'outlinerchildren') nsDragAndDrop.startDrag(event, historyDNDObserver);"
+            ondblclick="if (validClickConditions(event)) OpenURL(event.ctrlKey || event.metaKey);">
     <template>
       <rule>
         <outlinerchildren>