9c8a770bde30a6e0796616a35af3f7360f9c8aab: Bug 1387793 - Downgrade assertion to warning. r=kinetik
Alfredo Yang <ayang@mozilla.com> - Thu, 10 Aug 2017 11:39:44 +0800 - rev 374178
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1387793 - Downgrade assertion to warning. r=kinetik
5370f4a8426806f0644e0a01f45f60aaade70b56: Bug 1388032 - Remove BrowserCustomizeToolbar. r=dao
Hossain Al Ikram <hossainalikram@gmail.com> - Mon, 07 Aug 2017 10:59:00 -0400 - rev 374177
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1388032 - Remove BrowserCustomizeToolbar. r=dao
56f67af11f80f83efa2a6f68726b5b808d454151: Bug 1386369 - Part 3: Use double quotes for strings, a=eslint-bustage
Michael Layzell <michael@thelayzells.com> - Thu, 10 Aug 2017 16:12:40 -0400 - rev 374176
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1386369 - Part 3: Use double quotes for strings, a=eslint-bustage MozReview-Commit-ID: LYVrPDQ81LU
272d381012677aedc86462ebd8e45b0b30884655: Bug 1386369 - Part 2: Stop emitting bhr-thread-hang in the content process, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 10 Aug 2017 14:32:08 -0400 - rev 374175
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1386369 - Part 2: Stop emitting bhr-thread-hang in the content process, r=froydnj MozReview-Commit-ID: 1A7CorZepdQ
51516e2306a418cc3cafab4a63835b24058aab33: Bug 1386369 - Part 1: Change test_BHRObserver to not listen to bhr-thread-hang in content, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 10 Aug 2017 14:25:13 -0400 - rev 374174
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1386369 - Part 1: Change test_BHRObserver to not listen to bhr-thread-hang in content, r=froydnj MozReview-Commit-ID: 7nFBdtiXFc2
6bca451909471b425da98f2658db0bc1676da1fa: Bug 1380081 - Part 14: Include BHRTelemetryService and xpt files in the package, r=froydnj
Michael Layzell <michael@thelayzells.com> - Tue, 01 Aug 2017 16:22:40 -0400 - rev 374173
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 14: Include BHRTelemetryService and xpt files in the package, r=froydnj MozReview-Commit-ID: IH56INaSOoK
314ddbc9d70f07767da4fd0c61b73d31044e2e2c: Bug 1380081 - Part 13: Add telemetry documentation for the new ping format, r=bsmedberg, r=froydnj
Michael Layzell <michael@thelayzells.com> - Tue, 25 Jul 2017 11:16:29 -0400 - rev 374172
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 13: Add telemetry documentation for the new ping format, r=bsmedberg, r=froydnj MozReview-Commit-ID: G4hFZcR2EGL
cc4253078c54640c285d49b4d9ae56017219d0b9: Bug 1380081 - Part 12: Add a test for BHR observer notifications, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:32:38 -0400 - rev 374171
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 12: Add a test for BHR observer notifications, r=froydnj This patch just adds some tests for the new stuff which we added. MozReview-Commit-ID: 2jG75AEUJfS
9158ca4292bb0f1d1f1b65802bd73e8292da7252: Bug 1380081 - Part 11: Simplify the HangAnnotations abstraction, r=froydnj
Michael Layzell <michael@thelayzells.com> - Fri, 21 Jul 2017 16:57:48 -0400 - rev 374170
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 11: Simplify the HangAnnotations abstraction, r=froydnj HangAnnotations was very complex, required a separate allocation, and used this unfortunate virtual interface implementation which made it harder to do interesting things with it (such as serialize it over IPC). This new implementation is much simpler and more concrete, making HangAnnotations simply be a nsTArray<Annotation>. This also simplifies some of the IPC code which was added in part 7. MozReview-Commit-ID: EzaaxdHpW1t
d34e59a86079e6f119c38030c4fd00318f118ce4: Bug 1380081 - Part 10: Stop running BHR on beta, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:40:46 -0400 - rev 374169
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 10: Stop running BHR on beta, r=froydnj These changes are going to increase the amount of data which we collect from BHR a lot. It would be dangerous to run it on beta, especially considering how soon the next merge is. This should turn it off for 100% of beta users if I understand the logic correctly. MozReview-Commit-ID: 3HyEKWdXaqU
0b6c4c95323e0892df1acd281326665e23dcc320: Bug 1380081 - Part 9: Transmit BHR Hangs from the Content and GPU process to the parent process, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 12:49:28 -0400 - rev 374168
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 9: Transmit BHR Hangs from the Content and GPU process to the parent process, r=froydnj BHRTelemetryService only runs in the parent process (and we can only submit pings from there), so we need to send the data which we collect in the GPU and Content processes over IPC to the parent process. MozReview-Commit-ID: 8B5uZKbjNbU
8d0b5e469b69b94bb38ae1c9973a1677aa4664f0: Bug 1380081 - Part 8: Report bhr-thread-hang hangs to telemetry in a custom bhr ping, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:33:53 -0400 - rev 374167
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 8: Report bhr-thread-hang hangs to telemetry in a custom bhr ping, r=froydnj This patch adds the BHRTelemetryService which is a JS implemented XPCOM service that simply listens to the bhr-thread-hang observer notification, and uses the data it collects from it to submit telemetry pings. MozReview-Commit-ID: 2hPXAFmHrm5
6bc453505dc3f74398e2de70a616d9d0c5fb7787: Bug 1380081 - Part 7: Add all necessary data for BHR to nsIHangDetails, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:31:56 -0400 - rev 374166
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 7: Add all necessary data for BHR to nsIHangDetails, r=froydnj We're going to use HangDetails as the type containing hang information. We'll have a JS component which reads the data out of nsIHangDetails, builds the payload, and submits it to telemetry for us. We'll do it in JS because telemetry has to be submitted from JS. This patch also adds IPC serization for the relevant types so that we can send HangDetails objects over IPDL. MozReview-Commit-ID: CeikKabY9Vs
d77a13ef7a4918cf2943c0f34169d730f0fae632: Bug 1380081 - Part 6: Remove test_ThreadHangStats.js, r=chutten
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:29:53 -0400 - rev 374165
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 6: Remove test_ThreadHangStats.js, r=chutten A new test using the new APIs is introduced in a later part. This test no longer functions as ThreadHangStats is no longer present. MozReview-Commit-ID: HKmoP2An4GP
2811e49735568ceb37dd11bcc83fce1ca0a06bbf: Bug 1380081 - Part 5: Add some helper methods to HangAnnotations, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:27:19 -0400 - rev 374164
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 5: Add some helper methods to HangAnnotations, r=froydnj These will be used to implement IPC serialization and deserialization of the HangDetails object to send over IPC. This is a temporary measure as HangAnnotations is rewritten in part 11. MozReview-Commit-ID: 1WHNvhDrMF5
68bec64643389e1013beb1296a8856ca5cd3e0e0: Bug 1380081 - Part 4: Move BHR into its own component, r=froydnj
Michael Layzell <michael@thelayzells.com> - Thu, 20 Jul 2017 14:11:40 -0400 - rev 374163
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 4: Move BHR into its own component, r=froydnj MozReview-Commit-ID: 7TOGofAYM6W
bd63a8fecf00a2f185e8e87dc8e8e566e24faf9c: Bug 1380081 - Part 3: Remove the ThreadHangStats object and related code, r=froydnj
Michael Layzell <michael@thelayzells.com> - Mon, 17 Jul 2017 16:37:55 -0400 - rev 374162
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 3: Remove the ThreadHangStats object and related code, r=froydnj MozReview-Commit-ID: EtrktVmc3vP
b1fdf3c9f0742ec695aacf3fc14c48a6bd0c5df0: Bug 1380081 - Part 2: Remove ThreadHangStats from about:telemetry, r=chutten
Michael Layzell <michael@thelayzells.com> - Mon, 17 Jul 2017 16:37:39 -0400 - rev 374161
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 2: Remove ThreadHangStats from about:telemetry, r=chutten MozReview-Commit-ID: KTOSKobhNAJ
fe3893891adea4475aad9703e9740f64a8ba2117: Bug 1380081 - Part 1: Remove getChildThreadHangs, r=chutten
Michael Layzell <michael@thelayzells.com> - Mon, 17 Jul 2017 15:45:03 -0400 - rev 374160
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1380081 - Part 1: Remove getChildThreadHangs, r=chutten This is the first part of purging the existing telemetry code for ThreadHangStats from the tree. All of these features will be replaced with new code for BHR telemetry in the future. MozReview-Commit-ID: BhD5zY2LwUR
d461adfd884e329934bff39cb9afd0258b04d2de: Bug 1365309 - Part 3: Reduce expected frame count for GeckoProfiler.SuspendAndSample, r=njn
Michael Layzell <michael@thelayzells.com> - Tue, 08 Aug 2017 13:28:29 -0400 - rev 374159
Push 32318 by kwierso@gmail.com at Fri, 11 Aug 2017 20:16:01 +0000
Bug 1365309 - Part 3: Reduce expected frame count for GeckoProfiler.SuspendAndSample, r=njn This is necessary as with FramePointerStackWalk if the current frame is in glibc when we suspend the target thread we sometimes can't collect more than one native frame as there are no frame pointers. We'll still collect at least one entry, so this check relaxes the assertion. MozReview-Commit-ID: IbatTQwDti1
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip