searching for reviewer(gfritzsche)
cc295c6c964d: Bug 1428612 - Part 2: Add an assertion to check the histogram name pointer. r=gfritzsche a=lizzard
Eric Rahm <erahm@mozilla.com> - Wed, 17 Jan 2018 16:52:37 -0800 - rev 452126
Push 8620 by btara@mozilla.com at 2018-01-31 10:44 +0000
Bug 1428612 - Part 2: Add an assertion to check the histogram name pointer. r=gfritzsche a=lizzard This adds a dignostic assertion that the name pointer is actually within the static name table.
1ced99ea80c9: Bug 1428612 - Part 1: Use nsLiteralCString to wrap histogram names. r=gfritzsche a=lizzard
Eric Rahm <erahm@mozilla.com> - Wed, 17 Jan 2018 16:38:18 -0800 - rev 452125
Push 8620 by btara@mozilla.com at 2018-01-31 10:44 +0000
Bug 1428612 - Part 1: Use nsLiteralCString to wrap histogram names. r=gfritzsche a=lizzard This switches from `nsDependentCString` to `nsLiteralCString` when adding histograms to the table. It will help save some memory and should avoid the OOM smalls we have been seeing.
db9269c469a7: Bug 1432855 - Early-return if we're asked to clear the expired histogram. r=gfritzsche, a=lizzard
Chris H-C <chutten@mozilla.com> - Wed, 24 Jan 2018 16:26:29 -0500 - rev 452113
Push 8616 by ryanvm@gmail.com at 2018-01-30 15:40 +0000
Bug 1432855 - Early-return if we're asked to clear the expired histogram. r=gfritzsche, a=lizzard MozReview-Commit-ID: IH4BCDq8fgz
d47ec9fc2586: Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche
Aaron Klotz <aklotz@mozilla.com> - Thu, 16 Nov 2017 18:10:41 -0700 - rev 451218
Push 8551 by ryanvm@gmail.com at 2018-01-17 15:17 +0000
Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche MozReview-Commit-ID: 3Cw7XIrROTn
1f1f863ea198: Bug 1424760 (Part 4) - Try to make GC telemetry pings more robust r=gfritzsche
Paul Bone <pbone@mozilla.com> - Thu, 11 Jan 2018 14:43:33 +1100 - rev 451200
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1424760 (Part 4) - Try to make GC telemetry pings more robust r=gfritzsche + Log limitProperties failures. + Don't remove all the properties from the main GC object, some are needed later, just reduce it to the 7 necessary properties.
b4ecd0f0fc8d: Bug 1424760 (Part 3) - Add a TelemetryGC unit test r=gfritzsche
Paul Bone <pbone@mozilla.com> - Tue, 16 Jan 2018 10:46:00 +1100 - rev 451199
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1424760 (Part 3) - Add a TelemetryGC unit test r=gfritzsche toolkit/components/telemetry/tests/unit/test_TelemetryGC.js: toolkit/components/telemetry/tests/unit/xpcshell.ini: Add new unit test. toolkit/components/telemetry/GCTelemetry.jsm: Add an extra method to make testing easier. js/src/gc/Statistics.cpp: Add a reference to the unit test for anyone modifying this code.
aee5e05a7e0c: Bug 1424760 (Part 1) - Fix an off-by-one error r=gfritzsche
Paul Bone <pbone@mozilla.com> - Sat, 16 Dec 2017 02:35:25 +1100 - rev 451197
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1424760 (Part 1) - Fix an off-by-one error r=gfritzsche limitProperties would reject payloads with the maximum number of properties, when it should accept these and only reject when they exceed the maximum.
8962ed624e0c: Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche
Aaron Klotz <aklotz@mozilla.com> - Thu, 16 Nov 2017 18:10:41 -0700 - rev 451162
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche MozReview-Commit-ID: 3Cw7XIrROTn
073eb94e1d21: Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche
Aaron Klotz <aklotz@mozilla.com> - Thu, 16 Nov 2017 18:10:41 -0700 - rev 451151
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche MozReview-Commit-ID: 3Cw7XIrROTn
75591ea83b3c: Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche
Aaron Klotz <aklotz@mozilla.com> - Thu, 16 Nov 2017 18:10:41 -0700 - rev 451114
Push 8543 by ryanvm@gmail.com at 2018-01-16 14:33 +0000
Bug 1418131: Part 3 - Add Windows Security Center info to telemetry environment; r=gfritzsche MozReview-Commit-ID: 3Cw7XIrROTn
c2e1ad141fd7: Bug 1408139 - Remove probes: NEWTAB_PAGE_SHOWN, NEWTAB_PAGE_SITE_CLICKED, NEWTAB_PAGE_LIFE_SPAN. r=chutten,gfritzsche
Andrei Oprea <andrei.br92@gmail.com> - Fri, 08 Dec 2017 17:29:27 +0100 - rev 449953
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1408139 - Remove probes: NEWTAB_PAGE_SHOWN, NEWTAB_PAGE_SITE_CLICKED, NEWTAB_PAGE_LIFE_SPAN. r=chutten,gfritzsche PAGE_SHOW, SITE_CLICKED expired in version 35 and can safely be removed. LIFE_SPAN was triggering alerts because Activity Stream is the new default about:newtab and data for it is no longer being collected. MozReview-Commit-ID: 3TIG5jwRfDr
50d298fe8fb6: Bug 1417473 - Add the docs for hybrid content telemetry. r=chutten,gfritzsche
Alessio Placitelli <alessio.placitelli@gmail.com> - Wed, 29 Nov 2017 16:37:30 +0100 - rev 449731
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1417473 - Add the docs for hybrid content telemetry. r=chutten,gfritzsche MozReview-Commit-ID: 2qEiZcNU7SB
a365a5a59ad1: Bug 1417473 - Add the docs for hybrid content telemetry. r=chutten,gfritzsche
Alessio Placitelli <alessio.placitelli@gmail.com> - Wed, 29 Nov 2017 16:37:30 +0100 - rev 449700
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1417473 - Add the docs for hybrid content telemetry. r=chutten,gfritzsche MozReview-Commit-ID: 2qEiZcNU7SB
2e3bcb14cc4e: Bug 1422337 - Remove this test r=gfritzsche
Paul Bone <pbone@mozilla.com> - Thu, 04 Jan 2018 17:03:36 +1100 - rev 449680
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1422337 - Remove this test r=gfritzsche I think it should be okay that there are no GCs in a telemetry ping.
7a1a46675739: Bug 1352497 - Remove about:healthreport. r=gfritzsche,nechen
Dão Gottwald <dao@mozilla.com> - Tue, 28 Nov 2017 11:38:15 +0100 - rev 445698
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1352497 - Remove about:healthreport. r=gfritzsche,nechen MozReview-Commit-ID: 4FQ5aL2XrU5
be86ccde4f4a: Bug 1352497 - Remove about:healthreport. r=gfritzsche,nechen
Dão Gottwald <dao@mozilla.com> - Wed, 22 Nov 2017 13:51:08 +0100 - rev 445329
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1352497 - Remove about:healthreport. r=gfritzsche,nechen MozReview-Commit-ID: 8aPg9oDFIlu
f9122f1e3ab9: Bug 1419788 - Add Telemetry probe for browser.search.region, r=gfritzsche.
Florian Quèze <florian@queze.net> - Thu, 23 Nov 2017 12:15:41 +0100 - rev 445160
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1419788 - Add Telemetry probe for browser.search.region, r=gfritzsche.
8fa1f53e62a2: Bug 1412729 (part 1) - Update out-of-date comments r=gfritzsche
Paul Bone <pbone@mozilla.com> - Fri, 10 Nov 2017 23:42:04 +1100 - rev 444477
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1412729 (part 1) - Update out-of-date comments r=gfritzsche
5fa423b95ce4: Bug 1415340: Remove undefined (and unnecessary) property when rejecting a Promise in GMPDownloader. r=gfritzsche
Stephen A Pohl <spohl.mozilla.bugs@gmail.com> - Tue, 14 Nov 2017 10:49:12 -0500 - rev 443485
Push 8527 by Callek@gmail.com at 2018-01-11 21:05 +0000
Bug 1415340: Remove undefined (and unnecessary) property when rejecting a Promise in GMPDownloader. r=gfritzsche
1e9b49d67686: Bug 1419788 - Add Telemetry probe for browser.search.region, r=gfritzsche a=sylvestre
Florian Quèze <florian@queze.net> - Thu, 23 Nov 2017 12:15:41 +0100 - rev 442456
Push 8213 by sledru@mozilla.com at 2017-11-23 14:23 +0000
Bug 1419788 - Add Telemetry probe for browser.search.region, r=gfritzsche a=sylvestre
463e74189171: Bug 1386511 - Part 2: Improve style in TelemetryGC test. r=gfritzsche
Paul Bone <pbone@mozilla.com> - Tue, 31 Oct 2017 14:42:44 +1100 - rev 440685
Push 8120 by ryanvm@gmail.com at 2017-11-04 17:45 +0000
Bug 1386511 - Part 2: Improve style in TelemetryGC test. r=gfritzsche
fc96bf1b72ce: Bug 1406783 - texts for radio options to select ping data on about:telemetry should select option after clicked. r=gfritzsche
Sebastian Hengst <archaeopteryx@coole-files.de> - Sun, 08 Oct 2017 20:31:04 +0200 - rev 436415
Push 8114 by jlorenzo@mozilla.com at 2017-11-02 16:33 +0000
Bug 1406783 - texts for radio options to select ping data on about:telemetry should select option after clicked. r=gfritzsche MozReview-Commit-ID: AmJM953FIBT
2deaefe29741: Bug 1400675 - Fix reference to undefined property FirsRunURL. r=gfritzsche, a=lizzard FIREFOX_56b13_RELBRANCH
Tobias Markus <tobbi.bugs@googlemail.com> - Mon, 18 Sep 2017 11:01:37 +0200 - rev 431389
Push 7783 by ryanvm@gmail.com at 2017-09-21 00:21 +0000
Bug 1400675 - Fix reference to undefined property FirsRunURL. r=gfritzsche, a=lizzard MozReview-Commit-ID: ACfUm7KqWuY
84bc19f5cbe5: Bug 1400675 - Fix reference to undefined property FirsRunURL r=gfritzsche
Tobias Markus <tobbi.bugs@googlemail.com> - Mon, 18 Sep 2017 11:01:37 +0200 - rev 431065
Push 7777 by ryanvm@gmail.com at 2017-09-19 13:41 +0000
Bug 1400675 - Fix reference to undefined property FirsRunURL r=gfritzsche MozReview-Commit-ID: ACfUm7KqWuY
d543ff3bc906: Bug 1398851 - Temporarily skip test_TelemetrySend.js on Win64 debug due to frequent failures. r=gfritzsche
Ryan VanderMeulen <ryanvm@gmail.com> - Tue, 12 Sep 2017 09:00:36 -0400 - rev 429783
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1398851 - Temporarily skip test_TelemetrySend.js on Win64 debug due to frequent failures. r=gfritzsche
447ae1ff1b04: Bug 1397376 - Avoid copying on initializing histogram ranges r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Fri, 08 Sep 2017 12:13:44 -0700 - rev 429655
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397376 - Avoid copying on initializing histogram ranges r=gfritzsche Since LinearHistogram and its descendants inherit ranges_ from Histogram, and we wanted to replace the copying into a std::vec for Histogram, the simplest approach seemed to just be to precompute ranges for all histograms, exponential or otherwise. This should have the added benefit of reducing the memory footprint for those histograms, since they will benefit from the deduplication work that the precomputing script already does. MozReview-Commit-ID: JTV5Dej5ZIb
ba149daae44e: Bug 1397376 - Reduce size of gExponentialBucketLowerBoundIndex r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Thu, 07 Sep 2017 10:23:52 -0700 - rev 429654
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397376 - Reduce size of gExponentialBucketLowerBoundIndex r=gfritzsche This is a fairly small optimization - since the indices for this array never exceed the size of an int16_t, let's just use that instead to save a little bit of space. MozReview-Commit-ID: 8bRokjlvZ9p
34d4d0192028: Bug 1397376 - Deduplicate histogram static bucket data r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Thu, 07 Sep 2017 10:22:53 -0700 - rev 429653
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397376 - Deduplicate histogram static bucket data r=gfritzsche Since we already have the indices array, we can just point duplicate ranges at the first occurrence's index. MozReview-Commit-ID: 3f5os1xSp89
5e511ebfe505: bug 1373240 - Use somewhat-more-dynamically-allocated histogram storage r=Dexter,gfritzsche
Chris H-C <chutten@mozilla.com> - Tue, 29 Aug 2017 15:05:35 -0400 - rev 429230
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
bug 1373240 - Use somewhat-more-dynamically-allocated histogram storage r=Dexter,gfritzsche Switch from static multi-dimensional arrays to dynamic one-dimensional arrays that are only allocated in the parent process. MozReview-Commit-ID: tyGEFhU2Fq
0181bf16af4f: bug 1373240 - Use somewhat-more-dynamically-allocated histogram storage r=Dexter,gfritzsche
Chris H-C <chutten@mozilla.com> - Tue, 29 Aug 2017 15:05:35 -0400 - rev 429205
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
bug 1373240 - Use somewhat-more-dynamically-allocated histogram storage r=Dexter,gfritzsche Switch from static multi-dimensional arrays to dynamic one-dimensional arrays that are only allocated in the parent process. MozReview-Commit-ID: tyGEFhU2Fq
827fae5a03b4: Bug 1396020 - Expose hard-coded strings in about:telemetry r=gfritzsche
Francesco Lodolo (:flod) <flod@lodolo.net> - Sat, 02 Sep 2017 09:27:12 +0200 - rev 428272
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1396020 - Expose hard-coded strings in about:telemetry r=gfritzsche MozReview-Commit-ID: 1ddOE9cGX4Z
53a7f7b59fbd: Bug 1390269 - Convert TOUCH_ENABLED_DEVICE to an opt-out probe. r=gfritzsche, data=rweiss
Jim Mathies <jmathies@mozilla.com> - Fri, 01 Sep 2017 16:21:18 -0500 - rev 427898
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1390269 - Convert TOUCH_ENABLED_DEVICE to an opt-out probe. r=gfritzsche, data=rweiss MozReview-Commit-ID: CSqpYQlOJpt
17b77db08079: Bug 1343855 - Part 2 - Add support for key whitelists in keyed histograms. r=gfritzsche, data-review=francois
Alessio Placitelli <alessio.placitelli@gmail.com> - Wed, 17 May 2017 04:10:00 +0200 - rev 426344
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1343855 - Part 2 - Add support for key whitelists in keyed histograms. r=gfritzsche, data-review=francois MozReview-Commit-ID: H2pE3VivWIX
b23d59dbab75: Bug 1386186 - Added firstPaint scalar, timestamps scalar section, and tests. r=gfritzsche
Vanessa Gutierrez <vgutierrez@mozilla.com> - Tue, 22 Aug 2017 11:01:00 -0400 - rev 426068
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1386186 - Added firstPaint scalar, timestamps scalar section, and tests. r=gfritzsche
e240d265b038: Bug 1390441 - Fix usePingSenderOnShutdown test via reseting TelemetrySend in tests. r=gfritzsche
Kate Ustiuzhanina <kustiuzhanina@mozilla.com> - Tue, 22 Aug 2017 15:28:18 +0100 - rev 425688
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1390441 - Fix usePingSenderOnShutdown test via reseting TelemetrySend in tests. r=gfritzsche
e8a5813bc9b9: Bug 1386619 - Add documentation for 'update' pings edge cases. r=gfritzsche
Alessio Placitelli <alessio.placitelli@gmail.com> - Mon, 21 Aug 2017 12:00:36 +0200 - rev 425417
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1386619 - Add documentation for 'update' pings edge cases. r=gfritzsche MozReview-Commit-ID: 77Ahi9wr76O
9430ca9565da: Bug 1385351 - Remove unused old class r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Sat, 12 Aug 2017 00:23:19 +0200 - rev 425402
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385351 - Remove unused old class r=gfritzsche MozReview-Commit-ID: 4Q7oX2zF4el
bc3d6103ff79: Bug 1385351 - Emphasize the ping selector button in about:telemetry r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 00:58:55 +0200 - rev 425401
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385351 - Emphasize the ping selector button in about:telemetry r=gfritzsche Add a triangle next to the ping selection button that emphasize the to the current ping selection popup state. MozReview-Commit-ID: GOzv01S0i9h
493584127e47: Bug 1389750 - Fix about:telemetry photon design r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Mon, 14 Aug 2017 22:16:50 +0200 - rev 424296
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1389750 - Fix about:telemetry photon design r=gfritzsche MozReview-Commit-ID: DN7MtZT7Va5
47cf76ef4d3d: Bug 1385270 - Fix about:telemetry subsection display r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 01:16:02 +0200 - rev 424291
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385270 - Fix about:telemetry subsection display r=gfritzsche Show all subsection after comming back from another section. MozReview-Commit-ID: I02cR2IaBrm
84e7b3d7c7cf: Bug 1385351 - Remove unused old class r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Sat, 12 Aug 2017 00:23:19 +0200 - rev 424114
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385351 - Remove unused old class r=gfritzsche MozReview-Commit-ID: 4Q7oX2zF4el
c89766a47689: Bug 1385351 - Emphasize the ping selector button in about:telemetry r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 00:58:55 +0200 - rev 424113
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385351 - Emphasize the ping selector button in about:telemetry r=gfritzsche Add a triangle next to the ping selection button that emphasize the to the current ping selection popup state. MozReview-Commit-ID: GOzv01S0i9h
6beb0e4ddfd1: Bug 1388695 - Show the payload for non main ping in about:telemetry r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 13:29:56 +0200 - rev 423820
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388695 - Show the payload for non main ping in about:telemetry r=gfritzsche Add a raw payload section when looking at a non main ping. MozReview-Commit-ID: 2dU8apRfzLP
454fabe957f2: Bug 1389151 - Show the home section for non main ping in about:telemetry r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Thu, 10 Aug 2017 18:52:17 +0200 - rev 423803
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1389151 - Show the home section for non main ping in about:telemetry r=gfritzsche MozReview-Commit-ID: J9oFy0YDI5z
0d9cf59817bb: Bug 1388691 - Use the firefox json viewer in about:telemetry r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 13:04:28 +0200 - rev 423546
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388691 - Use the firefox json viewer in about:telemetry r=gfritzsche Allow to open the full ping data in the firefox json viewer. MozReview-Commit-ID: C8rqxmY3qtT
f636e51b75dd: Bug 1388748 - Don't resize histogram ranges on init r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Wed, 09 Aug 2017 10:21:09 -0700 - rev 423542
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388748 - Don't resize histogram ranges on init r=gfritzsche We were using std::vector::assign, which resizes the vector to match the incoming data. This isn't what we want, as ranges_ has already been sized to bucket_count_ + 1. Instead, just use a copy. MozReview-Commit-ID: EGuW5jj7Rpq
2d0086d41415: Bug 1376605 - Refactor process payload assembly. r=gfritzsche
Anthony Miyaguchi <amiyaguchi@mozilla.com> - Mon, 07 Aug 2017 19:09:04 +0200 - rev 423503
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1376605 - Refactor process payload assembly. r=gfritzsche
b356aa6fa0ef: Bug 1388456 - Make the ping selection in about:telemetry clearer r=gfritzsche
flyingrub <flyinggrub@gmail.com> - Wed, 09 Aug 2017 01:31:28 +0200 - rev 423500
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388456 - Make the ping selection in about:telemetry clearer r=gfritzsche Show the ping selector popup next to its toggle button in about:telemetry. Underline the fact that no archived pings are available. MozReview-Commit-ID: CwSLOvuhXFn
b8e78ebe648e: Bug 1383210 - Use precomputed histogram buckets r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Fri, 04 Aug 2017 10:02:28 -0700 - rev 422976
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1383210 - Use precomputed histogram buckets r=gfritzsche The log and exp calls in base::Histogram::InitializeBucketRange() were showing up in profiles. This patch uses the precomputed buckets for exponential histograms instead of computing them at runtime. Though linear histograms do show up in the profile that prompted this change, they contribute much less, and due to the trivial nature of generating these, it's unlikely that a static cache would provide much if any speedup. MozReview-Commit-ID: IavFwoWjFhk
277c7e4952a8: Bug 1383210 - Use precomputed histogram buckets r=gfritzsche
Doug Thayer <dothayer@mozilla.com> - Fri, 04 Aug 2017 10:02:28 -0700 - rev 422965
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1383210 - Use precomputed histogram buckets r=gfritzsche The log and exp calls in base::Histogram::InitializeBucketRange() were showing up in profiles. This patch uses the precomputed buckets for exponential histograms instead of computing them at runtime. Though linear histograms do show up in the profile that prompted this change, they contribute much less, and due to the trivial nature of generating these, it's unlikely that a static cache would provide much if any speedup. MozReview-Commit-ID: IavFwoWjFhk