tree de62cfca3538f4bb51d8ca72d703a17c8c1175ce
parent 0fa5ada2261a29ad9be7195c8cdb8cb50bdd0742
author Tony Wickham <twickham@google.com> 1449251297 -0800
committer Tony Wickham <twickham@google.com> 1449251297 -0800

Cleanup keyboard code.

This should be the last keyboard CL.
- Fix bug: couldn't focus All Apps button in some cases when the All
Apps column was skipped over. Also added test case for this.
- Stop explicitly passing countX and countY to handleKeyEvent, as
these had to match the matrix dimensions anyways.
- Rename createSparseMatrix() - there were 3 methods of the same name,
but all had different purposes. This is confusing both from a
readability standpoint and also when looking at stack traces.

Change-Id: I08ba8411674fcea43a608856c114dee8dbd22398