4a7dd87506167cf613c905f0249cceb70c7a6b7c: Bug 1382304: hg/gitignore captures/ dir. r=nalexander
Michael Comella <michael.l.comella@gmail.com> - Wed, 19 Jul 2017 11:08:06 -0700 - rev 369761
Push 32203 by archaeopteryx@coole-files.de at Thu, 20 Jul 2017 08:55:28 +0000
Bug 1382304: hg/gitignore captures/ dir. r=nalexander MozReview-Commit-ID: 9PZoAItKsRI
eb1d92b2b6a4161492561250f51bae5bafeda68a: Merge inbound to central, a=merge CLOSED TREE
Wes Kocher <wkocher@mozilla.com> - Wed, 19 Jul 2017 17:29:56 -0700 - rev 369760
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Merge inbound to central, a=merge CLOSED TREE MozReview-Commit-ID: CBL7SUEUGKV
6fe44a6c4e46426eebee98c121800c0263a06b47: Backed out 13 changesets (bug 1366294) for leaking base::Histogram::FactoryGet a=backout CLOSED TREE
Wes Kocher <wkocher@mozilla.com> - Wed, 19 Jul 2017 12:49:46 -0700 - rev 369759
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Backed out 13 changesets (bug 1366294) for leaking base::Histogram::FactoryGet a=backout CLOSED TREE Backed out changeset f2d79c13799a (bug 1366294) Backed out changeset b722d638f6ed (bug 1366294) Backed out changeset aee317b1445d (bug 1366294) Backed out changeset 14f9f9521d4f (bug 1366294) Backed out changeset beb5f00c4b19 (bug 1366294) Backed out changeset 72635bc1ba25 (bug 1366294) Backed out changeset c387459a798b (bug 1366294) Backed out changeset 476daf9a5846 (bug 1366294) Backed out changeset e7ce6a06c131 (bug 1366294) Backed out changeset 1cc736607aab (bug 1366294) Backed out changeset 175c3ccb0015 (bug 1366294) Backed out changeset 96b594816a21 (bug 1366294) Backed out changeset 57400dd449ad (bug 1366294) MozReview-Commit-ID: J6PyVMGwJ74
f2d79c13799ab0ca6eeb044aabdf873ac225efd2: bug 1366294 - Part 13 - Fix telemetry-using tests. r=Dexter
Chris H-C <chutten@mozilla.com> - Mon, 17 Jul 2017 11:28:55 -0400 - rev 369758
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 13 - Fix telemetry-using tests. r=Dexter Some tests were using "secret" APIs to get at telemetry knowledge from various processes in various ways. Adjust them so that they work with the new ways of doing things. MozReview-Commit-ID: 2iBL00HVGyg
b722d638f6ed9d6a04ed4fa8285029c781c843b7: bug 1366294 - Part 12 - Clean up after onesself. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Fri, 14 Jul 2017 15:02:54 -0400 - rev 369757
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 12 - Clean up after onesself. r=gfritzsche Without the StatisticsRecorder cleaning up after us we need to now manually delete the Histograms we create. MozReview-Commit-ID: 8jrPPBQmU0Z
aee317b1445d926a176b31c14ae1e627fc0b57fd: bug 1366294 - Part 11 - Fix OSX Build. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Thu, 06 Jul 2017 17:05:30 -0400 - rev 369756
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 11 - Fix OSX Build. r=gfritzsche MozReview-Commit-ID: DSljXr8ZRhy
14f9f9521d4f2e30e084233a6a1f8cc1160eaca4: bug 1366294 - Part 10 - Nail down count histogram semantics. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Tue, 04 Jul 2017 10:12:23 -0400 - rev 369755
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 10 - Nail down count histogram semantics. r=gfritzsche Previously we assumed count histograms were always present in payloads. This was an erroneous assumption as count histograms were only 0 if they were session histograms, or if they were from subsession histograms from subsessions _after_ a subsession when they held a non-0 value. So let's just treat count histograms as normal histograms from now on, without any of this "sometimes 0" nonsense. This simplifies the code, tests, and our understanding... and _should_ have few/zero downstream effects since the existing behaviour was so poorly-understood (though exactly tested). MozReview-Commit-ID: BH108ksygGw
beb5f00c4b195c51fc8cb37ae21068bd21b4138f: bug 1366294 - Part 9 - Fix Windows Build. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Wed, 28 Jun 2017 11:00:41 -0700 - rev 369754
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 9 - Fix Windows Build. r=gfritzsche MozReview-Commit-ID: 8P2W8fD2xNi
72635bc1ba2511e8e9ce7303982e8b89d1d050aa: bug 1366294 - Part 8 - Include a message on telemetry testfails. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Fri, 23 Jun 2017 16:39:59 -0400 - rev 369753
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 8 - Include a message on telemetry testfails. r=gfritzsche I just added some short identifiable messages to help identify which part of the test failed. MozReview-Commit-ID: 3AX2iucGiRx
c387459a798bbc74ace160813e18563aca8ca94e: bug 1366294 - Part 7 - Use keyed histogram snaphots to build payloads. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Fri, 23 Jun 2017 16:38:21 -0400 - rev 369752
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 7 - Use keyed histogram snaphots to build payloads. r=gfritzsche TelemetrySession's getKeyedHistograms asks for each keyed histogram individually. This is inefficient and doesn't work well with the storage refactor. So, plumb through a subsession keyed histogram snapshot API and convert TelemetrySession over to using it. MozReview-Commit-ID: Af9dTqw99UA
476daf9a58465e1954419c874148493cbac534cc: bug 1366294 - Part 6 - Update test_TelemetryHistograms for new snapshots format r=gfritzsche
Chris H-C <chutten@mozilla.com> - Fri, 23 Jun 2017 15:15:11 -0400 - rev 369751
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 6 - Update test_TelemetryHistograms for new snapshots format r=gfritzsche MozReview-Commit-ID: FN6y1zTqFt4
e7ce6a06c131dfe37ceea24a3b09a5b5d6475127: bug 1366294 - Part 5 - Send appropriately-structured Telemetry data to JS r=gfritzsche
Chris H-C <chutten@mozilla.com> - Tue, 20 Jun 2017 15:03:10 -0400 - rev 369750
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 5 - Send appropriately-structured Telemetry data to JS r=gfritzsche Previously we were doing bad string manipulation nonsense. Now when asked for snapshots C++ can return a properly-formated Object tree. MozReview-Commit-ID: HAvIbgzUvMU
1cc736607aab9744c015e9dc0734dc35511db825: bug 1366294 - Part 4 - Small cleanups to previous patches. r=gfritzsche
Chris H-C <chutten@mozilla.com> - Fri, 23 Jun 2017 16:42:21 -0400 - rev 369749
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
bug 1366294 - Part 4 - Small cleanups to previous patches. r=gfritzsche MozReview-Commit-ID: 7PioVNfUUNZ
175c3ccb00155c5fe5426fbd7b1f0faa70aca11b: Bug 1366294 - Part 3 - Refactor TelemetryHistogram storage. r=chutten
Georg Fritzsche <georg.fritzsche@googlemail.com> - Thu, 08 Jun 2017 16:15:38 +0700 - rev 369748
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1366294 - Part 3 - Refactor TelemetryHistogram storage. r=chutten Previously we used the base::StatisticsRecorder object for storage by name. This is keyed by histogram name, which doesn't match our storage reality anymore. Instead we use a name to refer to a set of histogram instances that record data from different processes, as well as separating session and subsession data. In this re-write, we instead introduce the following lookup paths (managed in TelemetryHistogram.cpp): - Main storage: - (histogramId, processId, sessionOrSubsession) -> Histogram* - (histogramId, processId) -> KeyedHistogram* (this handles subsessions internally) - Lookup: - (histogramName) -> histogramId - (HistogramID) -> bool (is recording enabled for this histogram?) This is wrapped with a few lookup functions. This also allows us to keep HistogramIDs in the JS histogram instances now, instead of pointers to Histogram instances. That means Histogram instance life-time management is now properly contained inside TelemetryHistogram.cpp. MozReview-Commit-ID: 5yijGv7mc89
96b594816a21fcafb313b710211ccfa6b1a4237e: Bug 1366294 - Part 2 - Cleanup Chromium Histogram code. r=chutten
Georg Fritzsche <georg.fritzsche@googlemail.com> - Tue, 06 Jun 2017 16:48:11 +0700 - rev 369747
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1366294 - Part 2 - Cleanup Chromium Histogram code. r=chutten - A histogram name identifies a set of histogram instances, for which storage and lookup will be handled in TelemetryHistogram.cpp. So we remove the names from histogram code. - Various unused macros in the header are removed. - Remaining traces of StatisticsRecorder are removed from the Histogram class code. - Some unused methods are dropped that were about printing histograms to ASCII etc. MozReview-Commit-ID: BF2rLSpKOJ8
57400dd449ad5dd77f88054e06c647023eaa3866: Bug 1366294 - Part 1 - Remove base::StatisticsRecorder. r=chutten
Georg Fritzsche <georg.fritzsche@googlemail.com> - Sat, 03 Jun 2017 01:09:11 +0700 - rev 369746
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1366294 - Part 1 - Remove base::StatisticsRecorder. r=chutten The Chromium IPC histogram code used the StatisticsRecorder object for storage. This is keyed by histogram name, which doesn't match our storage reality anymore. Instead we use a name to refer to a set of histogram instances that record data from different processes, as well as separating session and subsession data. Consequently we need to rewrite this storage, which means StatisticsRecorder is not used anymore. MozReview-Commit-ID: 1LC7YubpKaD
cf1afed54d07d18524b35395d427cc840caf5019: Bug 1379643: When running under sandbox level >= 3, parent should retain IStream of marshaled interface to be destroyed later; r=jimm
Aaron Klotz <aklotz@mozilla.com> - Wed, 19 Jul 2017 12:07:45 -0600 - rev 369745
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1379643: When running under sandbox level >= 3, parent should retain IStream of marshaled interface to be destroyed later; r=jimm MozReview-Commit-ID: Egb6Yahdbxm
24dc2160940436a8d5bfcf2a591c7936df0f00ee: Bug 1379280 - Only do async keyboard scrolling for a selection, not a focused element. r=smaug
Ryan Hunt <rhunt@eqrion.net> - Thu, 13 Jul 2017 15:53:26 -0500 - rev 369744
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1379280 - Only do async keyboard scrolling for a selection, not a focused element. r=smaug This commit changes async keyboard scrolling to be enabled only if the content to scroll is from a selection. This works around the problem of detecting whether an arbitrary element has key listeners that should prevent async key scrolling, because when they have the focus we will have disabled async key scrolling. MozReview-Commit-ID: 6HhSuGZNsMX
5b0d1fdf2d20b75caf46d07af956ba5251674359: Bug 1379280 - Move FocusTarget include from nsIPresShell to PresShell. r=smaug
Ryan Hunt <rhunt@eqrion.net> - Thu, 13 Jul 2017 13:00:21 -0500 - rev 369743
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1379280 - Move FocusTarget include from nsIPresShell to PresShell. r=smaug FocusTarget used to be needed in nsIPresShell, but it was moved to PresShell. MozReview-Commit-ID: HQFpT9OJaUz
6d68d842f601c310626d2ccedab94b38da321b72: Bug 1381736 - Fix modifier masks for APZ keyboard shortcuts. r=masayuki
Ryan Hunt <rhunt@eqrion.net> - Tue, 18 Jul 2017 02:03:14 -0400 - rev 369742
Push 32202 by kwierso@gmail.com at Thu, 20 Jul 2017 00:30:04 +0000
Bug 1381736 - Fix modifier masks for APZ keyboard shortcuts. r=masayuki MozReview-Commit-ID: LGQzHabEqrz
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip