tree 33bbf0e5cffa92e6035dfbe3fdeafbd39a0bee1c
parent 4a3b5baec2fb2ee21a62788b7ac9f413479ea0aa
author Dave Borowitz <dborowitz@google.com> 1447860358 -0500
committer David Pursehouse <david.pursehouse@sonymobile.com> 1447909450 +0900

SiteIndexer: Always scan changes from repo/db

This was changed to use ChangeCache in a2b7358d, with the intention of
binding ChangeCache to ScanningChangeCacheImpl in Reindex.
Unfortunately I forgot about the online reindexing case, where
ChangeCache is bound to SearchingChangeCacheImpl in a running master.
This had the effect of omitting from the list of changes to reindex
any changes that were not already present in the index. In many cases
this is fine, but if a change is missing for some reason (including a
race condition during creation), it would not be found.

Fix this by exposing the method in ScanningChangeCacheImpl to actually
scan a repo, and using that from both its Loader and SiteIndexer
directly.

Change-Id: Id384ff93abbb7b6c54819b6c2672fb1f45dc55bc