diff options
author | Mohan Srinivasan <srmohan@google.com> | 2017-01-31 16:06:32 -0800 |
---|---|---|
committer | Mohan Srinivasan <srmohan@google.com> | 2017-01-31 16:06:32 -0800 |
commit | fa5fbd7358636f4fb8a1242949d34c6393e8446f (patch) | |
tree | d3751ab8267470ba658865d6e2f92331c9f10ae3 /app-launcher | |
parent | 963830e56bf4de9c5dbf4f13106315e546f4a0d7 (diff) | |
download | extras-fa5fbd7358636f4fb8a1242949d34c6393e8446f.tar.gz |
Use TotalTime instead of ThisTime as app launch latency metric
Use TotalTime instead of ThisTime (both reported by am) as our
activity launch latency metric. TotalTime is more representative
of user perceived latency. Also add back YouTube to the list of
apps we launch test by default.
Test: Time app launches manually for the various apps, and check
that TotalTime better represents the launch latency.
Change-Id: I127a22ae409ee0141d847e711e09f7873dbb420f
Signed-off-by: Mohan Srinivasan <srmohan@google.com>
Diffstat (limited to 'app-launcher')
-rwxr-xr-x | app-launcher/app-launcher | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/app-launcher/app-launcher b/app-launcher/app-launcher index 44e62f8d..f02b3dcc 100755 --- a/app-launcher/app-launcher +++ b/app-launcher/app-launcher @@ -45,19 +45,13 @@ getstats () { app=$2 echo "Data for $app :" - # Chrome related Hack alert : - # For Chrome launches, "TotalTime" is the right metric to measure. That is closer - # to what the user seems Chrome launch time as. So special case Chrome here. - - if [ $app == Chrome ]; then - launch_time_string="TotalTime" - else - launch_time_string="ThisTime" - fi + # Activity Manager reports ThisTime and TotalTime. TotalTime seems to be + # a more measure of the launch from the users perspective. So using TotalTime + # as our metric for launch latency # From Activity Manager - echo "Launch Time :" - fgrep $launch_time_string $infile | awk '{print $2}' | computestats + echo "Launch Time (TotalTime) :" + fgrep TotalTime $infile | awk '{print $2}' | computestats # Data from simpleperf echo "cpu-cycles :" @@ -360,8 +354,8 @@ launch_phone_apps() { getstats camera-$model Camera launch_app com.google.android.apps.maps com.google.android.maps.MapsActivity > maps-$model getstats maps-$model Maps -# launch_app com.google.android.youtube com.google.android.apps.youtube.app.WatchWhileActivity > youtube-$model -# getstats youtube-$model YouTube + launch_app com.google.android.youtube com.google.android.apps.youtube.app.WatchWhileActivity > youtube-$model + getstats youtube-$model YouTube } usage() { |