diff options
| author | 2019-02-26 17:11:24 -0800 | |
|---|---|---|
| committer | 2019-04-09 21:04:49 +0000 | |
| commit | ecd21be250c434d854833aa368610c879fea9245 (patch) | |
| tree | 68eecccf43a4ac9292cbc3e338bbc97419787dab /test/1337-gc-coverage/gc_coverage.cc | |
| parent | fc8e7598fe013bebff210b48347daf80316ac0a4 (diff) | |
Fix tracing shutdown
Method tracing was being disabled before the listener was removed.
In some cases, this would cause the stack restoration to trigger
trace entries - resulting in out-of-order method exits.
By removing the listeners before disabling tracing, no one is around
to receive the erroneous stack restoration "exits".
Bug: 129716777
Test: Builds, ran on x86 emulator:
adb shell am start -S -W --start-profiler \
/data/local/tmp/test.trace --sampling 1 \
-a \android.intent.action.MAIN \
-n com.android.calendar/com.android.calendar.AllInOneActivity && \
sleep 5 && \
adb shell am profile stop com.android.calendar && \
adb pull /data/local/tmp/test.trace
and inspected resulting tracefile; parsed fine in Android Studio 3.0.1
Change-Id: I90f0751f4b4253c7c9711d05d53f31b3e545fc3b
Diffstat (limited to 'test/1337-gc-coverage/gc_coverage.cc')
0 files changed, 0 insertions, 0 deletions