diff options
| author | Eddie Ringle <eddie.ringle@gmail.com> | 2014-06-29 16:43:28 -0400 |
|---|---|---|
| committer | George Mount <mount@google.com> | 2015-06-29 13:13:01 -0700 |
| commit | 22369d519d41a0b50bffc62b244f52229a907040 (patch) | |
| tree | aa48237ac91f850e1654fafb51fc09a6064d297a /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | 82e595fd6e3a9438b090106c226bb2f8bb2a6254 (diff) | |
Fix FragmentTransaction replace() behavior
Bug 22174959
According to FragmentTransaction's reference, replace()
should be equivalent to remove()'ing every fragment in
a given container and then adding the specified fragment
to replace those that were removed.
Commit ee76efb74b5886f98cdfebfbefe9b69224e016fb broke
this intended behavior in the framework.
This patch makes it so that the removal loop doesn't
end prematurely. It also fixes an issue where the
replace operation would remove fragments regardless
of their container after encountering the first fragment
with a matching container ID.
Relevant issues:
http://b.android.com/43265
http://b.android.com/52112
http://b.android.com/53393
http://b.android.com/68856
http://b.android.com/70803
Change-Id: Ica4691746ab8979ed974a998e85324e4feacc5e3
Signed-off-by: Eddie Ringle <eddie.ringle@gmail.com>
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions