092857175e36c1c9c0d8d74436f432dc57deba11: Bug 1400096 - Annotate mozmemory_wrap.c windows functions as MOZ_MEMORY_API. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 15 Sep 2017 10:12:24 +0900 - rev 430588
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1400096 - Annotate mozmemory_wrap.c windows functions as MOZ_MEMORY_API. r=njn
It happens to work because of mozglue.def, but we might as well have the
right annotations (which will also make things correct when building this
file to C++)
6cf651d919d1aa049d0b88c1ca8f91c0c33ee56b: Bug 1400096 - Remove mozmem_malloc_impl wrapping for operator methods exposed in mozmemory_wrap.c. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 15 Sep 2017 09:54:01 +0900 - rev 430587
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1400096 - Remove mozmem_malloc_impl wrapping for operator methods exposed in mozmemory_wrap.c. r=njn
This used to be necessary because those functions might be prefixed with
__wrap_, and linked against with -Wl,--wrap, but that's not been the case
since
bug 1077366. Furthermore, mozmem_malloc_impl nowadays only does
something on Windows, and those operators are only exposed on Android.
ba8d3ecc2412f0025e715a8ecd9a527ca44ef5e0: Bug 1397307 - P10. Remove uncessary loop. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 14 Sep 2017 14:45:10 +0200 - rev 430586
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P10. Remove uncessary loop. r=gerald
We only process a demuxed sample at a time. Waiting until one is decoded to do the next pending ones.
MozReview-Commit-ID: JlXhyPzso8U
7558fb4b16997621dab4b82e3dd0a5d81e0c4488: Bug 1397307 - P9. Pass video frame rate to RemoteVideoDecoder and GPU process. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 18:29:40 +0200 - rev 430585
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P9. Pass video frame rate to RemoteVideoDecoder and GPU process. r=mattwoodrow
MozReview-Commit-ID: BDSO332f3B6
0cf29b887325d88990f8c6b3ff77ba28e416424a: Bug 1397307 - P8. Pass averaged video frame rate to constructor. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 17:55:03 +0200 - rev 430584
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P8. Pass averaged video frame rate to constructor. r=mattwoodrow
MozReview-Commit-ID: FMFz3RdFsHA
1df89fb6ff5d24981157557a82de345b6da3f100: Bug 1397307 - P7. Display video resolution and frame rate in debug data. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 17:40:42 +0200 - rev 430583
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P7. Display video resolution and frame rate in debug data. r=gerald
MozReview-Commit-ID: 9vsheKkAm4p
3fbbe2c4ec7be1057bbca4eab35bfbb4bad8de9c: Bug 1397307 - P6. Calculate average video frame rate as video is playing. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 21:20:09 +0200 - rev 430582
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P6. Calculate average video frame rate as video is playing. r=gerald
We unfortunately can't store this information in the VideoInfo as typically the framerate isn't found in the container's metadata. Additionally, the VideoInfo object is readable-only as it is shared across threads.
As such, we can only estimate it as we demux samples.
MozReview-Commit-ID: 5HB33ubfGAs
768bf2920dab786813f9a698bd5c5b1778f09508: Bug 1397307 - P5. Avoid creating two decoders on first sample. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 21:02:24 +0200 - rev 430581
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P5. Avoid creating two decoders on first sample. r=gerald
Don't unnecessarily, create a decoder, flush, shutdown and create a new one on the first sample.
MozReview-Commit-ID: 8utEX5JEmq8
3fa8d29b3026c85ee6b0e956e2db1a490d0aefc7: Bug 1397307 - P4. Fix style. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 12 Sep 2017 11:09:15 +0200 - rev 430580
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P4. Fix style. r=gerald
MozReview-Commit-ID: 1Q3kwsDlAhI
9d597b2bcf3518432ec8325b07b0a8b85b1666ca: Bug 1397307 - P3. Remove unused method. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 11 Sep 2017 17:37:44 +0200 - rev 430579
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P3. Remove unused method. r=gerald
Code was incorrect anyway.
MozReview-Commit-ID: Bf0O4Mhu1P6
70899b90b6fe4a1be0650318a14ad40bcb604948: Bug 1397307 - P2. Wrap boolean in structure to prevent unwanted conversion. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Sep 2017 15:45:38 +0200 - rev 430578
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P2. Wrap boolean in structure to prevent unwanted conversion. r=gerald
Automatic conversion (say from int to bool) makes DecoderParam difficult to extend.
MozReview-Commit-ID: G0T7jPogskN
c60b5da550ea3b664a9a81fb5dd9708bf7491657: Bug 1397307 - P1. Make method const. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Sep 2017 12:49:01 +0200 - rev 430577
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1397307 - P1. Make method const. r=gerald
MozReview-Commit-ID: 2UrTZroXpRG
1536fa69bad4d6ee7f1f4c265fab57504f6263d9: Bug 1399758 - Measure ImageValue objects. r=heycam.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 14 Sep 2017 18:48:19 +1000 - rev 430576
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1399758 - Measure ImageValue objects. r=heycam.
We have about 11,500 of these when loading gmail in a Stylo-enabled build, from
SpecifiedUrls; the objects themselves account for about 1.3 MiB of memory, and the
strings within them about 2.9 MiB.
We also have a very small number of them on the Gecko side.
MozReview-Commit-ID: AduCIaDIzGG
e2f8c9f76b711c89b04ca0e2f2ac324e638ac704: servo: Merge #18515 - Measure ImageValue objects (from nnethercote:bug-1399758); r=heycam
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 14 Sep 2017 22:06:43 -0500 - rev 430575
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
servo: Merge
#18515 - Measure ImageValue objects (from nnethercote:bug-1399758); r=heycam
We have about 11,500 of these when loading gmail in a Stylo-enabled build, from
SpecifiedUrls; the objects themselves account for about 1.3 MiB of memory, and
the strings within them about 2.9 MiB.
We also have a very small number of them on the Gecko side.
<!-- Please describe your changes on the following line: -->
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #__ (github issue number if applicable).
<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because tested on Gecko side.
<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
Source-Repo:
https://github.com/servo/servo
Source-Revision:
db689094f1413f542961cbe0d19a3ed82a11f682
f2aa9d36be3f30e79a76277ab5dc88e2ed7c7d64: Bug 1384608 Show complete themes with legacy extensions r=rhelmer
Andrew Swan <aswan@mozilla.com> - Wed, 13 Sep 2017 13:23:39 -0700 - rev 430574
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1384608 Show complete themes with legacy extensions r=rhelmer
MozReview-Commit-ID: 6jiQZ8SQpzI
eae22389b267d0603db9162d878881505dd94519: Bug 1393150 prevent remote extensions when e10s is off, r=bz,kmag
Shane Caraveo <scaraveo@mozilla.com> - Thu, 14 Sep 2017 15:12:45 -0700 - rev 430573
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1393150 prevent remote extensions when e10s is off, r=bz,kmag
MozReview-Commit-ID: HjLLa9vx2UW
004c28e9a39e1d804e7d4ce62a7cbfcc231c89b9: servo: Merge #18504 - Some minor HTMLMediaElement improvements (from servo:media); r=emilio
Anthony Ramine <n.oxyde@gmail.com> - Thu, 14 Sep 2017 20:51:51 -0500 - rev 430572
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
servo: Merge
#18504 - Some minor HTMLMediaElement improvements (from servo:media); r=emilio
Source-Repo:
https://github.com/servo/servo
Source-Revision:
1b9a5ea7197cb59f48b2010895ad2a98fef08bc4
f36106f3f52a9bb2769860d20568c589fc2498d0: Backed out changeset 0e349b74bfc6 (bug 1399921) for failure to thrive on Mac
Phil Ringnalda <philringnalda@gmail.com> - Thu, 14 Sep 2017 20:36:41 -0700 - rev 430571
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Backed out changeset 0e349b74bfc6 (
bug 1399921) for failure to thrive on Mac
MozReview-Commit-ID: HoBhxzIzn2d
3947ec46871535ef97a419fd6e4d891f796fa797: Bug 1400073 - Hide MOZ_debug from getSupportedExtensions. - r=daoshengmu
Jeff Gilbert <jgilbert@mozilla.com> - Thu, 14 Sep 2017 16:52:36 -0700 - rev 430570
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1400073 - Hide MOZ_debug from getSupportedExtensions. - r=daoshengmu
MozReview-Commit-ID: LFTrUOGLh36
edd825dd3b1f017ca18b1eb50e2d439e02425240: Bug 1398974: Follow-up: Skip adding request filters to favicon.ico requests. r=me
Kris Maglione <maglione.k@gmail.com> - Thu, 14 Sep 2017 19:06:52 -0700 - rev 430569
Push
7768 by ryanvm@gmail.com at Sat, 16 Sep 2017 16:13:49 +0000
Bug 1398974: Follow-up: Skip adding request filters to favicon.ico requests. r=me
We're getting relatively frequent intermittent shutdown crashes when stream
filters are added during ts_paint tests. Skipping favicon.ico stream filters
should prevent that until we have a more reliable fix.