svn commit: r1371635 - in /incubator/wave/trunk/src/org/waveprotocol/box/webclient/search: SimpleSearch.java WaveBasedDigest.java

[yurize]

Author: yurize
Date: Fri Aug 10 10:07:09 2012
New Revision: 1371635

URL: http://svn.apache.org/viewvc?rev=1371635&view=rev
Log:
Fixes WAVE-363 - number marks of unread blips dont changing in real time.
https://reviews.apache.org/r/6475/

Modified:
    
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/SimpleSearch.java
    
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java

Modified: 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/SimpleSearch.java
URL: 
http://svn.apache.org/viewvc/incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/SimpleSearch.java?rev=1371635&r1=1371634&r2=1371635&view=diff
==============================================================================
--- 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/SimpleSearch.java
 (original)
+++ 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/SimpleSearch.java
 Fri Aug 10 10:07:09 2012
@@ -79,8 +79,7 @@ public final class SimpleSearch implemen
     void activate(WaveContext wave) {
       Preconditions.checkState(dynamicDigest == null);
       dynamicDigest = WaveBasedDigest.create(wave, staticDigest);
-      // TODO(vjrj) add the listener again when this issue is addressed:
-      // https://issues.apache.org/jira/browse/WAVE-354
+      dynamicDigest.addListener(this);
       fireOnChanged();
     }
 

Modified: 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java
URL: 
http://svn.apache.org/viewvc/incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java?rev=1371635&r1=1371634&r2=1371635&view=diff
==============================================================================
--- 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java
 (original)
+++ 
incubator/wave/trunk/src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java
 Fri Aug 10 10:07:09 2012
@@ -161,6 +161,7 @@ public final class WaveBasedDigest
     }
   }
 
+  @SuppressWarnings("unused")
   private void invalidateLmt() {
     lastModified = NO_TIME;
   }
@@ -248,10 +249,8 @@ public final class WaveBasedDigest
 
   @Override
   public void onLastModifiedTimeChanged(ObservableWavelet wavelet, long 
oldTime, long newTime) {
-    if (oldTime >= lastModified || lastModified <= newTime) {
-      invalidateLmt();
-      fireOnChanged();
-    }
+    // TODO (Yuri Z.): Invoke fireOnChanged() here in case lastModifiedTime 
changed after solving
+    // the issue https://issues.apache.org/jira/browse/WAVE-354.
   }
 
   @Override