Closed Bug 1036695 Opened 8 years ago Closed 8 years ago

enable mozApps debug logging when running tests

Categories

(Core Graveyard :: DOM: Apps, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla33

People

(Reporter: myk, Assigned: myk)

Details

Attachments

(1 file)

Now that we enable mozApps debug logging with a pref, we should enable it when running tests.
Attachment #8453427 - Flags: review?(fabrice)
Yes please!
Attachment #8453427 - Flags: review?(fabrice) → review+
https://hg.mozilla.org/mozilla-central/rev/ebd23dac96c4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Why does this need to be on when running all tests? It generates a large amount of console spew that makes trying to debug tests more annoying than it needs to be
Flags: needinfo?(myk)
(In reply to Dave Townsend [:mossop] from comment #4)
> Why does this need to be on when running all tests? It generates a large
> amount of console spew that makes trying to debug tests more annoying than
> it needs to be

It doesn't need to be on, but it's been quite helpful for debugging issues with the DOM app registry and the various app runtimes.

Is there some spew that particularly annoyed you recently? It's quite possible that we're overlogging, and perhaps we can reduce the spew enough to stop being a bother while still getting useful info for debugging issues.
Flags: needinfo?(myk) → needinfo?(dtownsend)
Here is the top of a test log:

-*- Webapps.jsm : Saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Success saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Success saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
Shumway is registered
0 INFO *** Start BrowserChrome Test Results ***
1 INFO checking window state
2 INFO TEST-START | browser/base/content/test/general/browser_tab_dragdrop.js
3 INFO Entering test 
SystemMessageCache: init-*-*- Langpacks: getAdditionalLanguages http://example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://test1.example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://test1.example.org:8000/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://sub1.test1.example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_priv.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_cert.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_cert.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_inst.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_priv.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://acertified.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://aprivileged.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest
-*-*- Langpacks: Languages found: ({langs:{}})

The Langpacks lines also repeat whenever a content process restarts which in some of the tests I work on happens frequently.

I guess I don't understand why you can't just enable the apps logging for the apps tests.
Flags: needinfo?(dtownsend)
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.