1144126 - Log utility tray events behind a pref to debug status bar freezing

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Guillaume Marty [:gmarty]]

To fix the problem of the status bar freezing under random circumstances, we'll do the following:

* Log detais to the SD card such as:
  * The utility tray opening/closing events
  * The value of StatusBar._paused
  * Anything we can think of and that can be related to this issue
* This log will be enable via a pref
* Tell dogfooder to activate this setting

Comment 1

[Guillaume Marty [:gmarty]]

The WIP is in this branch (not ready for r+ tho):
https://github.com/gmarty/gaia/tree/Bug-1144126-Log-utility-tray-events-behind-a-pref-to-debug-status-bar-freezing

Comment 2

[Julien Wajsberg [:julienw]]

FWIW I added such logs in my dogfooding 2.1 build but couldn't reproduce since I'm back next to a computer.

Comment 3

[Guillaume Marty [:gmarty]]

On hold till the patch for Bug 1139470 lands.

Comment 4

[Michael Henretty [:mikehenrty][:mhenretty]]

Guillaume, can we finish this off? It looks like bug 1154635 is still happening even after our latest statusbar sync fix, so we should get the logs on the person's device who can reproduce this.

Comment 5

[Hermes Cheng (inactive after July 27, 2015)]

Since blocking bug 1154635 which is a 2.2 blocker, nom tihs one, too.

Comment 6

[Guillaume Marty [:gmarty]]

Let's wait for a few days after Bug 1154800 lands and see if working on this bug is still needed.

Comment 7

[Gregor Wagner [:gwagner]]

Not blocking. Request uplift approval if needed to fix dependent bug.

Comment 8

[GitHub Autolander Bot]

Attachment: [gaia] gmarty:Bug-1144126-Log-utility-tray-events-behind-a-pref-to-debug-status-bar-freezing > mozilla-b2g:master

Comment 9

[Guillaume Marty [:gmarty]]

Attachment: Github PR

This patch logs interesting events and values when enabled.
To enable it, create a new settings called `statusbar.log-events.enabled` and set it to true.

When deactivated, it should not impact performance.

Comment 10

[Etienne Segonzac (:etienne)]

Comment on attachment 8604671 [details] [review]
Github PR

sounds good, r=me minus the file header change

2 suggestions:
- please contribute any idea on how to generalize this to https://groups.google.com/forum/#!topic/mozilla.dev.gaia/5gfXlJUhWJo

- if this is aimed at dogfooders we should add a way to turn on the setting in the developer settings pane (there's already a "Debug" category).

Cheers!

Comment 11

[Guillaume Marty [:gmarty]]

With the revamp of the status bar in 2.5, this patch is not used anymore. Let's close this bug.