Bug 400543 - Add TRANSITION_DOWNLOAD. r=mano, a=schrep

toolkit/components/places/public/nsINavHistoryService.idl

@@ -1199,6 +1199,11 @@ interface nsINavHistoryService : nsISupports
    */
   const unsigned long TRANSITION_REDIRECT_TEMPORARY = 6;
 
+  /**
+   * Set when the transition is a download.
+   */
+  const unsigned long TRANSITION_DOWNLOAD = 7;
+
   /**
    * True if there is any history. This can be used in UI to determine whether
    * the "clear history" button should be enabled or not. This is much better

toolkit/components/places/src/nsNavHistory.cpp

@@ -1960,9 +1960,10 @@ nsNavHistory::AddVisit(nsIURI* aURI, PRTime aTime, PRInt64 aReferringVisit,
     mDBGetPageVisitStats->Reset();
     scoper.Abandon();
 
-    // hide embedded links and redirects, everything else is visible,
+    // Hide only embedded links, redirects, and downloads
     // See the hidden computation code above for a little more explanation.
-    hidden = (aTransitionType == TRANSITION_EMBED || aIsRedirect);
+    hidden = (aTransitionType == TRANSITION_EMBED || aIsRedirect ||
+              aTransitionType == TRANSITION_DOWNLOAD);
 
     typed = (aTransitionType == TRANSITION_TYPED);