1227188 - Autophone - use new browser start logcat message to determine start time

Status: RESOLVED FIXED

(Testing Graveyard :: Autophone, defect)

Description

[Bob Clary [:bc] (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/db9e19d4bbb9 provides a means of determining when Fennec has started. We should begin using this on trunk and fall back to the older methods if it is not found.

Comment 1

[Bob Clary [:bc] (inactive)]

taking

Comment 2

[Bob Clary [:bc] (inactive)]

The current message appears after Throbber start so it isn't useful. See bug 1214810 comment 5

Comment 3

[Bob Clary [:bc] (inactive)]

Attachment: bug-1227188-v1.patch

Adds support for the new Fennec application start message. This will increase the detected start time leading to a apparent improvement in Throbber start, stop which is solely due to this change in measurement.

Note, I also included a fix for detecting Start proc on Android 5.1 where the pid and user are included in the Start proc message. Example: Start proc 3682:org.mozilla.fennec_bclary/u0a54 for activity org.mozilla.fennec_bclary/org.mozilla.gecko.webapp.Dispatcher

Comment 4

[Geoff Brown [:gbrown]]

Comment on attachment 8699602 [details] [diff] [review]
bug-1227188-v1.patch

Review of attachment 8699602 [details] [diff] [review]:
-----------------------------------------------------------------

::: tests/s1s2test.py
@@ +283,5 @@
>                                              (logcat_prefix, throbber_prefix))
>          re_throbber_stop_time = re.compile('%s %s stop' %
>                                             (logcat_prefix, throbber_prefix))
> +        self.loggerdeco.debug(
> +            'analyze_logcat: re_start_time: %s' % re_start_time.pattern)

This seems like a low-value thing to log.

Comment 5

[Bob Clary [:bc] (inactive)]

Ok. removed the low value log messages.

https://github.com/mozilla/autophone/commit/90a7885549f32facffd6bb2829c679dc534dd5c2