tree 474559ae5089f04bfe1284afc6ba72e34d20a568
parent 04e2c9ef5663262fc9dd08ffc4b59fe305a017f3
author Tony Wickham <twickham@google.com> 1446687872 -0800
committer Tony Wickham <twickham@google.com> 1446687872 -0800

Stop explicitly snapping to page when navigating with keyboard.

When an icon grabs focus, it already snaps to its own page, so
the explicit snapToPage() is redundant. And in the case where no icon
will grab the focus, the explicit call puts us in a weird state
because the page snaps but the focus doesn't change (see bug).

(This happens, for instance, if you are moving from an icon on the
bottom right of page 1 and the only icon on page 2 is in the top
left. In this case, the second icon won't take focus because the
angle is too wide according to FocusLogic#handleDpadHorizontal().)

Bug: 25434120
Change-Id: I17b0ef7d86426a2599bc948bf2313dca7e112364