searching for reviewer(tnikkel)
46bb469ae5f43eae987a3e85b31009655e7835e2: Bug 1749780 - Update libjxl to 9a74bd70b7932750deb78a8aebd6e041ce7f8b01 r=tnikkel
Kagami Sascha Rosylight <krosylight@mozilla.com> - Thu, 13 Jan 2022 12:49:48 +0000 - rev 604456
Push 39147 by ncsoregi@mozilla.com at Thu, 13 Jan 2022 21:57:49 +0000
Bug 1749780 - Update libjxl to 9a74bd70b7932750deb78a8aebd6e041ce7f8b01 r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D135769
fe2b3753928238178b95c43921a5f82ac75bdc5b: Bug 1749780 - Update libjxl r=tnikkel
Kagami Sascha Rosylight <krosylight@mozilla.com> - Thu, 13 Jan 2022 01:54:46 +0000 - rev 604440
Push 39145 by ctuns@mozilla.com at Thu, 13 Jan 2022 09:33:36 +0000
Bug 1749780 - Update libjxl r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D135769
9de7d3c3aa7473c75a165db7034890c7ef51238d: Bug 1742985 - Add test to ensure zooming works correctly following GPU process restart. r=tnikkel
Jamie Nicol <jnicol@mozilla.com> - Fri, 07 Jan 2022 13:27:49 +0000 - rev 603961
Push 39127 by apavel@mozilla.com at Fri, 07 Jan 2022 21:31:35 +0000
Bug 1742985 - Add test to ensure zooming works correctly following GPU process restart. r=tnikkel Depends on D135328 Differential Revision: https://phabricator.services.mozilla.com/D135329
6bdd63376651ad836609ffa27ca6f5515ef96dfc: Bug 1742985 - Add more methods to nsIGfxInfo for GPU process testing. r=tnikkel
Jamie Nicol <jnicol@mozilla.com> - Fri, 07 Jan 2022 13:27:49 +0000 - rev 603960
Push 39127 by apavel@mozilla.com at Fri, 07 Jan 2022 21:31:35 +0000
Bug 1742985 - Add more methods to nsIGfxInfo for GPU process testing. r=tnikkel Add KillGPUProcessForTests, which kills the GPU process without generating a crash dump (unlike the existing CrashGPUProcessForTests). Additionally add EnsureGPUProcessReadyForTests, which returns a promise that resolves to true when the GPU process is enabled and ready, and false if it is disabled. If called while the GPU process is being (re)started, it will not resolve until it has finished launching (or was disabled due to error). Finally, make GPUProcessHost::IsConnected check whether the process handle is valid. This ensures it returns false immediately following a call to KillProcess but prior to the GPUChild being destroyed. This means tests can call EnsureGPUProcessReadyForTests immediately after KillGPUProcessForTests or CrashGPUProcessForTests, and it will reliably wait for the new process to launch, as intended. Depends on D135207 Differential Revision: https://phabricator.services.mozilla.com/D135328
441e0b2582cc86f15ae0a2b6d0e835240e112471: Bug 1742985 - Ensure ZoomConstraints get refreshed after GPU process restart. r=tnikkel
Jamie Nicol <jnicol@mozilla.com> - Fri, 07 Jan 2022 13:27:48 +0000 - rev 603959
Push 39127 by apavel@mozilla.com at Fri, 07 Jan 2022 21:31:35 +0000
Bug 1742985 - Ensure ZoomConstraints get refreshed after GPU process restart. r=tnikkel Following a GPU process restart ZoomConstraints do not currently get set for the newly recreated APZCTreeManagers, meaning it is no longer possible to asynchronously zoom pages. To solve this, we make ZoomConstraintsClient observe a new "compositor-reinitialized" topic. We send this notification in GPUProcessManager::HandleProcessLost() to notify ZoomConstraintsClients for parent process documents, and in ContentChild::RecvReinitRendering() for documents in their respective content processes. This must be performed after the compositor has been reinitialized so that the APZCTreeManagerChild is able to send the constraints to the APZCTreeManagerParent in the compositor process. Differential Revision: https://phabricator.services.mozilla.com/D135207
808ac93bdd9847272087ba981173bc877cff0425: Bug 1748437: link from `LayoutUtils`'s `getElementBoundingScreenRect` documentation to a definition of "screen" coordinates. r=tnikkel
Mirko Brodesser <mbrodesser@mozilla.com> - Wed, 05 Jan 2022 09:13:36 +0000 - rev 603757
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1748437: link from `LayoutUtils`'s `getElementBoundingScreenRect` documentation to a definition of "screen" coordinates. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D135045
9ff648d32b4d6aedd40dd75a177894f6baa64fb1: Bug 1348786 - Make browser_test_swipe_gesture.js work on Windows. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:08 +0000 - rev 603756
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Make browser_test_swipe_gesture.js work on Windows. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134639
d1a04dde5a734cfcd86aa7d690b9323d45e8010f: Bug 1348786 - Get `browserStopped` promise before sending swipe gestures. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:08 +0000 - rev 603755
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Get `browserStopped` promise before sending swipe gestures. r=tnikkel There's a race condition where the page navigation caused by swipe gestures has already stopped when the promise for the pan gestures was resolved. Differential Revision: https://phabricator.services.mozilla.com/D134638
c4d2d0f6137e907d0ec099badc8321c71988d576: Bug 1348786 - Use SwipeTracker on Windows. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:08 +0000 - rev 603754
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Use SwipeTracker on Windows. r=tnikkel This code is equivalent with what nsChildView.mm does. Differential Revision: https://phabricator.services.mozilla.com/D134369
72c816ff1334be627d0e1259f024bb23a443a685: Bug 1348786 - Factor out the last part of shouldConsiderStartingSwipeFromEvent into a new function. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:07 +0000 - rev 603753
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Factor out the last part of shouldConsiderStartingSwipeFromEvent into a new function. r=tnikkel The `PanGestureInput`, the argument of the new function named CanTriggerSwipe, is generated by `theEvent`, the `PanGesgureInput.mPanDisplacement.{x,y}` are calculated by `theEvent.scrollingDelta{X,Y}` * `BackingScaleFactor()`, so both inequalities are equivalent. [1] https://searchfox.org/mozilla-central/rev/07c3bd159f79f81f86050d5e002a14ed741f34f7/widget/cocoa/nsChildView.mm#3394-3395,3408 Differential Revision: https://phabricator.services.mozilla.com/D134368
67859633bb443ac64f44ad7097862775d0d31d4c: Bug 1348786 - Factor out functions to (may) trigger a swipe gesture into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:07 +0000 - rev 603752
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Factor out functions to (may) trigger a swipe gesture into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134367
acefc474e9607002873118faf0aad86c4c598e05: Bug 1348786 - Move SendMayStartSwipe() into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:07 +0000 - rev 603751
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move SendMayStartSwipe() into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134365
1c6284530243f16f9ad52adbec5b7a166c62288e: Bug 1348786 - Move ReportSwipeStarted() and TrackScrollEventAsSwipe() into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:06 +0000 - rev 603750
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move ReportSwipeStarted() and TrackScrollEventAsSwipe() into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134364
5f838131ae32656e8d8ba9c3fc30b93ff5b85df4: Bug 1348786 - Move mCurrentPanGestureBelongsToSwipe into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:06 +0000 - rev 603749
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move mCurrentPanGestureBelongsToSwipe into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134363
70520fc1660ed9ea3c6be3e020c127a516c287ec: Bug 1348786 - Move mSwipeEventQueue from nsChildView into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:06 +0000 - rev 603748
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move mSwipeEventQueue from nsChildView into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134362
e96d9b6dc70c90ded6f96db5b8b66c1a3d6e9899: Bug 1348786 - Move SwipeTracker class as reusable for other platforms. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:05 +0000 - rev 603747
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move SwipeTracker class as reusable for other platforms. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134361
28a5cb06ece848b08704c887c36c19e618b1002f: Bug 1348786 - Use GetDefaultScaleInternal instead of BackingScaleFactor in SwipeTracker::ProcessEvent. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:05 +0000 - rev 603746
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Use GetDefaultScaleInternal instead of BackingScaleFactor in SwipeTracker::ProcessEvent. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134360
c057fa40caf7e14c7b25fa968b4d901056d7f831: Bug 1348786 - Move nsChildView::DispatchWindowEvent(WidgetGUIEvent&) into nsBaseWidget. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 05 Jan 2022 09:08:04 +0000 - rev 603745
Push 39118 by abutkovits@mozilla.com at Wed, 05 Jan 2022 17:03:24 +0000
Bug 1348786 - Move nsChildView::DispatchWindowEvent(WidgetGUIEvent&) into nsBaseWidget. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D134359
c688da89e7ee84828349174286ce2345c76d09bd: Bug 1747927 - Minor ImageTracker cleanup. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 04 Jan 2022 13:30:18 +0000 - rev 603623
Push 39113 by smolnar@mozilla.com at Tue, 04 Jan 2022 16:58:41 +0000
Bug 1747927 - Minor ImageTracker cleanup. r=tnikkel Don't bother going through the approximately visible image frames if the locking state doesn't change. Differential Revision: https://phabricator.services.mozilla.com/D134961
3cd27ead74b31f4e667384507e8dd7816c3abc1e: Bug 1746310 - Remove nsMenuPopupFrame::GenerateFrames and related code. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 23 Dec 2021 12:55:34 +0000 - rev 603045
Push 39089 by apavel@mozilla.com at Thu, 23 Dec 2021 21:57:35 +0000
Bug 1746310 - Remove nsMenuPopupFrame::GenerateFrames and related code. r=tnikkel The root of the problem is that nsMenuPopupFrame::GenerateFrames calls into frame construction without making sure that styles are clean. So it was pretty much working by chance, sorta. I was going to fix this by adding the necessary flushes before calling GenerateFrames, but on closer inspection, the front-end has effectively already implemented this optimization by only generating the relevant DOM on popupShowing: https://searchfox.org/mozilla-central/rev/a11b63915bd7810a03635d733123448ab5bfcad3/toolkit/content/widgets/menupopup.js#87-91 And for menulists on creation: https://searchfox.org/mozilla-central/rev/a11b63915bd7810a03635d733123448ab5bfcad3/toolkit/content/widgets/menupopup.js#151 After bug 1714846 we even destroy frames as needed, for panels. So I think all of this complexity is unwarranted, and if we need some of it we should implement it in the front-end like bug 1714846 did, and I'd rather do this than flushing styles and so on. There's one tweak I had to do to an nsPlaceholderFrame assertion. The reason is that now the nsMenuPopupFrames do get their NS_FRAME_FIRST_REFLOW bit cleared here: https://searchfox.org/mozilla-central/rev/bd25b1ca76dd5d323ffc69557f6cf759ba76ba23/layout/xul/nsMenuPopupFrame.cpp#557 Because the IsLeaf() condition here is no longer true: https://searchfox.org/mozilla-central/rev/bd25b1ca76dd5d323ffc69557f6cf759ba76ba23/layout/xul/nsMenuPopupFrame.cpp#532 It doesn't change anything though, because this condition never holded for popups consistently. Differential Revision: https://phabricator.services.mozilla.com/D134331
4f982fc49e6eeba397af54a4538c7e63869717d2: Bug 1355821 - Allow a half pixel difference if the top level document gets scaled under 1.0. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Tue, 21 Dec 2021 01:50:37 +0000 - rev 602787
Push 39081 by mlaza@mozilla.com at Tue, 21 Dec 2021 04:41:29 +0000
Bug 1355821 - Allow a half pixel difference if the top level document gets scaled under 1.0. r=tnikkel Our mochitest harness doesn't specify meta viewport tags, so the test document basically gets scaled down on mobile platforms, thus scroll position is aligned with screen pixels (by ClampAndAlignWithLayerPixels in nsGfxScroll.cpp) rather than CSS pixels, we should allow a half pixel difference on such platforms. Differential Revision: https://phabricator.services.mozilla.com/D134349
224cf9419c8066dd1467c5cd8e7e2e709f46c12f: Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel
Joel Maher <jmaher@mozilla.com> - Fri, 10 Dec 2021 16:19:54 +0000 - rev 601772
Push 39054 by nbeleuzu@mozilla.com at Fri, 10 Dec 2021 21:58:52 +0000
Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132728
653b181d3cefe4e706dce0685e9662a490694c12: Bug 1742051 - Allow factor-of-2 scaling for vector images without intrinsic size. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Fri, 10 Dec 2021 14:54:02 +0000 - rev 601761
Push 39054 by nbeleuzu@mozilla.com at Fri, 10 Dec 2021 21:58:52 +0000
Bug 1742051 - Allow factor-of-2 scaling for vector images without intrinsic size. r=tnikkel If a vector image does not have an intrinsic size as returned by VectorImage::GetWidth and VectorImage::GetHeight, currently factor-of-2 scaling is disabled. With this patch, we just assume a default size of 100x100, adjusted by the intrinsic ratio if available, to get the baseline size for determing appropriate factor-of-2 sizes. Differential Revision: https://phabricator.services.mozilla.com/D133395
036452a35172770e32d2d3bccf00528a44696f5e: Bug 1745177 - Update libjpeg-turbo to 2.1.2. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Thu, 09 Dec 2021 21:39:26 +0000 - rev 601651
Push 39051 by nfay@mozilla.com at Fri, 10 Dec 2021 09:10:34 +0000
Bug 1745177 - Update libjpeg-turbo to 2.1.2. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D133349
9505197158716faadc101286218960166e038c6f: Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel
Joel Maher <jmaher@mozilla.com> - Thu, 09 Dec 2021 19:41:48 +0000 - rev 601634
Push 39051 by nfay@mozilla.com at Fri, 10 Dec 2021 09:10:34 +0000
Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132728
76491e4adee2a198690c6fc306906b34ceaea107: Bug 1745177 - Update libjpeg-turbo to 2.1.2. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Thu, 09 Dec 2021 15:48:37 +0000 - rev 601568
Push 39050 by ncsoregi@mozilla.com at Fri, 10 Dec 2021 05:31:59 +0000
Bug 1745177 - Update libjpeg-turbo to 2.1.2. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D133349
e1f23710c4180dd5c1ed928cb06d4dd39576ef0a: Bug 1704876 - Fix a shutdown crash when we race to update an animated vector image. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Tue, 07 Dec 2021 03:10:10 +0000 - rev 601173
Push 39044 by nfay@mozilla.com at Tue, 07 Dec 2021 15:51:17 +0000
Bug 1704876 - Fix a shutdown crash when we race to update an animated vector image. r=tnikkel If we are shutting down, the document for a VectorImage may be cleared. If a refresh tick raced with the shutdown, we might try to deref the null document. Differential Revision: https://phabricator.services.mozilla.com/D132953
5d547fd3bf593f6809c3c6de2366351901a61345: Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel
Joel Maher <jmaher@mozilla.com> - Tue, 07 Dec 2021 00:47:09 +0000 - rev 601166
Push 39044 by nfay@mozilla.com at Tue, 07 Dec 2021 15:51:17 +0000
Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132728
cd4c095076e0607db464558b79d03356b7884224: Bug 1743793 - Update libjxl and highway r=tnikkel
Kagami Sascha Rosylight <krosylight@mozilla.com> - Mon, 06 Dec 2021 23:43:32 +0000 - rev 601149
Push 39042 by apavel@mozilla.com at Tue, 07 Dec 2021 04:09:38 +0000
Bug 1743793 - Update libjxl and highway r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132952
feff6d3ebdcd9b086e57fcb93b03d71fe38a73b1: Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel
Joel Maher <jmaher@mozilla.com> - Mon, 06 Dec 2021 13:43:37 +0000 - rev 601085
Push 39041 by malexandru@mozilla.com at Mon, 06 Dec 2021 21:54:35 +0000
Bug 1743907 - remove reftest sandbox references to webrender. r=ahal,tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132728
356c521826254fbae435b212923b20801d171950: Bug 1743761 - Ensure we invalidate substituted images when the preferred size is ready. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Fri, 03 Dec 2021 03:14:01 +0000 - rev 600989
Push 39032 by mlaza@mozilla.com at Fri, 03 Dec 2021 09:48:16 +0000
Bug 1743761 - Ensure we invalidate substituted images when the preferred size is ready. r=tnikkel Previously with ImageContainers, we would put the new preferred surface into the ImageContainer. When we check if we should invalidate, it would have a different image key, and hence invalidate the image frame and schedule a paint. With ImageProviders, it returns the same key in this case, because the ImageProvider represents a particular surface. As such, we need to actually track when we get a substituted ImageProvider, and invalidate the image frame more aggressively to ensure we get the preferred size. Differential Revision: https://phabricator.services.mozilla.com/D132583
218cc619458ff27b88646a5794044c0e2c5f5ff9: Bug 1743377 - Prefer using to typedef in imgLoader. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 29 Nov 2021 15:18:13 +0000 - rev 600376
Push 39021 by ncsoregi@mozilla.com at Mon, 29 Nov 2021 21:53:24 +0000
Bug 1743377 - Prefer using to typedef in imgLoader. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132347
3fe9e1643aafa1f8f5ba19b012b90b0d39a422b5: Bug 1711061 - Part 13. Remove the now unused ImageContainer and related code for images. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:40 +0000 - rev 600334
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 13. Remove the now unused ImageContainer and related code for images. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D126606
2eef1762a4e8b96bfc80aaa14db91e8bc5e85a4d: Bug 1711061 - Part 12. Change the display list to use WebRenderImageProvider. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:40 +0000 - rev 600333
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 12. Change the display list to use WebRenderImageProvider. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D126605
466af290693ca9abd7b15dbc84b11ee71e350afc: Bug 1711061 - Part 11. Implement VectorImage::GetImageProvider. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:40 +0000 - rev 600332
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 11. Implement VectorImage::GetImageProvider. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D126604
d7f9835a65e8278145405539295275e28fa9f9d0: Bug 1711061 - Part 10. Implement BlobSurfaceProvider for non-rasterized blob recordings. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:39 +0000 - rev 600331
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 10. Implement BlobSurfaceProvider for non-rasterized blob recordings. r=tnikkel This WebRenderImageProvider/ISurfaceProvider subclass provides the implementation for blob recordings. This is mostly just taking the functionality that was previously in SourceSurfaceBlobImage. Differential Revision: https://phabricator.services.mozilla.com/D126602
7503e1289b9290943e1397dfa33999f58c861c70: Bug 1711061 - Part 9. Add blob recording support to SurfaceCache. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:39 +0000 - rev 600330
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 9. Add blob recording support to SurfaceCache. r=tnikkel Now that we no longer have the extra layer of ImageContainers providing a superficial level of caching/reuse of existing blob recordings, we need some way to share recordings. This part adds support to SurfaceCache to store BlobSurfaceProvider objects. This includes the specialized code for invalidating SVG images. In particular this is useful for animated SVG images. In general we want to avoid changing the image key whenever possible so that we avoid reallocating the underlying buffers in the compositor process for the rasterized blob images. We also need to track the ImageIntRegion used by the recording. If a caller only wants a slice of the SVG image, then we need to track this differentiation in our cache entries. At this time, we don't allow substitutes for entries with a region exclusion. Differential Revision: https://phabricator.services.mozilla.com/D126603
889fcfb23434aea59635e50280e732eb986cd263: Bug 1711061 - Part 8. Rename SourceSurfaceBlobImage.h/cpp to BlobSurfaceProvider.h/cpp. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:38 +0000 - rev 600329
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 8. Rename SourceSurfaceBlobImage.h/cpp to BlobSurfaceProvider.h/cpp. r=tnikkel Rename the file in preparation for the switch over to WebRenderImageProvider. Differential Revision: https://phabricator.services.mozilla.com/D126601
7d2c3c3a53394d7d1ed850846207929421354c9c: Bug 1711061 - Part 7. Remove support for ImageContainer-based SVG image blob recordings. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:38 +0000 - rev 600328
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 7. Remove support for ImageContainer-based SVG image blob recordings. r=tnikkel This will be replaced by a WebRenderImageProvider-based implementation in a later part in this series. Differential Revision: https://phabricator.services.mozilla.com/D126600
3b3078df854d624c251041f17983eb623156f084: Bug 1711061 - Part 6. Implement RasterImage::GetImageProvider. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:38 +0000 - rev 600327
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 6. Implement RasterImage::GetImageProvider. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D126599
43278d9a5da0185fe8632671785ed1f0803fb77c: Bug 1711061 - Part 5. Expose an ImageProviderId for tracking surface ownership. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:37 +0000 - rev 600326
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 5. Expose an ImageProviderId for tracking surface ownership. r=tnikkel This will be used by layers as a replacement for ImageContainer's ContainerProducerID for tracking if the same imgIRequest/imgIContainer own the cached WebRenderImageProvider. Differential Revision: https://phabricator.services.mozilla.com/D126598
d6a41b6555bb88e1c8376d16062a5a32b8f7d430: Bug 1711061 - Part 4. Implement AnimationSurfaceProvider for animated rasterized images. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:37 +0000 - rev 600325
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 4. Implement AnimationSurfaceProvider for animated rasterized images. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D126790
de019933be1f918230bc03c1bf899a7bbd85e784: Bug 1711061 - Part 3. Add WebRenderImageProvider and implement for rasterized providers. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:36 +0000 - rev 600324
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 3. Add WebRenderImageProvider and implement for rasterized providers. r=tnikkel This provides the framework to allow ISurfaceProvider objects to implement WebRenderImageProvider. It is straightforward for rasterized providers (DecodedSurfaceProvider, and SimpleSurfaceProvider). Later parts in this series will provide the necessary changes for blob recordings and for animatedi images. Differential Revision: https://phabricator.services.mozilla.com/D126597
41590355b53497e013bd8921208d21565a8a3dc7: Bug 1711061 - Part 2. Add imgIContainer::GetImageProvider skeleton. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:36 +0000 - rev 600323
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 2. Add imgIContainer::GetImageProvider skeleton. r=tnikkel In later parts in this series, GetImageProvider will replace GetImageContainerAtSize. This will be a more specialized and lower overhead means to get a wr::ImageKey for a particular surface. Differential Revision: https://phabricator.services.mozilla.com/D126596
a7c6389a767fa193a7652a448b4683f31fd882ae: Bug 1711061 - Part 1. Remove SourceSurfaceMappedData surface deduplication from memory reports. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Sat, 27 Nov 2021 11:47:36 +0000 - rev 600322
Push 39016 by smolnar@mozilla.com at Sat, 27 Nov 2021 22:00:34 +0000
Bug 1711061 - Part 1. Remove SourceSurfaceMappedData surface deduplication from memory reports. r=tnikkel We no longer use SourceSurfaceMappedData because we only support SourceSurfaceSharedData-backed imgFrame in the SurfaceCache now. Differential Revision: https://phabricator.services.mozilla.com/D126595
b1dceea241d2fa5f5ac50b630d464a2b27f27120: Bug 1742919 - Protect the access to mState in AsyncPanZoomController::IsAutoscroll() with the APZC lock. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 26 Nov 2021 23:29:12 +0000 - rev 600314
Push 39015 by ctuns@mozilla.com at Sat, 27 Nov 2021 09:28:10 +0000
Bug 1742919 - Protect the access to mState in AsyncPanZoomController::IsAutoscroll() with the APZC lock. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D132283
b36a5fe267df2735d3efc940f76ea02ab8397146: Bug 1743006 - Tweak fuzz value for subframe-scrollbar-zoomed-out-async-scroll.html. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 26 Nov 2021 23:27:19 +0000 - rev 600313
Push 39015 by ctuns@mozilla.com at Sat, 27 Nov 2021 09:28:10 +0000
Bug 1743006 - Tweak fuzz value for subframe-scrollbar-zoomed-out-async-scroll.html. r=tnikkel Needed to account for differences between native and non-native scrollbar themes. Differential Revision: https://phabricator.services.mozilla.com/D132282
c32f29e1e65470ab3b592f0fba60a32291f995fe: Bug 1742254 - Decrease the size of the scrollable rect of the subframe in subframe-scrollbar-zoomed-out-async-scroll.html. r=tnikkel
Botond Ballo <botond@mozilla.com> - Wed, 24 Nov 2021 05:24:56 +0000 - rev 599991
Push 39006 by nerli@mozilla.com at Wed, 24 Nov 2021 15:52:24 +0000
Bug 1742254 - Decrease the size of the scrollable rect of the subframe in subframe-scrollbar-zoomed-out-async-scroll.html. r=tnikkel On Android, there is no lower bound on the length of the scroll thumb, and the dimensions of the subframe in this test were such that the thumb was barely a speck on the screen, making the test not very useful. Differential Revision: https://phabricator.services.mozilla.com/D131990
891cba7d04e91ccfefb35a51a05d34dadb6cd5e6: Bug 1742729 - Fix seen_pipeline_ids assert r=tnikkel
Glenn Watson <git@intuitionlibrary.com> - Wed, 24 Nov 2021 00:46:38 +0000 - rev 599982
Push 39006 by nerli@mozilla.com at Wed, 24 Nov 2021 15:52:24 +0000
Bug 1742729 - Fix seen_pipeline_ids assert r=tnikkel The assert needs to occur before the calls to create the reference frame. Differential Revision: https://phabricator.services.mozilla.com/D131989
2d8d58edaaa35c5c611a5a05b4f56db5f2412577: Bug 1634763 - In non-Fission mode, downgrade the assertion about the result of the WebRender display list being consistent with the APZ scroll data to non-fatal. r=tnikkel
Botond Ballo <botond@mozilla.com> - Tue, 23 Nov 2021 08:02:58 +0000 - rev 599860
Push 39003 by mlaza@mozilla.com at Tue, 23 Nov 2021 16:53:32 +0000
Bug 1634763 - In non-Fission mode, downgrade the assertion about the result of the WebRender display list being consistent with the APZ scroll data to non-fatal. r=tnikkel This assertion is known to affect some scenarios involving inactive scroll frames (see comment 10-11 in this bug). In Fission mode, this is resolved by making all scroll frames active. In non-Fission mode, we don't plan to pursue a specific fix so downgrade the assertion to avoid annoyance for users running debug builds. Differential Revision: https://phabricator.services.mozilla.com/D131886