Closes #63 adding history items in real time. This doesn't update tho…

…se items with the favicon or meta data since its not ready at that time. We would need to add a page listener for when the meta data is retrieved to get that.

data/js/history.js

@@ -298,11 +298,14 @@ var Application = Backbone.View.extend({
 
     // this will help get single history additions
     addon.on("history:add", function(item) {
+      var hi;
       if (item) {
-        console.log("ITEM", item);
-        hl.add(new HistoryItem(item));
-      } else {
-        console.log("NOT ITEM", item);
+        hi = hl.findWhere({ url : item.url });
+        if (!hi) {
+          hl.add(new HistoryItem(item));
+        } else {
+          hi.set(item);
+        }
       }
     });
 

lib/history/service.js

@@ -24,7 +24,11 @@ const nsIHistory = Class({
     Ci.nsINavHistoryObserver
   ]),
   onVisit: function (aURI, aVisitID, aTime, aSessionID, aReferringID, aTransitionType) {
-    emit(this, "visit", { url : aURI.spec, time : Math.floor(aTime / 1000), title : history.getPageTitle(aURI) });
+    // ignore redirects
+    if (aTransitionType === history.TRANSITION_REDIRECT_TEMPORARY || history.TRANSITION_REDIRECT_PERMANENT) {
+      return;
+    }
+    emit(this, "visit", { url : aURI.spec, time : Math.floor(aTime / 1000), title : history.getPageTitle(aURI) || "" });
   },
   onBeginUpdateBatch: function () {},
   onEndUpdateBatch: function () {},

lib/page/setup.js

@@ -36,6 +36,15 @@ on(History, 'delete', data => {
   page.emit('history:removed', data);
 });
 
+on(History, 'visit', data => {
+  page.emit('history:add', data);
+});
+
+on(History, 'title:changed', data => {
+  // history will check for dupes
+  page.emit('history:add', data);
+});
+
 page.on("history:events:delete", ({ message: url }) => {
   // remove actual history
   remove(url);