supersize hitting dex accounting error log for ChromeModernPublic.apk |
|||
Issue descriptionDiscovered by our bot on bug 851535 . Interestingly, does not reproduce for monochrome_apk. Command: tools/binary_size/supersize archive foo.size --apk-file out-gn/Release/apks/Ch romeModernPublic.apk Log looks like: E 43690 Node size too large, check for node processing errors. dex_expected_size=6546088 total_node_size=6559549
,
Jun 13 2018
Opps, that was for monochrome. This is for ChromeModernPublic.apk. Apkanalyzer output: P d 47363»59953»6607108»<TOTAL> P d 30273»33523»3906805»org P d 30273»33451»3905005»org.chromium P d 21976»24694»2921677»org.chromium.chrome P d 21975»24693»2878813»org.chromium.chrome.browser P d 1856»»2090»»246562»»org.chromium.chrome.browser.compositor P d 909»997»129970»»org.chromium.chrome.browser.compositor.layouts P d 408»456»69247»org.chromium.chrome.browser.compositor.layouts.phone P d 246»269»46245»org.chromium.chrome.browser.compositor.layouts.phone.stack ... Command: third_party/android_tools/sdk/tools/bin/apkanalyzer dex packages out/Release/apks/ChromeModernPublic.apk --proguard-mappings out/Release/apks/ChromeModernPublic.apk.mapping > ~/tmp/chromemodernpackages Disk size: unzip -l -v out/Release/apks/ChromeModernPublic.apk | grep classes.dex 6577292 Defl:N 3161559 52% 2001-01-01 00:00 8ba087ae classes.dex
,
Jun 13 2018
Notice that apkanalyzer reports 29816 bytes more than the uncompressed disk size of classes.dex.
Even including the other optional dex we have, it is still reporting more than disk size of all dex files.
All dex:
unzip -l -v out/Release/apks/ChromeModernPublic.apk | grep dex 10636 Defl:N 4956 53% 2001-01-01 00:00 17d8e604 assets/webapk5.dex
1 Stored 1 0% 2001-01-01 00:00 84b12bae assets/webapk_dex_version.txt
6577292 Defl:N 3161559 52% 2001-01-01 00:00 8ba087ae classes.dex
,
Jun 14 2018
Already fixed, just need to revert the assert change once it trickles into the Android SDK: https://github.com/JesusFreke/smali/commit/f16b420b701ac0e423cd044d61602cc890d7ee46
,
Yesterday
(44 hours ago)
Not worth tracking here anymore by the sounds of it. |
|||
►
Sign in to add a comment |
|||
Comment 1 by wnwen@chromium.org
, Jun 13 2018