c88a7744a050eeb9597fedd9acf4836cbc9edecd: Bug 1393399 P1 - add telemetry probes; r=francois,gerald
Kaku Kuo <kaku@mozilla.com> - Fri, 01 Sep 2017 11:00:44 +0800 - rev 420899
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1393399 P1 - add telemetry probes; r=francois,gerald Here, we calculate the recovery time from two different start points because that when GPU crashes, VideoDecoderChild::ActorDestory() is called, but VideoDecoderChild doesn't dispatch events to trigger decoder recreation immediately. Instead, it waits until a new VideoDecoderManager has been recreated [1], and then rejects all pending promises to MFR with a NS_ERROR_DOM_MEDIA_NEED_NEW_DECODER, which triggers playback pipeline recreation. These two probes help us to know the gap between "GPU crash" and "MFR is notified". We don't need to wait for VideoDecoderManager recreation if we're certain that we don't create remote video decoder anymore. And if the gap is large, it might be a significant optimization. [1] http://searchfox.org/mozilla-central/rev/51b3d67a5ec1758bd2fe7d7b6e75ad6b6b5da223/dom/media/ipc/VideoDecoderChild.cpp#124-145 MozReview-Commit-ID: DcI2CwWORxZ
38a6312f4f8c5aebb4b6ca1d205e1992b434b780: Bug 1393343 - move 'restore previous session' item into main menu and hide when auto-restore is enabled, r=mikedeboer
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Fri, 25 Aug 2017 11:40:49 +0100 - rev 420898
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1393343 - move 'restore previous session' item into main menu and hide when auto-restore is enabled, r=mikedeboer MozReview-Commit-ID: B3J9c936RL
1f29a8f5c69dd6b0937c3368b7d088454b8e65d1: servo: Merge #18351 - Refactor `.sort_by()` calls to use `.partial_cmp()` (from mateon1:profile-sort_by-partial_cmp); r=emilio
Mateusz Naściszewski <matin1111@wp.pl> - Sat, 02 Sep 2017 03:54:49 -0500 - rev 420897
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
servo: Merge #18351 - Refactor `.sort_by()` calls to use `.partial_cmp()` (from mateon1:profile-sort_by-partial_cmp); r=emilio Changes the closures passed to `sort_by` in this file with a simpler, and more correct version. Previously, potential NaNs in the array would float to the top. Either way, the program would crash, as the `get_statistics` function asserts the array it gets is sorted, which always fails with a NaN. Because of that, this change should not affect functionality. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./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 statistics collected by --profile should not have NaN values in the first place. <!-- 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: 7dcd3ae50d090170fec4217248cc7864a1d2413a
4fb65ac5a58719f8afe0fe5e4056f7e8f40ad846: Bug 1374815 - fix sidebar hover styling and unify photon styling with the default, r=nhnt11
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Mon, 21 Aug 2017 17:37:27 +0100 - rev 420896
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1374815 - fix sidebar hover styling and unify photon styling with the default, r=nhnt11 This consolidates the photon styles into the old pre-photon styles, simplifying the CSS. As a result, it fixes the layout of the bookmarks menu as well. There's still some work left to do there to make submenus align properly with their parent once opened on Windows, but that isn't a regression caused by this bug so I am not addressing it here. I have deliberately not updated the checkmark besides aligning it properly on OS X - it seems we need more design details as to how that should look in the sidebar header menu as well as the bookmarks panel before changing that further (if indeed it does need further changes). MozReview-Commit-ID: JPl2VwUh3e1
f62dccfbc68e420b1a7538af09374e7edf446b25: Bug 1395527 part 2 - Delegate IsASCII and IsUTF8 to encoding_rs. r=froydnj
Henri Sivonen <hsivonen@hsivonen.fi> - Fri, 01 Sep 2017 11:53:31 +0300 - rev 420895
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1395527 part 2 - Delegate IsASCII and IsUTF8 to encoding_rs. r=froydnj MozReview-Commit-ID: 49AGBPjW4Ca
06b701902e64ee55d8ce677ceee6db7fa7a1e44d: Bug 1395527 part 1 - Add microbenchmarks for IsASCII and IsUTF8. r=froydnj
Henri Sivonen <hsivonen@hsivonen.fi> - Thu, 31 Aug 2017 14:46:25 +0300 - rev 420894
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1395527 part 1 - Add microbenchmarks for IsASCII and IsUTF8. r=froydnj MozReview-Commit-ID: 3oploVISQ3F
1fdc9b1dbb9e5b4f4ef03934dfa1827f3581d725: merge mozilla-central to autoland. r=merge a=merge
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 02 Sep 2017 10:57:13 +0200 - rev 420893
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
merge mozilla-central to autoland. r=merge a=merge
b24a12ef6a1d0a5e00b3f72d5c6988070acdc0ec: servo: Merge #18350 - stylo: Expose -moz-system-metric and -moz-empty-except-children-with-localname (from emilio:ship-system-metric-stuff); r=bholley,manishearth
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 02 Sep 2017 02:49:29 -0500 - rev 420892
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
servo: Merge #18350 - stylo: Expose -moz-system-metric and -moz-empty-except-children-with-localname (from emilio:ship-system-metric-stuff); r=bholley,manishearth They don't have the flag in Gecko. Though I wonder if we could unship them from non-UA sheets. Bug: 1396048 Reviewed-by: bholley MozReview-Commit-ID: LGzGDjCZpJC Source-Repo: https://github.com/servo/servo Source-Revision: e0e09538fa97f5568c52266ee969ec21432e72f7
7a5f1092920acfd6bf70e055e40c96084185f258: Backed out changeset 820cb0437789 (bug 1394302) for failing new test, at least on Linux x64. r=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 02 Sep 2017 10:42:45 +0200 - rev 420891
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Backed out changeset 820cb0437789 (bug 1394302) for failing new test, at least on Linux x64. r=backout
e990298e1596c5d5df0a73c929c0dacef88f5e53: Backed out changeset 40934888ed06 (bug 1395527) for Android bustage at uriloader/exthandler/android/nsMIMEInfoAndroid.cpp:62: too many arguments to function 'bool IsUTF8(const nsACString&)'. r=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 02 Sep 2017 10:39:06 +0200 - rev 420890
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Backed out changeset 40934888ed06 (bug 1395527) for Android bustage at uriloader/exthandler/android/nsMIMEInfoAndroid.cpp:62: too many arguments to function 'bool IsUTF8(const nsACString&)'. r=backout
cae6eeaf3f0c73cb62b87f465e9dd69ea2132bd3: Backed out changeset d59b62713c66 (bug 1395527)
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 02 Sep 2017 10:38:20 +0200 - rev 420889
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Backed out changeset d59b62713c66 (bug 1395527)
d59b62713c66023954833e89374c5ecc2b92df72: Bug 1395527 part 2 - Delegate IsASCII and IsUTF8 to encoding_rs. r=froydnj
Henri Sivonen <hsivonen@hsivonen.fi> - Fri, 01 Sep 2017 11:53:31 +0300 - rev 420888
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1395527 part 2 - Delegate IsASCII and IsUTF8 to encoding_rs. r=froydnj MozReview-Commit-ID: 49AGBPjW4Ca
40934888ed06465982350390750128f1dbdc831b: Bug 1395527 part 1 - Add microbenchmarks for IsASCII and IsUTF8. r=froydnj
Henri Sivonen <hsivonen@hsivonen.fi> - Thu, 31 Aug 2017 14:46:25 +0300 - rev 420887
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1395527 part 1 - Add microbenchmarks for IsASCII and IsUTF8. r=froydnj MozReview-Commit-ID: 3oploVISQ3F
7c4c81a8b6996831b41b7a4865a7d7d04188887a: Bug 1393054 - Remove extra box-shadow from urlbar. r=dao
Dale Harvey <dale@arandomurl.com> - Fri, 01 Sep 2017 16:23:50 +0100 - rev 420886
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1393054 - Remove extra box-shadow from urlbar. r=dao MozReview-Commit-ID: GH1LAfxQ2kQ
230d2e70112d007d0f00711030c66ed7c295c314: servo: Merge #18340 - Use the better TokenStream API in dom_struct (from servo:dom-struct-errors); r=SimonSapin
Anthony Ramine <n.oxyde@gmail.com> - Sat, 02 Sep 2017 00:48:29 -0500 - rev 420885
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
servo: Merge #18340 - Use the better TokenStream API in dom_struct (from servo:dom-struct-errors); r=SimonSapin Before: ```rust error[E0412]: cannot find type `SourceBuffer` in this scope --> /Users/nox/src/servo/components/script/dom/mediasource.rs:25:1 | 25 | #[dom_struct] | ^^^^^^^^^^^^^ did you mean `SourceBufferList`? ``` After: ```rust error[E0412]: cannot find type `SourceBuffer` in this scope --> /Users/nox/src/servo/components/script/dom/mediasource.rs:28:39 | 28 | source_buffers: DOMRefCell<Vec<JS<SourceBuffer>>>, | ^^^^^^^^^^^^ did you mean `SourceBufferList`? ``` Source-Repo: https://github.com/servo/servo Source-Revision: 32b04b3f517cebcba34a23e6930860bbedb1431b
820cb0437789ef1d3c2aa054e803fa0203b45090: Bug 1394302 - stylo: Add mochitest; r=birtles
Manish Goregaokar <manishearth@gmail.com> - Thu, 31 Aug 2017 17:26:51 -0700 - rev 420884
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1394302 - stylo: Add mochitest; r=birtles MozReview-Commit-ID: DJ3zFqqMNXk
e68ecb05643fd4e722ae39e431a2b2d0d8d095d1: Bug 1391169 Part 2: Add a test of cssText with combinations of namespaces, universal selectors, and combinators. r=SimonSapin
Brad Werth <bwerth@mozilla.com> - Tue, 22 Aug 2017 16:21:46 -0700 - rev 420883
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1391169 Part 2: Add a test of cssText with combinations of namespaces, universal selectors, and combinators. r=SimonSapin MozReview-Commit-ID: GZZEGR0zyp6
06223c54aef5313881a04f663c47c9ae48e05f41: servo: Merge #18344 - Change selector::to_css function to handle combinators in between universal selectors (from bradwerth:selectorSerialize); r=SimonSapin
Brad Werth <bwerth@mozilla.com> - Fri, 01 Sep 2017 23:41:10 -0500 - rev 420882
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
servo: Merge #18344 - Change selector::to_css function to handle combinators in between universal selectors (from bradwerth:selectorSerialize); r=SimonSapin https://bugzilla.mozilla.org/show_bug.cgi?id=1391169 https://reviewboard.mozilla.org/r/172368/ --- <!-- 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: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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: fd95696d31c4d7964d1d008d839d4a79066ce7f1
1ac1cb838d99d5ebf4e0897fc69a46978f5a3723: Backed out changeset d2305bd57a97 (bug 1393140) for changed behavior on request from jorgk. r=backout a=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 02 Sep 2017 23:48:23 +0200 - rev 420881
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Backed out changeset d2305bd57a97 (bug 1393140) for changed behavior on request from jorgk. r=backout a=backout MozReview-Commit-ID: HGulTnPx6TF
74b0ceebc6d142547b5c55c3d7b2550478c000c1: Bug 1395997 - Use the photon styles for the DevTools splitter color. r=pbro
Gabriel Luong <gabriel.luong@gmail.com> - Sat, 02 Sep 2017 13:40:09 -0400 - rev 420880
Push 81 by fmarier@mozilla.com at Tue, 05 Sep 2017 20:09:12 +0000
Bug 1395997 - Use the photon styles for the DevTools splitter color. r=pbro
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip