tree 9087d2d5b38b786e5ce423cc006f7eb58d32cbf6
parent ca557f7a4eae9656a00b6df242a8e189e480f3da
author Dave Borowitz <dborowitz@google.com> 1475589537 -0400
committer Dave Borowitz <dborowitz@google.com> 1475590216 -0400

VisibilityCache: Use streams internally

This is not a huge win in terms of readability or boilerplate
reduction over the old approach using Collections2 methods. However,
using streams makes it clearer where we're materializing to a list to
avoid repeated iteration. In the old code, calling isEmpty() on a
collection produced by Collections2.filter was unexpectedly O(n); in
the new code, we have an explicit collect(toList()) step.

Change-Id: I873fdb99129129ac4e66b522caf16e86c2589fb4