Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d2376279 authored by Garfield Tan's avatar Garfield Tan
Browse files

libhwui: Use wc instead of du to monitor file write 

du returns the disk space a file occupies, which isn't guaranteed to be
0 even if the file is empty on all file systems. Use "wc -c" instead to
obtain the file size.

Bug: 184878226
Test: The script waits for the file to have content on file system that
uses disk space to store empty files.
Test: Skia capture still works for Pixel.

Change-Id: Ib5f75b6712b476fc521a531fba029f91f4b281b5
parent 001bc116

libs/hwui/tests/scripts/skp-capture.sh

+2 −2
Original line number Diff line number Diff line
@@ -76,8 +76,8 @@ banner '...WAITING FOR APP INTERACTION...' 
# so we continue to show the "waiting for app interaction" message as long as the app still requires

# interaction to draw more frames.

adb_test_file_nonzero() {

    # grab first byte of `du` output

    X="$(adb shell "du \"$1\" 2> /dev/null | dd bs=1 count=1 2> /dev/null")"

    # grab first byte of `wc -c` output

    X="$(adb shell "wc -c \"$1\" 2> /dev/null | dd bs=1 count=1 2> /dev/null")"

    test "$X" && test "$X" -ne 0

}

timeout=$(( $(date +%s) + $phase1_timeout_seconds))