searching for reviewer(mattwoodrow)
e7e7717d923630782861c01221c5fc5048b04c37: Bug 1551053 - Check both list end and non-container items separately during display item merging r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Mon, 13 May 2019 21:13:01 +0000 - rev 532483
Push 11268 by csabou@mozilla.com at Tue, 14 May 2019 15:24:22 +0000
Bug 1551053 - Check both list end and non-container items separately during display item merging r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D30941
1df96f5e640231e8ddbc8ed1dadaef69b55d4e62: Bug 1548484 - respect Cairo blit size limits when using BasicCompositor. r=mattwoodrow
Lee Salzman <lsalzman@mozilla.com> - Fri, 10 May 2019 03:00:34 +0000 - rev 532158
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1548484 - respect Cairo blit size limits when using BasicCompositor. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D30571
58de421f3fd072e481877f80e1c7c2e398027b17: Bug 1548795 - P1. Don't use StaticPrefs in GPU process. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 09 May 2019 23:12:12 +0000 - rev 532148
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1548795 - P1. Don't use StaticPrefs in GPU process. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D29984
f6b5ab8f054ad48b1826c5af211a1a6c0bb214bb: Bug 1549860 - Make display item merging nsDisplayWrapList specific operation r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Wed, 08 May 2019 15:51:23 +0000 - rev 531901
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1549860 - Make display item merging nsDisplayWrapList specific operation r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D30283
b5a3f0b9ce243f5cb610ee849d35637d3402c35b: Bug 1549751 - Part 3: Use nsPaintedDisplayItem in WR and FLB r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Wed, 08 May 2019 13:50:09 +0000 - rev 531888
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1549751 - Part 3: Use nsPaintedDisplayItem in WR and FLB r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D30226
69a48ceb1ecb6c0857a5a258d653be3b0a7b93b6: Bug 1549751 - Part 2: Add nsPaintedDisplayItem r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Wed, 08 May 2019 13:49:56 +0000 - rev 531887
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1549751 - Part 2: Add nsPaintedDisplayItem r=mattwoodrow Move ApplyOpacity(), CanApplyOpacity(), Paint() and PaintWithClip() methods to nsPaintedDisplayItem Differential Revision: https://phabricator.services.mozilla.com/D30225
61e59c0325ddeba0bfe6a3624a60d0e7150ac632: Bug 1549751 - Part 1: Add nsDisplayList::Iterator r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Wed, 08 May 2019 14:11:40 +0000 - rev 531886
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1549751 - Part 1: Add nsDisplayList::Iterator r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D30224
b16160b39a3f5a55211c18eaabc1e48e5b9ec505: Bug 1546955 - Part 2: Make all display item constructors take nsDisplayListBuilder and frame as the first two parameters r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Sun, 05 May 2019 21:45:31 +0000 - rev 531511
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1546955 - Part 2: Make all display item constructors take nsDisplayListBuilder and frame as the first two parameters r=mattwoodrow Depends on D28844 Differential Revision: https://phabricator.services.mozilla.com/D28846
f9c160fec2c4965a1942d62209dfded9b9701240: Bug 1546955 - Part 1: Invoke display item copy-constructors through MakeClone() r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Sun, 05 May 2019 21:05:33 +0000 - rev 531510
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1546955 - Part 1: Invoke display item copy-constructors through MakeClone() r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D28844
7df5932c9da0928f3cc0ece509293e5aeb98864b: Bug 1547277. If the size of the document as recorded in the frame/view tree and the document viewer diverge as a result of a call to GetContentSize make sure to invalidate when they converge again because what we draw is changing. r=mattwoodrow
Timothy Nikkel <tnikkel@gmail.com> - Sun, 05 May 2019 03:21:41 +0000 - rev 531440
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1547277. If the size of the document as recorded in the frame/view tree and the document viewer diverge as a result of a call to GetContentSize make sure to invalidate when they converge again because what we draw is changing. r=mattwoodrow The code comment mostly explains things. Differential Revision: https://phabricator.services.mozilla.com/D29953
762224e2d5e9076f67088e396046d92813d27459: Bug 1547420 - Handle bounds overflow in FrameLayerBuilder::PaintItems() r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Fri, 03 May 2019 15:15:44 +0000 - rev 531311
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1547420 - Handle bounds overflow in FrameLayerBuilder::PaintItems() r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D29754
849b94e8913547d915ef4e54a0f0f2f479f579d0: Bug 1548410 - Disable DXVA on Qualcomm WoA drivers. r=mattwoodrow
Chris Pearce <cpearce@mozilla.com> - Thu, 02 May 2019 02:42:31 +0000 - rev 531042
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1548410 - Disable DXVA on Qualcomm WoA drivers. r=mattwoodrow This driver version is known to have busg which cause the output of green frames from the decoder, and to cause BSODs. Differential Revision: https://phabricator.services.mozilla.com/D29603
7c58a86df1c6aa72126951ba9b2930a6ce5de99e: Bug 1540785 - Part 2: Devirtualize nsDisplayItemBase::GetPerFrameKey() r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Mon, 22 Apr 2019 19:58:01 +0000 - rev 529219
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540785 - Part 2: Devirtualize nsDisplayItemBase::GetPerFrameKey() r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27580
5f7f77eac50e0fe1082411aa5c566714281fdbd4: Bug 1540785 - Part 1: Introduce nsDisplayItemBase r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Mon, 22 Apr 2019 19:57:59 +0000 - rev 529218
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540785 - Part 1: Introduce nsDisplayItemBase r=mattwoodrow *** Differential Revision: https://phabricator.services.mozilla.com/D27579
817642dc917c886910c8fb9ad856ec153c2655c3: Bug 1526972 - P8: Reset InInvalidSubtree when processing unrelated frames. r=mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Thu, 18 Apr 2019 02:41:33 +0000 - rev 528807
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1526972 - P8: Reset InInvalidSubtree when processing unrelated frames. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26141
4e0d56e43a739eab3fcba4d2c65efa1280060887: Bug 1526972 - P6: Mark invalidated SubDocument frame as modified. r=miko,mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Thu, 18 Apr 2019 02:38:24 +0000 - rev 528805
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1526972 - P6: Mark invalidated SubDocument frame as modified. r=miko,mattwoodrow Depends on D24463 Differential Revision: https://phabricator.services.mozilla.com/D26139
185e608c385272703bd89da77aa0d9a4d2c7a024: Bug 1544478 - Remove unused method Image::GetBuffer(). r=mattwoodrow
Markus Stange <mstange@themasta.com> - Tue, 16 Apr 2019 17:04:41 +0000 - rev 528533
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544478 - Remove unused method Image::GetBuffer(). r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27604
4ab1a110e5d50f11983bbe9b4500c44383767328: Bug 1544478 - Stub out SharedPlanarYCbCrImage::AdoptData because it can't be called without crashing, which means it's unused. r=mattwoodrow
Markus Stange <mstange@themasta.com> - Tue, 16 Apr 2019 17:03:23 +0000 - rev 528532
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544478 - Stub out SharedPlanarYCbCrImage::AdoptData because it can't be called without crashing, which means it's unused. r=mattwoodrow GetBuffer() would always return null here. Differential Revision: https://phabricator.services.mozilla.com/D27603
d5ac9681d3ce59c427804b4f5add2daa1fcc06d0: Bug 1544478 - Add a reftest. r=mattwoodrow
Markus Stange <mstange@themasta.com> - Tue, 16 Apr 2019 17:03:14 +0000 - rev 528531
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544478 - Add a reftest. r=mattwoodrow The reftests in the webm-video directory are currently only run in non-e10s mode (bug 1544849), so at the moment, this test only provides coverage on Linux 32bit debug builds and Android Fennec builds. Differential Revision: https://phabricator.services.mozilla.com/D27601
cf3fbdaa46b93a7b63100fda94974e7810899417: Bug 1544478 - Respect SharedRGBImage's stride during YUV->RGBA conversion. r=mattwoodrow
Markus Stange <mstange@themasta.com> - Tue, 16 Apr 2019 17:02:59 +0000 - rev 528530
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544478 - Respect SharedRGBImage's stride during YUV->RGBA conversion. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27600
bacdb7ee8f0e2b9fa68729e90e6b2dafdb2d3209: Bug 1544583 - Change RDL list index to uint32_t r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Mon, 15 Apr 2019 23:16:30 +0000 - rev 528405
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544583 - Change RDL list index to uint32_t r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27623
472d0da9b5a6d2f509a8d0c80ec08dac565ea4d5: Bug 1529992 - don't apply shadow adjustment to text bounds in gecko with WR r=mattwoodrow
Alexis Beingessner <a.beingessner@gmail.com> - Mon, 15 Apr 2019 23:13:47 +0000 - rev 528401
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1529992 - don't apply shadow adjustment to text bounds in gecko with WR r=mattwoodrow And tell webrender to do them itself (they have fundamental mismatches in how they define shadow bounds). Differential Revision: https://phabricator.services.mozilla.com/D26845
ddf29d68efb2667fbdfa60b21b4c6f6585d40822: Bug 1529992 - don't apply shadow adjustment to text bounds in gecko with WR r=mattwoodrow
Alexis Beingessner <a.beingessner@gmail.com> - Mon, 15 Apr 2019 17:25:26 +0000 - rev 528355
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1529992 - don't apply shadow adjustment to text bounds in gecko with WR r=mattwoodrow And tell webrender to do them itself (they have fundamental mismatches in how they define shadow bounds). Differential Revision: https://phabricator.services.mozilla.com/D26845
7d6cd4e207055a667e1136ba200603f2420c671c: Bug 1543482 - Add the irregular-area flag for frames with masks and clip paths. r=mattwoodrow,botond
Kartikaya Gupta <kgupta@mozilla.com> - Mon, 15 Apr 2019 16:02:02 +0000 - rev 528337
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1543482 - Add the irregular-area flag for frames with masks and clip paths. r=mattwoodrow,botond This will force APZ to do a main-thread hit-test for the cases where the shapes are too complex to easily send to the compositor. Differential Revision: https://phabricator.services.mozilla.com/D27455
7d9507696389b81ea47798b0528769422bd3ec96: Bug 1543482 - Extract a helper to identify clip paths WR can handle without masks. r=mattwoodrow
Kartikaya Gupta <kgupta@mozilla.com> - Mon, 15 Apr 2019 11:24:53 +0000 - rev 528336
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1543482 - Extract a helper to identify clip paths WR can handle without masks. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27454
91fe817e863159b414b9e37c363899b6872af691: Bug 1544074 - Remove WS_EX_LAYERED and WS_EX_TRANSPARENT r=mattwoodrow
sotaro <sotaro.ikeda.g@gmail.com> - Mon, 15 Apr 2019 01:24:26 +0000 - rev 528281
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1544074 - Remove WS_EX_LAYERED and WS_EX_TRANSPARENT r=mattwoodrow Bug 1525183 imported chromium workaround. But it caused a device specific regression. The change removes WS_EX_LAYERED and WS_EX_TRANSPARENT, but it keeps WS_EX_NOREDIRECTIONBITMAP, since it is necessary for memory usage reduction. Differential Revision: https://phabricator.services.mozilla.com/D27461
815543d81a1d2ab79ed4e27612dba769e4b697a8: Bug 1526972 - P8: Reset InInvalidSubtree when processing unrelated frames. r=mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Mon, 15 Apr 2019 00:23:21 +0000 - rev 528268
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1526972 - P8: Reset InInvalidSubtree when processing unrelated frames. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26141
2ea2f8533078605736d98959a36c596414d793de: Bug 1526972 - P6: Mark invalidated SubDocument frame as modified. r=miko,mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Mon, 15 Apr 2019 00:23:17 +0000 - rev 528266
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1526972 - P6: Mark invalidated SubDocument frame as modified. r=miko,mattwoodrow Depends on D24463 Differential Revision: https://phabricator.services.mozilla.com/D26139
ee00c85b09c5980f33100ea03b7f09c32f5fce64: Bug 1540581 - P20. Change fuzzying leeway on macOS. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 23:24:55 +0000 - rev 527964
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P20. Change fuzzying leeway on macOS. r=mattwoodrow There are slight differences produced between the earlier 422 output surface and the 420; interestingly the image is sharper with 420. Differential Revision: https://phabricator.services.mozilla.com/D27020
841b6b61757792b59cee7412512e16e9a61b8989: Bug 1540581 - P18. Move destination texture type choice to the D3D11 texture allocator. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:39:12 +0000 - rev 527962
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P18. Move destination texture type choice to the D3D11 texture allocator. r=mattwoodrow It allows for more readable code, not having to store multiple times different storage type across multiple objects. Now each class does one task and only deal with a single texture data type. Differential Revision: https://phabricator.services.mozilla.com/D26473
575c1e8291c7e3c6c2a3b7160a7b862b304520cc: Bug 1540581 - P13. Properly handle colorspace with D3D11 compositors. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:38:55 +0000 - rev 527958
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P13. Properly handle colorspace with D3D11 compositors. r=mattwoodrow Also fix 10/12 bits images composition with old D3D11 compositor. Differential Revision: https://phabricator.services.mozilla.com/D26468
a0176a275ee5a60ce1bef7bde28055cc49cc2a26: Bug 1540581 - P12. Pass YUV colorspace information with WMF decoder. r=bryce,mattwoodrow.
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:38:51 +0000 - rev 527957
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P12. Pass YUV colorspace information with WMF decoder. r=bryce,mattwoodrow. Differential Revision: https://phabricator.services.mozilla.com/D26467
bfef6e9e74d53eaa3ef44f27498f31c2b5aee34f: Bug 1540581 - P11. Handle colorspace correctly with D3D11 images. r=mattwoodrow,nical
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:38:49 +0000 - rev 527956
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P11. Handle colorspace correctly with D3D11 images. r=mattwoodrow,nical Differential Revision: https://phabricator.services.mozilla.com/D26466
c8d5854c0ef078fe1f725530c8fc4479abe65b1c: Bug 1540581 - P5. Add ColorSpace info to MacIOSurface. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:36:31 +0000 - rev 527950
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P5. Add ColorSpace info to MacIOSurface. r=mattwoodrow It would have been nicer and more logical to use a MacIOSurfaceImage to store the colorspace information, however the data passed around is a IOSurface Id via a SurfaceDescriptorMacIOSurface; the original data structure isn't kept; so we must transit the data at the same time as the IOSurface Id. Also remove unnecessary test (new operator is infallible). Differential Revision: https://phabricator.services.mozilla.com/D26058
9a4305ff9a2dd501279f98c01ea0609952c4b72c: Bug 1540581 - P4. Add ColorSpace/ColorDepth component to NV12 gfx::Effect. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:36:19 +0000 - rev 527949
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P4. Add ColorSpace/ColorDepth component to NV12 gfx::Effect. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26057
a6a6f88b71c7294faf8b21492a5f3ac6c904aa58: Bug 1540581 - P1. Fix NV12 image with OGL compositor. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:35:41 +0000 - rev 527946
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540581 - P1. Fix NV12 image with OGL compositor. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26054
f2eda1d5abb8bb5a73dc91d18ae0654272d0518c: Bug 1493898 - P6. Move YUVColorSpace definition in the gfx namespace. r=mattwoodrow.
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:41:33 +0000 - rev 527945
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1493898 - P6. Move YUVColorSpace definition in the gfx namespace. r=mattwoodrow. YUVColorSpace is inseparable from the bit depth as the matrix coefficients to be calculated need the bit depth information. So let's put the two types together. gfx namespace also makes more sense as that's where we find IntRect, IntSize and other. The extent of the changes highlight how much similar data structures are duplicated across the code, to the point it's scary. Differential Revision: https://phabricator.services.mozilla.com/D25347
529a9a8efae1411d6e858da014db27a48f235b4b: Bug 1493898 - P1. Add preliminary BT2020 colorspace support. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 11 Apr 2019 12:40:40 +0000 - rev 527940
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1493898 - P1. Add preliminary BT2020 colorspace support. r=mattwoodrow Only active with webrender and AL. Differential Revision: https://phabricator.services.mozilla.com/D25342
35caf615155e3fc9638eed7b00cd99ac08dd4f7b: Bug 1543744 - use clip bounds when drawing masks with Skia. r=mattwoodrow
Lee Salzman <lsalzman@mozilla.com> - Thu, 11 Apr 2019 19:00:32 +0000 - rev 527920
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1543744 - use clip bounds when drawing masks with Skia. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D27112
f69e63080fc61dfcd28b531a3416559eac47648a: Bug 1541985 - Merge nsCharClipDisplayItem and nsDisplayText r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Wed, 10 Apr 2019 01:41:22 +0000 - rev 527513
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1541985 - Merge nsCharClipDisplayItem and nsDisplayText r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26843
68ea0add020c9fa119a74ae6216f9fcabf926f9f: Bug 1532731 - Handle 3D transformed ancestors when calculating display port transform. r=mattwoodrow
Jamie Nicol <jnicol@mozilla.com> - Tue, 09 Apr 2019 21:39:02 +0000 - rev 527486
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1532731 - Handle 3D transformed ancestors when calculating display port transform. r=mattwoodrow ClientTiledPaintedLayer::GetTransformToAncestorsParentLayer calculates the transform from the layer to its display port ancestor's parent. This transform is then applied to the calculated display port. However, if the display port ancestor participates in a preserve-3d context then the scroll offset will not be included in the that layer's transform, it will instead be on the root layer of the preserve-3d context. This was causing the critical display port to remain still as the contents of a perspective transform were scrolled, resulting in content being permanently painted in low-precision as the page was scrolled down. Instead, if the display port ancestor participates in a 3d context, we must find the root of that 3d context then calculate the transform to *that* layer's parent. Differential Revision: https://phabricator.services.mozilla.com/D26821
10584c20a8985a3fe5a350ec6f5a5e94b941dfc0: Bug 1536423 - Remove SVGCharClipDisplayItem r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Thu, 04 Apr 2019 18:04:49 +0000 - rev 526847
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1536423 - Remove SVGCharClipDisplayItem r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D26191
664efc1162c431bfcc1127da5c578de727e4e21a: Bug 1541426 - Part 2: Remove unused display item constructor parameter aAnonymous r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Thu, 04 Apr 2019 17:36:53 +0000 - rev 526846
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1541426 - Part 2: Remove unused display item constructor parameter aAnonymous r=mattwoodrow Depends on D25953 Differential Revision: https://phabricator.services.mozilla.com/D25954
ea3a31b1e48ee983c708ac72f7a10a7885551512: Bug 1541426 - Part 1: Remove nsDisplayWrapList from nsDisplayPerspective r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Thu, 04 Apr 2019 17:36:51 +0000 - rev 526845
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1541426 - Part 1: Remove nsDisplayWrapList from nsDisplayPerspective r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D25953
08f895aaa82a0604bf9fae118509b6f86d341ba2: Bug 1413567. Enable retained display lists for android. r=mattwoodrow
Timothy Nikkel <tnikkel@gmail.com> - Thu, 28 Mar 2019 23:02:38 +0000 - rev 526793
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1413567. Enable retained display lists for android. r=mattwoodrow Even though we don't look at the chrome value on android in nsLayoutUtils::AreRetainedDisplayListsEnabled(), I flipped it just in case someone looks at that pref somewhere else. Depends on D25280 Differential Revision: https://phabricator.services.mozilla.com/D25281
53f0254dfc7e43bc153bcef26fa3a87fd62b6334: Bug 1413567. Adjust test expectations for RDL on android. r=mattwoodrow
Timothy Nikkel <tnikkel@gmail.com> - Thu, 28 Mar 2019 23:01:00 +0000 - rev 526792
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1413567. Adjust test expectations for RDL on android. r=mattwoodrow All three failures are small antialiasing differences in the text in the test. The text in the tests aren't even testing anything, they just describe what to expect from the test. Until we get fuzzy matching in web platform tests (bug 1478472) we just have to mark them as fails, but we can be very specific about the platform, so we're probably still passing these tests on other android jobs if they are run there (not sure). Differential Revision: https://phabricator.services.mozilla.com/D25280
040b950d93ae08f77f6294a8ef73ac7626d2d3a2: Bug 1532257 - Remove frequently-failing assertion. r=mattwoodrow
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 03 Apr 2019 13:42:56 +0000 - rev 526589
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1532257 - Remove frequently-failing assertion. r=mattwoodrow We're gonna delete all this code imminently anyway. Differential Revision: https://phabricator.services.mozilla.com/D25932
10a446ed2a38d6af1eaa8b6d51ad9f1333b247d8: Bug 1540908. If the layer manager revokes a transaction id then revert the last stored transaction id too. r=mattwoodrow
Timothy Nikkel <tnikkel@gmail.com> - Tue, 02 Apr 2019 12:08:21 +0000 - rev 526465
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1540908. If the layer manager revokes a transaction id then revert the last stored transaction id too. r=mattwoodrow This prevents calls to nsPresContext::NotifyInvalidation from using a transaction id that has already been revoked. Android does "repeat" transactions, where one EndTransaction call results in two transactions. The first updates the "critical" area of painted content (immediately visible). The second updates the rest of the display port. What happens in the specific failing test that triggered this is the first transaction results in invalidations from layer tree invalidation. The second transaction nothing has changed, so it gets revoked. When EndTransaction returns to the nsDisplayList code and calls nsPresContext::NotifyInvalidation it uses the last transaction id stored on the layer manager. But this has already been revoked. Differential Revision: https://phabricator.services.mozilla.com/D25702
0ea914bf03b719cd0821564b551dbb384ba95ff5: Bug 1479145 - Give RGB textures a 32-byte aligned stride on macOS in order to improve texture upload efficiency on certain drivers. r=mattwoodrow
Markus Stange <mstange@themasta.com> - Fri, 29 Mar 2019 20:11:12 +0000 - rev 525741
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1479145 - Give RGB textures a 32-byte aligned stride on macOS in order to improve texture upload efficiency on certain drivers. r=mattwoodrow In particular, it looks like this alignment is required by the Intel driver on macOS if you want to avoid CPU copies. It was already known that the efficiency gains from client storage only materialize if you follow certain restrictions: - The textures need to use the TEXTURE_RECTANGLE_ARB texture target. - The textures' format, internalFormat and type need to be chosen from a small list of supported configurations. Unsupported configurations will trigger format conversions on the CPU. - The GL_TEXTURE_STORAGE_HINT_APPLE may need to be set to shared or cached. - glTextureRangeAPPLE may or may not make a difference. It now appears that the stride alignment is another requirement: When uploading textures which otherwise comply with the above requirements, the Intel driver will still make copies using the CPU if the texture's stride is not 32-byte aligned. These CPU copies are reflected in a high CPU usage (as observed in Activity Monitor) and they show up in profiles as time spent inside _platform_memmove under glrUpdateTexture. However, when uploading 32-byte stride aligned textures which comply with the above requirements, this CPU usage goes away. There might still be hardware copies behind the scenes, but they no longer take up CPU time. Differential Revision: https://phabricator.services.mozilla.com/D25316
5a70120ae2d349d2ad8a8ac3530f4bd390677d92: Bug 1539892 - replace FeatureChange and GPUDeviceStatus IPDL unions with native Maybe syntax; r=mattwoodrow
Alex Gaynor <agaynor@mozilla.com> - Thu, 28 Mar 2019 21:13:53 +0000 - rev 525592
Push 11265 by ffxbld-merge at Mon, 13 May 2019 10:53:39 +0000
Bug 1539892 - replace FeatureChange and GPUDeviceStatus IPDL unions with native Maybe syntax; r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D25259