searching for reviewer(jnicol)
f48ce1c53912aa61817dd88818ee11fd1ce49f35: Bug 1687219 - Update a bunch of telemetry expirations. r=jnicol
Jeff Muizelaar <jmuizelaar@mozilla.com> - Mon, 18 Jan 2021 09:05:02 +0000 - rev 563591
Push 38116 by malexandru@mozilla.com at Tue, 19 Jan 2021 03:30:20 +0000
Bug 1687219 - Update a bunch of telemetry expirations. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D102126
b4582fea9f92355b8ca61b0ff239d2d01beecf93: Bug 1686862 - Account for the new A8 glyphs texture in various places. r=jnicol,lsalzman
Markus Stange <mstange.moz@gmail.com> - Fri, 15 Jan 2021 16:03:18 +0000 - rev 563361
Push 38108 by ncsoregi@mozilla.com at Fri, 15 Jan 2021 21:53:19 +0000
Bug 1686862 - Account for the new A8 glyphs texture in various places. r=jnicol,lsalzman Differential Revision: https://phabricator.services.mozilla.com/D101862
98fb1ef4d6e13743bad4511d605b2d0c3e574fe6: Bug 1680769 - Add a test case. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 18 Dec 2020 09:27:04 +0000 - rev 561862
Push 38074 by csabou@mozilla.com at Mon, 04 Jan 2021 17:25:45 +0000
Bug 1680769 - Add a test case. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D98944
bf177e6625275332a646d7db8c2d6fbdd7a85e02: Bug 1682365 - Pt 5 - Refactor RenderTaskLocation r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Fri, 18 Dec 2020 22:23:40 +0000 - rev 561329
Push 38045 by ncsoregi@mozilla.com at Sat, 19 Dec 2020 09:33:21 +0000
Bug 1682365 - Pt 5 - Refactor RenderTaskLocation r=jnicol Once the new graph API is in place, it becomes possible to express an input dependency on a persistent target (for example, if wanting to read back from a picture cache tile for a mix-blend, or marking that a color target depends on a render task in a texture cache). To make that simpler to express, this patch adds a specific struct for render target locations that are persistent, and updates the surrounding code to use it. At the same time, introduce an Unallocated field for dynamic tasks that are not yet allocated, rather than using an Option. Differential Revision: https://phabricator.services.mozilla.com/D99305
e30c7d6f0717c2b84c1a5e51ad1608b85af946d0: Bug 1682365 - Pt 4 - Add RenderTaskGraphBuilder / RenderTaskGraph split r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Fri, 18 Dec 2020 22:23:40 +0000 - rev 561328
Push 38045 by ncsoregi@mozilla.com at Sat, 19 Dec 2020 09:33:21 +0000
Bug 1682365 - Pt 4 - Add RenderTaskGraphBuilder / RenderTaskGraph split r=jnicol This patch splits the graph building functionality into `RenderTaskGraphBuilder` and the graph querying code into the existing `RenderTaskGraph` struct. The Builder struct is retained frame to frame, which means there is no longer a need for the `RenderTaskGraphCounters` struct. The Graph struct is constructed per-frame by calling `end_pass` on the Builder. Although this doesn't do much different internally, it will make integration with the new task graph changes simpler. It also enforced during frame building when it is possible to add / query render tasks. A few unrelated tidy ups are included in this patch - mostly removing where the task graph is passed to from a few structs and methods that no longer require access to the graph. Differential Revision: https://phabricator.services.mozilla.com/D99297
1a59c94a034f49dc998a8d90de5c7f112a0f415b: Bug 1682365 - Pt 3 - Remove array textures from picture cache tiles. r=nical,jnicol
Glenn Watson <git@intuitionlibrary.com> - Fri, 18 Dec 2020 22:23:40 +0000 - rev 561327
Push 38045 by ncsoregi@mozilla.com at Sat, 19 Dec 2020 09:33:21 +0000
Bug 1682365 - Pt 3 - Remove array textures from picture cache tiles. r=nical,jnicol This patch makes picture cache tiles use normal textures instead of array textures. With this and the previous patch, WR no longer uses array textures at all (except when provided by the external image handler trait). Differential Revision: https://phabricator.services.mozilla.com/D99013
d7c981ef38fcddeb7f0adf18fc6ea978217a7b08: Bug 1679751 - Fix cargo test build. r=jnicol. CLOSED TREE
Nicolas Silva <nsilva@mozilla.com> - Fri, 18 Dec 2020 18:29:25 +0200 - rev 561257
Push 38044 by apavel@mozilla.com at Fri, 18 Dec 2020 21:38:02 +0000
Bug 1679751 - Fix cargo test build. r=jnicol. CLOSED TREE Reviewers: jnicol Reviewed By: jnicol Tags: #testing-approved Bug #: 1679751 Differential Revision: https://phabricator.services.mozilla.com/D100126
c250273ad967f07470a1420df55ab1629663f1ce: Bug 1679751 - More android 8 reftest fuzziness adjustments. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 18 Dec 2020 13:24:37 +0000 - rev 561249
Push 38044 by apavel@mozilla.com at Fri, 18 Dec 2020 21:38:02 +0000
Bug 1679751 - More android 8 reftest fuzziness adjustments. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D100104
a6b10fddf201f7ac8b4afffeca5c3ef889ff5857: Bug 1679751 - More reftest adjustments. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 18 Dec 2020 10:30:37 +0000 - rev 561236
Push 38043 by smolnar@mozilla.com at Fri, 18 Dec 2020 15:55:19 +0000
Bug 1679751 - More reftest adjustments. r=jnicol Once again, Android 8.0 Pixel2 debug being hidden in fuzzy queries means we don't see see them break. Differential Revision: https://phabricator.services.mozilla.com/D100088
6683c4e0bc96b2f5ea8d441cd190f54c57c5b644: Bug 1679751 - Fix reftest syntax error. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 18 Dec 2020 10:30:10 +0000 - rev 561235
Push 38043 by smolnar@mozilla.com at Fri, 18 Dec 2020 15:55:19 +0000
Bug 1679751 - Fix reftest syntax error. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D100087
66205d10a66a2892edb0740fd6c24b5f3e610ab2: Bug 1682365 - Pt 5 - Refactor RenderTaskLocation r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Thu, 17 Dec 2020 22:26:29 +0000 - rev 561198
Push 38042 by ccoroiu@mozilla.com at Fri, 18 Dec 2020 09:56:07 +0000
Bug 1682365 - Pt 5 - Refactor RenderTaskLocation r=jnicol Once the new graph API is in place, it becomes possible to express an input dependency on a persistent target (for example, if wanting to read back from a picture cache tile for a mix-blend, or marking that a color target depends on a render task in a texture cache). To make that simpler to express, this patch adds a specific struct for render target locations that are persistent, and updates the surrounding code to use it. At the same time, introduce an Unallocated field for dynamic tasks that are not yet allocated, rather than using an Option. Differential Revision: https://phabricator.services.mozilla.com/D99305
3187b453d85a5932e01b46c4d5c596a75c79c139: Bug 1682365 - Pt 4 - Add RenderTaskGraphBuilder / RenderTaskGraph split r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Thu, 17 Dec 2020 22:26:22 +0000 - rev 561197
Push 38042 by ccoroiu@mozilla.com at Fri, 18 Dec 2020 09:56:07 +0000
Bug 1682365 - Pt 4 - Add RenderTaskGraphBuilder / RenderTaskGraph split r=jnicol This patch splits the graph building functionality into `RenderTaskGraphBuilder` and the graph querying code into the existing `RenderTaskGraph` struct. The Builder struct is retained frame to frame, which means there is no longer a need for the `RenderTaskGraphCounters` struct. The Graph struct is constructed per-frame by calling `end_pass` on the Builder. Although this doesn't do much different internally, it will make integration with the new task graph changes simpler. It also enforced during frame building when it is possible to add / query render tasks. A few unrelated tidy ups are included in this patch - mostly removing where the task graph is passed to from a few structs and methods that no longer require access to the graph. Differential Revision: https://phabricator.services.mozilla.com/D99297
54c5a7cbdd7b1e1db66e6e70f009caa40615a3c6: Bug 1682365 - Pt 3 - Remove array textures from picture cache tiles. r=nical,jnicol
Glenn Watson <git@intuitionlibrary.com> - Thu, 17 Dec 2020 22:26:19 +0000 - rev 561196
Push 38042 by ccoroiu@mozilla.com at Fri, 18 Dec 2020 09:56:07 +0000
Bug 1682365 - Pt 3 - Remove array textures from picture cache tiles. r=nical,jnicol This patch makes picture cache tiles use normal textures instead of array textures. With this and the previous patch, WR no longer uses array textures at all (except when provided by the external image handler trait). Differential Revision: https://phabricator.services.mozilla.com/D99013
fec645aa40a68da809bf9fd6d6c93ae4a13aaba2: Bug 1682744. Mark mixed blend mode tests as passing. r=jnicol
Jeff Muizelaar <jmuizelaar@mozilla.com> - Wed, 16 Dec 2020 14:21:21 +0000 - rev 560938
Push 38039 by abutkovits@mozilla.com at Thu, 17 Dec 2020 09:27:48 +0000
Bug 1682744. Mark mixed blend mode tests as passing. r=jnicol Bug 1682398 caused us to layerize mixed-blend-mode less aggressively which means that we're using Skia everywhere for these tests now instead of relying on the GPU. Differential Revision: https://phabricator.services.mozilla.com/D99896
11ffdf919a4a1d0a29a7718a3a53a8a4b4468434: Bug 1680769 - Fix incorrect returned size in some cases. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 11 Dec 2020 14:58:54 +0000 - rev 560339
Push 38023 by ncsoregi@mozilla.com at Fri, 11 Dec 2020 21:30:49 +0000
Bug 1680769 - Fix incorrect returned size in some cases. r=jnicol When deallocating the last item of the region the region's size is changed in free and we end up returning the wrong value. The faulty code is actually removed in a later patch from this series but I'd like to fix the regressing now and land the patch that removes the bad code during the next train. Differential Revision: https://phabricator.services.mozilla.com/D99458
0ec48e93ed40092089a92ea4dd67a94b4e7040bd: Bug 1676559 - Pt 8 - Move render target pool from renderer to frame building. r=nical,jnicol
Glenn Watson <git@intuitionlibrary.com> - Tue, 08 Dec 2020 20:44:48 +0000 - rev 559876
Push 38015 by rmaries@mozilla.com at Wed, 09 Dec 2020 03:49:03 +0000
Bug 1676559 - Pt 8 - Move render target pool from renderer to frame building. r=nical,jnicol This is an incremental but important step to implementing render tasks as a proper graph. By moving the render target management to the frame building step, we know the texture_id of all sub-passes before the batching is done for any passes that use these as inputs. This means that we can directly reference the texture_id during batch, rather that the old `RenderTaskCache` and `PrevPassAlpha` / `PrevPassColor` enum fields (although removal of all these will be done in the next patch). Another advantage of this is that we have much better knowledge of which targets are required for rendering a given frame, so these can be allocated up front at the start of a frame. This may be a better allocation pattern for some drivers. We also have better knowledge available on when a texture can be invalidated, and the render target pool management is simpler since it is the same as the way other texture cache textures are handled. Differential Revision: https://phabricator.services.mozilla.com/D98547
f1ad5c9b8c33a2acd52ca2ac6183d53b84bbeeb0: Bug 1680769 - Fix the texture cache allocating rectangles multiple times (again). r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Mon, 07 Dec 2020 17:32:43 +0000 - rev 559718
Push 38012 by dluca@mozilla.com at Tue, 08 Dec 2020 09:45:00 +0000
Bug 1680769 - Fix the texture cache allocating rectangles multiple times (again). r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D98941
95c9143cca21adf342535ac5e7c08e34032b9cb4: Bug 1676559 - Pt 8 - Move render target pool from renderer to frame building. r=nical,jnicol
Glenn Watson <git@intuitionlibrary.com> - Sun, 06 Dec 2020 20:59:42 +0000 - rev 559592
Push 38009 by ccoroiu@mozilla.com at Mon, 07 Dec 2020 09:28:23 +0000
Bug 1676559 - Pt 8 - Move render target pool from renderer to frame building. r=nical,jnicol This is an incremental but important step to implementing render tasks as a proper graph. By moving the render target management to the frame building step, we know the texture_id of all sub-passes before the batching is done for any passes that use these as inputs. This means that we can directly reference the texture_id during batch, rather that the old `RenderTaskCache` and `PrevPassAlpha` / `PrevPassColor` enum fields (although removal of all these will be done in the next patch). Another advantage of this is that we have much better knowledge of which targets are required for rendering a given frame, so these can be allocated up front at the start of a frame. This may be a better allocation pattern for some drivers. We also have better knowledge available on when a texture can be invalidated, and the render target pool management is simpler since it is the same as the way other texture cache textures are handled. Differential Revision: https://phabricator.services.mozilla.com/D98547
75add21b1e226bdf8e83fe87998aafac2ecdac10: Bug 1679939 - Remove experimental PLS functionality. r=jnicol,nical
Glenn Watson <git@intuitionlibrary.com> - Tue, 01 Dec 2020 19:28:19 +0000 - rev 558952
Push 37995 by ncsoregi@mozilla.com at Wed, 02 Dec 2020 03:28:13 +0000
Bug 1679939 - Remove experimental PLS functionality. r=jnicol,nical The pixel-local-storage functionality was an experiment for faster drawing of clip masks on low end tiled GPUs. However, it's never reached a point where it was shippable and showing clear performance wins. This patch removes the experimental PLS support - we can always revive it from git history if we ever want to consider it again. Differential Revision: https://phabricator.services.mozilla.com/D98290
d7ae1b44e7a6084835bd0fffae8b4391f464f92c: Bug 1677696 - Use texture array shader variant for clear tiles. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 27 Nov 2020 15:53:06 +0000 - rev 558564
Push 37985 by dluca@mozilla.com at Sat, 28 Nov 2020 09:27:06 +0000
Bug 1677696 - Use texture array shader variant for clear tiles. r=jnicol The dummy texture is a texture array. The bug was caused by selecting the non-array version of the shader while binding a texture array. Differential Revision: https://phabricator.services.mozilla.com/D97991
88d559804ae95728240e48d60b3bff51ca7f2959: Bug 1624446 - Adjust reftest fuzziness. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Thu, 26 Nov 2020 16:54:32 +0000 - rev 558551
Push 37984 by nbeleuzu@mozilla.com at Fri, 27 Nov 2020 15:53:21 +0000
Bug 1624446 - Adjust reftest fuzziness. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D98002
b2d438ba2bdd2f2098ce1d4b3be1fd66c7eecf46: Bug 1678924. Lower switch to if-else chain. r=mattwoodrow,jnicol
Jeff Muizelaar <jmuizelaar@mozilla.com> - Thu, 26 Nov 2020 20:52:39 +0000 - rev 558523
Push 37984 by nbeleuzu@mozilla.com at Fri, 27 Nov 2020 15:53:21 +0000
Bug 1678924. Lower switch to if-else chain. r=mattwoodrow,jnicol It looks we hit the same gen6 compiler bug as we did in bug 1663344. Differential Revision: https://phabricator.services.mozilla.com/D98027
64036a891adc60e29df074ec1bf0bd62213eabb4: Bug 1677771 - Fix the texture cache allocating rectangles multiple times. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 20 Nov 2020 12:34:27 +0000 - rev 558175
Push 37969 by malexandru@mozilla.com at Fri, 20 Nov 2020 16:31:52 +0000
Bug 1677771 - Fix the texture cache allocating rectangles multiple times. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D97574
aaf0ff52517d0969684974084d8d4270f67598f1: Bug 1674443 - Introduce custom slab sizes for glyphs. r=jnicol,gw
Nicolas Silva <nsilva@mozilla.com> - Fri, 13 Nov 2020 15:16:04 +0000 - rev 557508
Push 37959 by btara@mozilla.com at Tue, 17 Nov 2020 21:55:29 +0000
Bug 1674443 - Introduce custom slab sizes for glyphs. r=jnicol,gw This is a modest win on hidpi and a larger win on lowdpi. We can improve further by introducing more granularity between 16 and 32px, this puts in place the basic infrastructure on top of which we can experiment. Differential Revision: https://phabricator.services.mozilla.com/D95870
801a132e9ae7c69fa21afe3c7c96195b48e3ad58: Bug 1645318 - Adjust reftest references. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Mon, 16 Nov 2020 10:55:34 +0000 - rev 557307
Push 37957 by malexandru@mozilla.com at Tue, 17 Nov 2020 09:44:06 +0000
Bug 1645318 - Adjust reftest references. r=jnicol Depends on D97132 Differential Revision: https://phabricator.services.mozilla.com/D97133
d20e39f7dcacd81ddfe342085a6b063d8ca896c0: Bug 1619715 - Adjust reftest references. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Mon, 16 Nov 2020 10:56:02 +0000 - rev 557306
Push 37957 by malexandru@mozilla.com at Tue, 17 Nov 2020 09:44:06 +0000
Bug 1619715 - Adjust reftest references. r=jnicol Differential Revision: https://phabricator.services.mozilla.com/D97132
2c2dc8dab741cf4e380bf427dd751dab2045aee1: Bug 1674443 - Lay regions out in a grid in the texture cache SVG dump. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 13 Nov 2020 15:20:57 +0000 - rev 557165
Push 37948 by ncsoregi@mozilla.com at Sat, 14 Nov 2020 09:46:25 +0000
Bug 1674443 - Lay regions out in a grid in the texture cache SVG dump. r=jnicol Also tweak the visualization in various ways so that having a large amount of regions (glyphs) doesn't bring down simple SVG viewing software. Differential Revision: https://phabricator.services.mozilla.com/D95758
bbbf988c294078355aaa7fb7c271c77876483904: Bug 1674443 - Use 128x128 regions in the glyph atlas. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 13 Nov 2020 15:15:43 +0000 - rev 557164
Push 37948 by ncsoregi@mozilla.com at Sat, 14 Nov 2020 09:46:25 +0000
Bug 1674443 - Use 128x128 regions in the glyph atlas. r=jnicol Since glyphs are rarely larger than 128x128, we can reduce the amount of wasted space from partially used glyph regions by having smaller ones (and more of them). Differential Revision: https://phabricator.services.mozilla.com/D95757
e18cf134339a9aef2e1f1cfa7b07f2aedafae7d2: Bug 1674443 - Support multiple texture region sizes. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Fri, 13 Nov 2020 15:15:36 +0000 - rev 557163
Push 37948 by ncsoregi@mozilla.com at Sat, 14 Nov 2020 09:46:25 +0000
Bug 1674443 - Support multiple texture region sizes. r=jnicol In the next patch we'll set a smaller region size for the glyph texture. Differential Revision: https://phabricator.services.mozilla.com/D95756
4e6e7829d164521667c5df5b67ef0bbcabb139a0: Bug 1673387 - Adjust reftest references. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Thu, 12 Nov 2020 22:26:32 +0000 - rev 557039
Push 37946 by dluca@mozilla.com at Fri, 13 Nov 2020 09:59:41 +0000
Bug 1673387 - Adjust reftest references. r=jnicol Changing the size or number of layers of textures unfortunately usually leads to small sampling differences which requires fixing in the refetest references. Differential Revision: https://phabricator.services.mozilla.com/D95680
a315ba98577fd7d78de7ab380fc140e6783d2437: Bug 1673387 - Adjust reftest references. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Thu, 12 Nov 2020 12:28:28 +0000 - rev 556904
Push 37943 by btara@mozilla.com at Thu, 12 Nov 2020 21:28:42 +0000
Bug 1673387 - Adjust reftest references. r=jnicol Changing the size or number of layers of textures unfortunately usually leads to small sampling differences which requires fixing in the refetest references. Differential Revision: https://phabricator.services.mozilla.com/D95680
a755e87bf875b2ae1f8a36595107be45c8c84299: Bug 1673387 - Adjust reftest references. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Thu, 05 Nov 2020 16:32:22 +0000 - rev 556042
Push 37924 by apavel@mozilla.com at Thu, 05 Nov 2020 21:37:48 +0000
Bug 1673387 - Adjust reftest references. r=jnicol Changing the size or number of layers of textures unfortunately usually leads to small sampling differences which requires fixing in the refetest references. Differential Revision: https://phabricator.services.mozilla.com/D95680
33fbbff5e7252b70666d0c5eaedd4b1bb51a74b7: Bug 1675159 - Pt 1 - Pass `buffer_age` via `render` rather than callback. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Wed, 04 Nov 2020 22:12:40 +0000 - rev 555920
Push 37922 by cbrindusan@mozilla.com at Thu, 05 Nov 2020 04:52:47 +0000
Bug 1675159 - Pt 1 - Pass `buffer_age` via `render` rather than callback. r=jnicol Implementing the Draw compositor via the native compositor interface is simpler if the buffer age is passed into the top level render method. Differential Revision: https://phabricator.services.mozilla.com/D95824
644505d8afbb7e297270d186a2cc5e37488be3f5: Bug 1675159 - Pt 1 - Pass `buffer_age` via `render` rather than callback. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Wed, 04 Nov 2020 20:04:02 +0000 - rev 555894
Push 37922 by cbrindusan@mozilla.com at Thu, 05 Nov 2020 04:52:47 +0000
Bug 1675159 - Pt 1 - Pass `buffer_age` via `render` rather than callback. r=jnicol Implementing the Draw compositor via the native compositor interface is simpler if the buffer age is passed into the top level render method. Differential Revision: https://phabricator.services.mozilla.com/D95824
badfff22ee8b50ab5ea56fbc93d1e22744454328: Bug 1673320 - Dump color linear and glyph texture cache arrays as SVG during frame captures. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Tue, 27 Oct 2020 09:34:38 +0000 - rev 554596
Push 37896 by rmaries@mozilla.com at Tue, 27 Oct 2020 15:58:19 +0000
Bug 1673320 - Dump color linear and glyph texture cache arrays as SVG during frame captures. r=jnicol The visualization shows texture arrays (one array per row) with allocations in blue and free space in dark gray, as well as the slab size for each region. This only dumps the texture arrays for color+linear images and glyphs since most of the interesting stuff is there, but adding other arrays will be simple if needed. Differential Revision: https://phabricator.services.mozilla.com/D94715
54f221c0ff5c51b1f151bc04e3512db6f45828a4: Bug 1670995 - run android mochitest/wpt on -qr by default. r=jnicol
Joel Maher <jmaher@mozilla.com> - Wed, 21 Oct 2020 19:16:06 +0000 - rev 554081
Push 37888 by smolnar@mozilla.com at Fri, 23 Oct 2020 09:46:15 +0000
Bug 1670995 - run android mochitest/wpt on -qr by default. r=jnicol run android mochitest/wpt on -qr by default Differential Revision: https://phabricator.services.mozilla.com/D94334
e8d4a6e4e9d708b788eeaec08d67b545aa05cfaf: Bug 1672214 - Place glyphs in their own texture array. r=jnicol
Nicolas Silva <nsilva@mozilla.com> - Tue, 20 Oct 2020 20:52:06 +0000 - rev 553795
Push 37882 by ccoroiu@mozilla.com at Wed, 21 Oct 2020 16:00:28 +0000
Bug 1672214 - Place glyphs in their own texture array. r=jnicol Depends on D94125 Differential Revision: https://phabricator.services.mozilla.com/D94126
c2f0c2e26efff9d57526e2322c97d234393108b3: Bug 1665068 - Preserve layers for inactive GV windows. r=jnicol,snorp
Agi Sferro <agi@sferro.dev> - Mon, 19 Oct 2020 18:28:49 +0000 - rev 553451
Push 37878 by smolnar@mozilla.com at Tue, 20 Oct 2020 09:40:32 +0000
Bug 1665068 - Preserve layers for inactive GV windows. r=jnicol,snorp Differential Revision: https://phabricator.services.mozilla.com/D93816
c4d6ac24a7f672cc0a8e79d89e64ed7c26d916af: Bug 1669567 - Increase the default tile size used for scrollbars. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Mon, 12 Oct 2020 00:09:44 +0000 - rev 553055
Push 37862 by apavel@mozilla.com at Thu, 15 Oct 2020 03:57:02 +0000
Bug 1669567 - Increase the default tile size used for scrollbars. r=jnicol Most platforms have scrollbars > 16 pixels in width. This typically reduces the number of native layers allocated on most platforms for scrollbars. This is quite important as the fixed overhead of each native layer is relatively high on DC and CA. Differential Revision: https://phabricator.services.mozilla.com/D92871
a99d2faf30deb671246e374ededc7b4d77916950: Bug 1670842 - Pt 1 - Remove option to disable picture caching. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Wed, 14 Oct 2020 06:06:45 +0000 - rev 553051
Push 37862 by apavel@mozilla.com at Thu, 15 Oct 2020 03:57:02 +0000
Bug 1670842 - Pt 1 - Remove option to disable picture caching. r=jnicol This patch removes the public API and high level logic for disabling picture caching for debugging and pinch-zoom in some cases. Follow up patches will remove and simplify the internal parts of WR that remain to handle the disabled picture caching code path. Differential Revision: https://phabricator.services.mozilla.com/D93446
9d453852d2fe30cd2968628a5b35f2af68e397a7: Bug 1670543 - Remove fence argument from AndroidHardwareBuffer::Unlock() r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Tue, 13 Oct 2020 00:31:30 +0000 - rev 552760
Push 37857 by malexandru@mozilla.com at Tue, 13 Oct 2020 09:40:53 +0000
Bug 1670543 - Remove fence argument from AndroidHardwareBuffer::Unlock() r=jnicol The function does not need fence argument. It could be handled within AndroidHardwareBuffer. Though all tested recent Android devices did not return valid fence. AndroidHardwareBuffer usage is not enabled on gecko. Differential Revision: https://phabricator.services.mozilla.com/D93217
18fb8d11fd2d8b85ff361c3323729a47ee4577ff: Bug 1669567 - Increase the default tile size used for scrollbars. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Sat, 10 Oct 2020 19:21:30 +0000 - rev 552619
Push 37853 by apavel@mozilla.com at Mon, 12 Oct 2020 09:36:05 +0000
Bug 1669567 - Increase the default tile size used for scrollbars. r=jnicol Most platforms have scrollbars > 16 pixels in width. This typically reduces the number of native layers allocated on most platforms for scrollbars. This is quite important as the fixed overhead of each native layer is relatively high on DC and CA. Differential Revision: https://phabricator.services.mozilla.com/D92871
4c410e101c8d70767dffce7a898268696419d7b3: Bug 1670148 - Fix release fence handling at AsyncImagePipelineManager on Android r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Fri, 09 Oct 2020 14:15:52 +0000 - rev 552561
Push 37850 by ncsoregi@mozilla.com at Sat, 10 Oct 2020 09:37:08 +0000
Bug 1670148 - Fix release fence handling at AsyncImagePipelineManager on Android r=jnicol By Bug 1661189, AndroidHardwareBuffer owning TextureHosts became to be stored in mTextureHostsUntilRenderSubmitted. AndroidHardwareBuffer usage is not enabled on gecko. Differential Revision: https://phabricator.services.mozilla.com/D93050
fe44655a70ca5c3f8230fbc287a08c70c45f88b9: Bug 1669567 - Use fixed size texture arrays for picture cache slices. r=jnicol
Glenn Watson <git@intuitionlibrary.com> - Wed, 07 Oct 2020 21:58:06 +0000 - rev 552011
Push 37845 by nbeleuzu@mozilla.com at Thu, 08 Oct 2020 16:14:24 +0000
Bug 1669567 - Use fixed size texture arrays for picture cache slices. r=jnicol Previously we used a single texture array for a given tile size, and resized the texture array as more tiles were needed. However, this results in expensive driver stalls and GPU copy times when resizing the array. Instead, use a fixed slice count for each texture array, and support multiple textures with the same tile size. This may result in slightly more draw calls during compositing of picture cache tiles due to batch breaks, but will remain a small number due to the limited number of picture cache tiles that are allocated at any one time. Differential Revision: https://phabricator.services.mozilla.com/D92715
357733872c55fac7aa8380207b174dae094604bb: Bug 1661371 - Use eglWaitSyncKHR() for andoid fence if possible r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Wed, 30 Sep 2020 06:45:30 +0000 - rev 550832
Push 37823 by cbrindusan@mozilla.com at Wed, 30 Sep 2020 21:45:29 +0000
Bug 1661371 - Use eglWaitSyncKHR() for andoid fence if possible r=jnicol For performance, it is better to use eglWaitSyncKHR() for andoid fence if possible. This change is not used by gecko yet. Differential Revision: https://phabricator.services.mozilla.com/D89882
590bd63c98117d1e1516ca9dafabb23c4f066132: Bug 1667927 - Do not close file descriptor of android::Fence if EGLSync is created r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Tue, 29 Sep 2020 07:32:14 +0000 - rev 550680
Push 37820 by dluca@mozilla.com at Tue, 29 Sep 2020 15:38:31 +0000
Bug 1667927 - Do not close file descriptor of android::Fence if EGLSync is created r=jnicol When EGLSync is created from fence file descriptor, gecko should not close it. It should be closed by gecko only when EGLSync creation is failed. Differential Revision: https://phabricator.services.mozilla.com/D91702
b60c4c84feab9006e8be92dc84939f4c6d5b591a: Bug 1665815 - Avoid using scissored glClear() on Android Emulator with SwiftShader r=geckoview-reviewers,jnicol,agi
sotaro <sotaro.ikeda.g@gmail.com> - Thu, 24 Sep 2020 09:43:21 +0000 - rev 550125
Push 37808 by apavel@mozilla.com at Thu, 24 Sep 2020 21:53:48 +0000
Bug 1665815 - Avoid using scissored glClear() on Android Emulator with SwiftShader r=geckoview-reviewers,jnicol,agi Emulator with SwiftShader is buggy when attempting to clear picture cache textures with a scissor rect set. Differential Revision: https://phabricator.services.mozilla.com/D90644
c5e25062d707bfc3db26e14c746b4edce77cec4d: Bug 1663344. Lower switch to if else chain. r=jnicol
Jeff Muizelaar <jmuizelaar@mozilla.com> - Wed, 23 Sep 2020 16:57:33 +0000 - rev 550048
Push 37807 by dluca@mozilla.com at Thu, 24 Sep 2020 09:34:15 +0000
Bug 1663344. Lower switch to if else chain. r=jnicol glslopt doesn't support switch statements in its IR so they get lowered to a collection of if statements. It seems like there's a driver bug on Intel gen6 that can cause this if chain to be miscompiled. If we manually lower the switch to an 'if else chain' it seems to avoid the problem. Differential Revision: https://phabricator.services.mozilla.com/D91172
138c5bca87c4f1738ce2fa692d3eabdc9c715ee4: Bug 1663574 - Prevent to send invalid file descriptor by OpDeliverReleaseFence message r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Wed, 09 Sep 2020 02:02:45 +0000 - rev 547944
Push 37767 by dluca@mozilla.com at Wed, 09 Sep 2020 09:39:57 +0000
Bug 1663574 - Prevent to send invalid file descriptor by OpDeliverReleaseFence message r=jnicol When invalid file descriptor is sent via IPC, it causes error log. Differential Revision: https://phabricator.services.mozilla.com/D89416
13ffdb654409bbf27b4ab52a356dfd341a25a3a2: Bug 1660750 - Fix timeout handling of AndroidHardwareBufferManager::WaitForBufferOwnership() r=jnicol
sotaro <sotaro.ikeda.g@gmail.com> - Tue, 08 Sep 2020 13:21:14 +0000 - rev 547926
Push 37767 by dluca@mozilla.com at Wed, 09 Sep 2020 09:39:57 +0000
Bug 1660750 - Fix timeout handling of AndroidHardwareBufferManager::WaitForBufferOwnership() r=jnicol Prevent infinite wait at mMonitor.Wait(). And gfxCriticalNote log with shorter wait period. Differential Revision: https://phabricator.services.mozilla.com/D89412