diff options
author | Bernardo Rufino <brufino@google.com> | 2018-06-27 12:50:44 +0100 |
---|---|---|
committer | Bernardo Rufino <brufino@google.com> | 2018-06-29 07:18:26 +0000 |
commit | 62863825e08cd79f272f5ebfb11c728843b5b2a5 (patch) | |
tree | 5fbcff9ebc8a72e972574c68d82b82447dca23d4 /tools/aapt2/StringPool_test.cpp | |
parent | 9a936e967760e0d9e7787f0a4642c692c6d5c969 (diff) |
Fix file matching w/ full-backup rules xml
Documentation is pretty vague:
https://developer.android.com/guide/topics/data/autobackup#XMLSyntax.
But there were a couple of issues:
* It was prematurely returning false without consuming the rest of the
includes (cause of the bug linked).
* It was using string comparison for checking if a file is in a
directory, which ended up flagging directories such as "a/b" as
containing files "a/b.txt".
Reviewers,
* Please, pay full attention to test cases.
* Since this is code move + code change, set diff as 2..latest to check
changes to the function.
Bug: 110720194
Test: atest BackupUtilsTest
Test: Backup and restore app w/ multiple directory includes, verify
everything restored
Change-Id: Ic0fea43156ce8fb641af69ae73679289a20c291c
Diffstat (limited to 'tools/aapt2/StringPool_test.cpp')
0 files changed, 0 insertions, 0 deletions