1285913 - test_ChildHistograms.js is failing locally on Linux

Status: RESOLVED WORKSFORME

(Toolkit :: Telemetry, defect, P3)

Description

[Alessio Placitelli [:Dexter]]

Attachment: failure_log.txt

The latest m-c [1] is making test_ChildHistograms.js [2] permanently fail locally on my system.

Apparently, TELEMETRY_TEST_* histograms are not being reported in content processes.

[1] - https://hg.mozilla.org/mozilla-central/rev/1bee8d2da23e
[2] - https://dxr.mozilla.org/mozilla-central/rev/23dc78b7b57e9f91798ea44c242a04e112c37db0/toolkit/components/telemetry/tests/unit/test_ChildHistograms.js

Comment 1

[Alessio Placitelli [:Dexter]]

The problem itself is:

 0:03.51 TEST_STATUS: Thread-3 parent  PASS Child payload should have keyed histograms. - true == true
 0:03.51 TEST_STATUS: Thread-3 parent  FAIL Should have count test histogram. - false == true
    /home/dexter/mozilla-central/objdir-ff-debug/_tests/xpcshell/toolkit/components/telemetry/tests/unit/test_ChildHistograms.js:check_histogram_values:38
    /home/dexter/mozilla-central/objdir-ff-debug/_tests/xpcshell/toolkit/components/telemetry/tests/unit/test_ChildHistograms.js:null:96
    _run_next_test@/home/dexter/mozilla-central/testing/xpcshell/head.js:1564:9
    do_execute_soon/<.run@/home/dexter/mozilla-central/testing/xpcshell/head.js:712:9
    _do_main@/home/dexter/mozilla-central/testing/xpcshell/head.js:209:5
    _execute_test@/home/dexter/mozilla-central/testing/xpcshell/head.js:544:5
    @-e:1:1
 0:03.51 LOG: Thread-3 parent INFO exiting test
 0:03.51 LOG: Thread-3 parent ERROR Unexpected exception 2147500036
undefined

With 

0:03.63 LOG: Thread-3 child INFO "CONSOLE_MESSAGE: (info) 1468233062919	Toolkit.Telemetry	TRACE	TelemetrySession#content::getHistograms - Skipping test histogram: TELEMETRY_TEST_FLAG"
 0:03.64 PROCESS_OUTPUT: Thread-3 (pid:28060) 

A few lines below.

Comment 2

[Chris H-C :chutten]

Which means this._testing is false. Bummer.

I happen to be rewriting the test for bug 1218576, so maybe this'll all shake out when that mass of patches is landable?

Comment 3

[Alessio Placitelli [:Dexter]]

(In reply to Chris H-C :chutten from comment #2)
> Which means this._testing is false. Bummer.
> 
> I happen to be rewriting the test for bug 1218576, so maybe this'll all
> shake out when that mass of patches is landable?

Sure!

Comment 4

[Alessio Placitelli [:Dexter]]

This is not happening any more.