More simplification/refactoring.

1 files changed, 3 insertions, 7 deletions

diff --git a/background_scripts/completion.coffee b/background_scripts/completion.coffee
index 95c50baf..08c33430 100644
--- a/background_scripts/completion.coffee
+++ b/background_scripts/completion.coffee
@@ -205,12 +205,12 @@ class DomainCompleter
       chrome.history.onVisitRemoved.addListener(@onVisitRemoved.bind(this))
       onComplete()
 
-  # We want each key in our domains hash to point to the most recent History entry for that domain.
   onPageVisited: (newPage) ->
     domain = @parseDomain(newPage.url)
     if domain
       @domains[domain] ||= { entry: newPage, referenceCount: 0 }
       slot = @domains[domain]
+      # We want each entry in our domains hash to point to the most recent History entry for that domain.
       slot.entry = newPage if slot.entry.lastVisitTime < newPage.lastVisitTime
       slot.referenceCount += 1
 
@@ -220,12 +220,8 @@ class DomainCompleter
     else
       toRemove.urls.forEach (url) =>
         domain = @parseDomain(url)
-        if domain
-          previousEntry = @domains[domain]
-          if previousEntry
-            previousEntry.referenceCount -= 1
-            if previousEntry.referenceCount == 0
-              delete @domains[domain]
+        if domain and @domains[domain] and ( @domains[domain].referenceCount -= 1 ) == 0
+          delete @domains[domain]
 
   parseDomain: (url) -> url.split("/")[2] || ""