From 3ae083cc6f1ef84082e63d086df06c80c234cdc8 Mon Sep 17 00:00:00 2001
From: "dietrich%mozilla.com" <dietrich%mozilla.com>
Date: Tue, 18 Jul 2006 16:56:22 +0000
Subject: [PATCH] Bug 258107 Location bar should sort based on order visited
 (r=mconnor)

---
 .../history/src/nsGlobalHistory.cpp           | 40 ++++++++++++++++++-
 .../components/history/src/nsGlobalHistory.h  |  2 +
 2 files changed, 41 insertions(+), 1 deletion(-)

diff --git a/toolkit/components/history/src/nsGlobalHistory.cpp b/toolkit/components/history/src/nsGlobalHistory.cpp
index 01d102d3690..f1a2f513b23 100644
--- a/toolkit/components/history/src/nsGlobalHistory.cpp
+++ b/toolkit/components/history/src/nsGlobalHistory.cpp
@@ -4329,6 +4329,9 @@ nsGlobalHistory::AutoCompleteTypedSearch(nsIAutoCompleteMdbResult2 **aResult)
   result->SetTokens(kToken_URLColumn, nsIAutoCompleteMdbResult2::kCharType, kToken_NameColumn, nsIAutoCompleteMdbResult2::kUnicharType);
   result->SetReverseByteOrder(mReverseByteOrder);
 
+  // Store hits in an nsIArray initially
+  nsCOMArray<nsIMdbRow> resultArray;
+
   nsCOMPtr<nsIMdbRow> row;
   mdb_pos pos;
   do {
@@ -4336,10 +4339,21 @@ nsGlobalHistory::AutoCompleteTypedSearch(nsIAutoCompleteMdbResult2 **aResult)
     if (!row) break;
     
     if (HasCell(mEnv, row, kToken_TypedColumn)) {
-      result->AddRow(row);
+      resultArray.AppendObject(row);
     }
   } while (row);
 
+  // sort it
+  AutoCompleteSortClosure closure;
+  closure.history = this;
+  resultArray.Sort(TypedSortComparison, NS_STATIC_CAST(void*, &closure));
+
+  // place the sorted array into the autocomplete results
+  count = resultArray.Count();
+  for (PRUint32 i = 0; i < count; ++i) {
+    result->AddRow(resultArray[i]);
+  }
+
   // Determine the result of the search
   PRUint32 matchCount;
   rv = result->GetMatchCount(&matchCount);
@@ -4572,6 +4586,30 @@ nsGlobalHistory::AutoCompleteCompare(nsAString& aHistoryURL,
   return Substring(aHistoryURL, 0, aUserURL.Length()).Equals(aUserURL);
 }
 
+int PR_CALLBACK 
+nsGlobalHistory::TypedSortComparison(nsIMdbRow *row1, nsIMdbRow *row2, 
+                                            void *closureVoid) 
+{
+  // cast our function parameters back into their real form
+  AutoCompleteSortClosure* closure = 
+      NS_STATIC_CAST(AutoCompleteSortClosure*, closureVoid);
+
+  PRInt64 lastVisitedTime1, lastVisitedTime2, diff;
+  closure->history->GetRowValue(row1,
+                                closure->history->kToken_LastVisitDateColumn,
+               &lastVisitedTime1);
+  closure->history->GetRowValue(row2,
+                                closure->history->kToken_LastVisitDateColumn,
+               &lastVisitedTime2);
+
+  LL_SUB(diff, lastVisitedTime2, lastVisitedTime1);
+  if(LL_CMP(diff, >, 0))
+    return 1;
+  else  if(LL_CMP(diff, <, 0))
+    return -1;
+  return 0;
+}
+
 int PR_CALLBACK 
 nsGlobalHistory::AutoCompleteSortComparison(nsIMdbRow *row1, nsIMdbRow *row2, 
                                             void *closureVoid) 
diff --git a/toolkit/components/history/src/nsGlobalHistory.h b/toolkit/components/history/src/nsGlobalHistory.h
index bf41ebcf6d6..fd3f679f1ee 100644
--- a/toolkit/components/history/src/nsGlobalHistory.h
+++ b/toolkit/components/history/src/nsGlobalHistory.h
@@ -228,6 +228,8 @@ protected:
                              AutocompleteExclude* aExclude);
   PR_STATIC_CALLBACK(int)
   AutoCompleteSortComparison(nsIMdbRow *row1, nsIMdbRow *row2, void *closureVoid);
+  PR_STATIC_CALLBACK(int)
+  TypedSortComparison(nsIMdbRow *row1, nsIMdbRow *row2, void *closureVoid);
 
   // AutoCompleteSortClosure - used to pass info into 
   // AutoCompleteSortComparison from the NS_QuickSort() function