| commit | e096e2633f7e8a6bada6cd7548b359a29ef2bc4e | [log] [tgz] |
|---|---|---|
| author | Robin Lee <rgl@google.com> | Mon Jan 09 14:46:10 2017 +0000 |
| committer | Robin Lee <rgl@google.com> | Mon Jan 09 19:53:46 2017 +0000 |
| tree | 8a1479f184bf0fa69e2e9d09eb5320c8f1e97f71 | |
| parent | cedaea61f931beb8099a6cda11a4fde3e39a212e [diff] |
Use stable item ID instead of URI for checkedItem This is faster, more correct, and also harder to get wrong. And it was wrong, there were quite common situations where the previous code threw CursorIndexOutOfBoundsException because getRingtonePosition() called getItemId() which called... getRingtonePosition() again. CC Bug: 30658854 Test: monkeytest: crashed this before; now we are crash-free. Change-Id: I68c06f2123fb84c15d4bbadec7be61c5cdfd1da7