Issue #430 - Filter hit counters aren't cleared

lib/filterStorage.js

Index: lib/filterStorage.js
===================================================================
--- a/lib/filterStorage.js
+++ b/lib/filterStorage.js
@@ -355,6 +355,8 @@
       filter.hitCount = 0;
       filter.lastHit = 0;
     }
+    let {FilterListener} = require("filterListener");

[saroyanm, 2014/06/09 11:28:02]

Called FilterListener module inside of the function for exports.FilterListener
to be already set.

[Wladimir Palant, 2014/06/12 11:58:38]

Actually, hardcoding this logic here doesn't seem too great. FilterStorage is a
generic component, it makes sense that it never uses FilterListener explicitly.
In fact, this logic should be implementable in FilterListener itself, in
onFilterChange:

  if (action == "hitCount" && newValue == 0)
  {
    // Filter hits are being reset, make sure these changes are saved.
    FilterListener.setDirty(0);
  }
  else if (action == "hitCount" || action == "lastHit")
    ...

This will allow some cleanups:

* FilterListener object should no longer be exported, it is an implementation
detail.
* ui.js no longer has to call FilterListener.setDirty(0) explicitly.

[saroyanm, 2014/06/12 16:56:02]

Thanks for idea. 
Done.

+    FilterListener.setDirty(0);
   },
 
   _loading: false,