Bug 1353264 - Disable testing native stack collection in Beta. r=froydnj, a=test-only
authorMichael Layzell <michael@thelayzells.com>
Fri, 04 Aug 2017 07:23:00 -0400
changeset 423385 3b4cf744da12607de4dda7830dd7a7114de84606
parent 423384 4923833da3f914e9453fc14feebab00cca320509
child 423386 5fb77be6beb6a24a25e618a4e194446a91679f5a
push id1517
push userjlorenzo@mozilla.com
push dateThu, 14 Sep 2017 16:50:54 +0000
treeherdermozilla-release@3b41fd564418 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj, test-only
bugs1353264
milestone56.0
Bug 1353264 - Disable testing native stack collection in Beta. r=froydnj, a=test-only MozReview-Commit-ID: 4Ssxa3iUcHK
toolkit/components/telemetry/tests/unit/test_ThreadHangStats.js
--- a/toolkit/components/telemetry/tests/unit/test_ThreadHangStats.js
+++ b/toolkit/components/telemetry/tests/unit/test_ThreadHangStats.js
@@ -79,27 +79,19 @@ function run_test() {
 
       ok(Array.isArray(endHangs.hangs));
       notEqual(endHangs.hangs.length, 0);
 
       ok(Array.isArray(endHangs.hangs[0].stack));
       notEqual(endHangs.hangs[0].stack.length, 0);
       equal(typeof endHangs.hangs[0].stack[0], "string");
 
-      // Make sure there is a nativeStacks field with native stacks in it.
+      // Make sure there is a nativeStacks field
       ok(Array.isArray(endHangs.nativeStacks.memoryMap));
       ok(Array.isArray(endHangs.nativeStacks.stacks));
-      ok(endHangs.nativeStacks.stacks.length > 0);
-
-      // Make sure that at least one of the hangs contains a native stack.
-      ok(endHangs.hangs.some((hang) => {
-        return typeof hang.nativeStack == "number" &&
-          hang.nativeStack < endHangs.nativeStacks.stacks.length &&
-          Array.isArray(endHangs.nativeStacks.stacks[hang.nativeStack]);
-      }));
 
       check_histogram(endHangs.hangs[0].histogram);
 
       do_test_finished();
     };
 
     check_results();
   });