diff options
| author | Paul Duffin <paulduffin@google.com> | 2018-07-04 15:24:12 +0100 |
|---|---|---|
| committer | Paul Duffin <paulduffin@google.com> | 2018-07-10 15:50:25 +0100 |
| commit | de79f4c2d0793bee4ece10ca41d2562f25dce255 (patch) | |
| tree | 37f53a6373d2a1c09977797331e943f7435bffe9 /annotations/generate_annotated_java_files.py | |
| parent | d55781402526a127b1fb411fd73ba47e7fec5f06 (diff) | |
Remove early error checking in ZipFile(File,int,Charset)
This change removes an unnecessary Android specific patch on 8u121-b13.
The patch was added in two changes:
The first was change adc40417 which according to the commit message was
done, along with a couple of other changes, to fix a unit test. Exactly
what unit test is not clear in the commit message.
The second was change d95a9080 which added special handling of a
non-existent file in order to maintain backwards compatibility and an
accompanying test.
Removing this whole block seems to have no effect on any of our unit
tests. The test added under d95a9080 still works and contrary to the
commit message of adc40417 no other unit tests fail.
Added extra test to check the behavior when attempting to load a zip
file that is too short. It passes before and after making this change.
The only difference in behavior was the error message reported but that
is not something on which applications should be depending.
Bug: 111061052
Test: checkbuild, flash, CtsLibcoreTestCases
Change-Id: I0b764745b5e29fbaa4c7aec241dc583e5d4d86ee
Diffstat (limited to 'annotations/generate_annotated_java_files.py')
0 files changed, 0 insertions, 0 deletions