searching for reviewer(tnikkel)
081aa9877fc18f4ff43201375a12f77bdf4becfb: Bug 1589022 - Ignore displayport properties in GetDisplayPortImpl() if APZ is disabled. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 11 Nov 2019 09:23:45 +0000 - rev 502511
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1589022 - Ignore displayport properties in GetDisplayPortImpl() if APZ is disabled. r=tnikkel This facilitates disabling APZ "live", such as when moving a tab from an APZ window into a non-APZ window. Depends on D51468 Differential Revision: https://phabricator.services.mozilla.com/D51469
408a199d1d47d782a0a6f43991d72a00a7a7da49: Bug 1589022 - Partial support for moving a tab between windows with different APZ enablement. r=nika,tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 18 Nov 2019 23:13:55 +0000 - rev 502510
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1589022 - Partial support for moving a tab between windows with different APZ enablement. r=nika,tnikkel This involves two new IPC messages (both async) to propagate the change in compositor options (of which APZ enablement is one) from the GPU process to the parent process (via PCompositorBridge) and on to the content process (via PBrowser). The support is only partial, in that going from non-APZ to APZ is only supported if APZ was enabled at the time the window was created. Depends on D51467 Differential Revision: https://phabricator.services.mozilla.com/D51468
e6305a1457f025d598507456b87945036b49bbab: Bug 1589022 - Make the assertion about a compositor options mismatch in RecvAdoptChild more nuanced. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 11 Nov 2019 09:23:45 +0000 - rev 502509
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1589022 - Make the assertion about a compositor options mismatch in RecvAdoptChild more nuanced. r=tnikkel If only the APZ enablement changed, produce a warning rather than an assertion. Differential Revision: https://phabricator.services.mozilla.com/D51467
ff4ff0dd4da373d91ebf6b5f9a13782a69de7847: Bug 1596494 - Call UpdateCompositionBoundsForRCDRSF only if we are in the top level content document. r=tnikkel
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Fri, 15 Nov 2019 01:54:05 +0000 - rev 502113
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1596494 - Call UpdateCompositionBoundsForRCDRSF only if we are in the top level content document. r=tnikkel Though there is another call site of UpdateCompositionBoundsForRCDRSF in nsLayoutUtils::CalculateRootCompositionSize, it's not clear to me whether it is necessary or not since we early return from the function in the case where |aIsRootContentDocRootScrollFrame| argument is true. We will audit it later in bug 1562505. Differential Revision: https://phabricator.services.mozilla.com/D53117
b79f9f171fb110ee5469a8b6a645da77a44bfde8: Bug 1586144 - Introduce an API to set the dynamic toolbar maximum height in GeckoView. r=geckoview-reviewers,tnikkel,snorp
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Thu, 14 Nov 2019 05:59:47 +0000 - rev 501888
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1586144 - Introduce an API to set the dynamic toolbar maximum height in GeckoView. r=geckoview-reviewers,tnikkel,snorp And deliver the value to the top content pres context, but it's not used in this commit. The value will be used in the next commit. One caveat is that areas covered by the dynamic toolbar will be outside of the content area, which means implementers of GeckoView needs to call setVerticalClipping with _negative_ values. Depends on D50416 Differential Revision: https://phabricator.services.mozilla.com/D50417
5e6b050c9427da9ae37ca4c7c8000ba5b47ceea8: Bug 1594425 - Part 3. Make image decoder gtests use SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Tue, 12 Nov 2019 18:22:41 +0000 - rev 501618
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1594425 - Part 3. Make image decoder gtests use SurfaceFormat::OS_RGBA. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D52009
f24e660c683168764dae60970e8437fb2a2d4a1b: Bug 1594425 - Part 2. Switch image decoders to use SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Tue, 12 Nov 2019 18:22:33 +0000 - rev 501617
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1594425 - Part 2. Switch image decoders to use SurfaceFormat::OS_RGBA. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D52008
5ffac45592a6604c0d34e4a09dd62e92c41e9cfc: Bug 1594425 - Part 1. Add gfxPlatform QCMS helpers for SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Tue, 12 Nov 2019 18:22:31 +0000 - rev 501616
Push 114172 by dluca@mozilla.com at Tue, 19 Nov 2019 11:31:10 +0000
Bug 1594425 - Part 1. Add gfxPlatform QCMS helpers for SurfaceFormat::OS_RGBA. r=tnikkel These helpers will be used by the image decoders in the next part of this series. Differential Revision: https://phabricator.services.mozilla.com/D52007
90ce21e7e469a6418e2d964aba45bbfe26e9e8fa: Bug 1594425 - Part 3. Make image decoder gtests use SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Fri, 08 Nov 2019 14:58:45 +0000 - rev 501350
Push 114168 by dluca@mozilla.com at Sun, 10 Nov 2019 03:08:55 +0000
Bug 1594425 - Part 3. Make image decoder gtests use SurfaceFormat::OS_RGBA. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D52009
54a28220958bbe934eafd7063fa1ed1c046ed54d: Bug 1594425 - Part 2. Switch image decoders to use SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Fri, 08 Nov 2019 14:58:43 +0000 - rev 501349
Push 114168 by dluca@mozilla.com at Sun, 10 Nov 2019 03:08:55 +0000
Bug 1594425 - Part 2. Switch image decoders to use SurfaceFormat::OS_RGBA. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D52008
f1c1e5f12327eee77905e9b6456e667d548b6c0e: Bug 1594425 - Part 1. Add gfxPlatform QCMS helpers for SurfaceFormat::OS_RGBA. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Fri, 08 Nov 2019 14:58:36 +0000 - rev 501348
Push 114168 by dluca@mozilla.com at Sun, 10 Nov 2019 03:08:55 +0000
Bug 1594425 - Part 1. Add gfxPlatform QCMS helpers for SurfaceFormat::OS_RGBA. r=tnikkel These helpers will be used by the image decoders in the next part of this series. Differential Revision: https://phabricator.services.mozilla.com/D52007
fdbf0586295d3aa444d4cb54a183ee9ba3273011: Bug 1592739 - Ignore the background-color CSS value on the window document's root element if that element has a -moz-appearance. r=tnikkel
Markus Stange <mstange@themasta.com> - Tue, 05 Nov 2019 18:47:30 +0000 - rev 500697
Push 114166 by apavel@mozilla.com at Thu, 07 Nov 2019 10:04:01 +0000
Bug 1592739 - Ignore the background-color CSS value on the window document's root element if that element has a -moz-appearance. r=tnikkel For regular elements, whenever -moz-appearance is used, the CSS background is ignored. Root elements were behaving specially, and the background color also needed to be adjusted. For example, for Windows 7, we have the following CSS rule; ``` :root { background-color: transparent; -moz-appearance: -moz-win-borderless-glass; } ``` This change makes the root element more consistent with other elements, so the extra `background-color: transparent` declaration is no longer necessary. This change does not let content documents opt out of forced opaqueness: Root content documents still get an opaque background color from an existing check further down in this method. Differential Revision: https://phabricator.services.mozilla.com/D51459
d0cf63ab46162fdbfe258943aa3789693c3efb38: Bug 1592739 - Make nsChildView::WidgetPaintsBackground() return true. r=tnikkel
Markus Stange <mstange@themasta.com> - Tue, 05 Nov 2019 18:47:21 +0000 - rev 500696
Push 114166 by apavel@mozilla.com at Thu, 07 Nov 2019 10:04:01 +0000
Bug 1592739 - Make nsChildView::WidgetPaintsBackground() return true. r=tnikkel On macOS, the OS window always comes with an opaque background for top level windows. This is the case even if Gecko determines the root element of the window to be transparent: Ever since bug 1162649, nsChildView/nsCocoaWindow ignore calls to SetTransparencyMode for top level windows and always stay opaque. Returning true from nsChildView::WidgetPaintsBackground() lets us indicate that we do not need an opaque backstop color to be added at the bottom of the display list. This backstop color would interfere with vibrant -moz-appearance rendering under the new vibrancy model. WidgetPaintsBackground() is only called in one place, in ComputeBackstopColor(): ``` nscolor PresShell::ComputeBackstopColor(nsView* aDisplayRoot) { nsIWidget* widget = aDisplayRoot->GetWidget(); if (widget && (widget->GetTransparencyMode() != eTransparencyOpaque || widget->WidgetPaintsBackground())) { // Within a transparent widget, so the backstop color must be // totally transparent. return NS_RGBA(0, 0, 0, 0); } // Within an opaque widget (or no widget at all), so the backstop // color must be totally opaque. The user's default background // as reported by the prescontext is guaranteed to be opaque. return GetDefaultBackgroundColorToDraw(); } ``` On Windows 7, the widget returns eTransparencyBorderlessGlass from GetTransparencyMode(), which also avoids the backstop color. Differential Revision: https://phabricator.services.mozilla.com/D51458
8aa8ed80ea4325031cd613b5f85a3d694231c2ba: Bug 1586496 - Add a gtest. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 23:00:42 +0000 - rev 500257
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1586496 - Add a gtest. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D51480
7cec59e15970dca1fa78f73ac3697445ebd50319: Bug 1586496 - If two taps are close in time but far in distance, still allow the second one to start a gesture. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 23:00:33 +0000 - rev 500256
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1586496 - If two taps are close in time but far in distance, still allow the second one to start a gesture. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D51479
d9da7dd620faa289ef6d493bb8a037d799e577da: Bug 1592902 - Extend gtest to cover this scenario. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 21:48:09 +0000 - rev 500255
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592902 - Extend gtest to cover this scenario. r=tnikkel Depends on D51448 Differential Revision: https://phabricator.services.mozilla.com/D51449
17e20df1d0926eec0183b2f6fde20821cbf40532: Bug 1592902 - Include the TOUCHING state in CanHandleScrollOffsetUpdate(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 21:48:07 +0000 - rev 500254
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592902 - Include the TOUCHING state in CanHandleScrollOffsetUpdate(). r=tnikkel Otherwise, a main-thread update can interrupt a touch drag near its very start, when we're still in the TOUCHING state while we're overcoming the touch start tolerance threshold. Depends on D51447 Differential Revision: https://phabricator.services.mozilla.com/D51448
fba08c688f41bb1e759d57d0cb21e6aed029378d: Bug 1592902 - Use ShouldCancelAnimationForScrollUpdate() for visual scroll updates as well. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 21:36:23 +0000 - rev 500253
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592902 - Use ShouldCancelAnimationForScrollUpdate() for visual scroll updates as well. r=tnikkel Depends on D51446 Differential Revision: https://phabricator.services.mozilla.com/D51447
701479e9ecf7b5b71999efb2542d9e5cb1e6170f: Bug 1592902 - Factor out a ShouldCancelAnimationForScrollUpdate() helper. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 21:26:51 +0000 - rev 500252
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592902 - Factor out a ShouldCancelAnimationForScrollUpdate() helper. r=tnikkel Depends on D51445 Differential Revision: https://phabricator.services.mozilla.com/D51446
a407a5cf22c57b855622b3bdcb61fd295e3d6414: Bug 1592902 - Add some logging. r=tnikkel
Botond Ballo <botond@mozilla.com> - Fri, 01 Nov 2019 20:55:09 +0000 - rev 500251
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592902 - Add some logging. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D51445
05fbf853b8b7fc570fa1e6694eb33b4049e46bec: Bug 1592435 - Add a gtest. r=tnikkel
Botond Ballo <botond@mozilla.com> - Thu, 31 Oct 2019 01:07:11 +0000 - rev 500091
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592435 - Add a gtest. r=tnikkel Depends on D51202 Differential Revision: https://phabricator.services.mozilla.com/D51203
31601597fea2d9f0dfe3f9d98efc58627229f438: Bug 1592435 - Don't allow a main thread scroll update to interrupt a touch-drag gesture. r=tnikkel
Botond Ballo <botond@mozilla.com> - Thu, 31 Oct 2019 01:06:14 +0000 - rev 500090
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592435 - Don't allow a main thread scroll update to interrupt a touch-drag gesture. r=tnikkel This makes the behaviour for touch-drags match the behaviour we already have for pan gesture events. Depends on D51201 Differential Revision: https://phabricator.services.mozilla.com/D51202
f170bef85771957490f8e6a0bdf1188806c669ed: Bug 1592435 - Avoid a shutdown-crash during gtests. r=tnikkel
Botond Ballo <botond@mozilla.com> - Thu, 31 Oct 2019 01:02:08 +0000 - rev 500089
Push 114164 by aiakab@mozilla.com at Tue, 05 Nov 2019 10:06:15 +0000
Bug 1592435 - Avoid a shutdown-crash during gtests. r=tnikkel This is unrelated to this bug, I just ran into it while running the tests. Differential Revision: https://phabricator.services.mozilla.com/D51201
a0a5b420415c9fa85902701ae5b77de77a4022a4: Bug 1590582 - Have AsyncPanZoomController::ConvertToGecko return a Maybe rather than using a bool + outparam. r=tnikkel
Botond Ballo <botond@mozilla.com> - Wed, 23 Oct 2019 22:39:48 +0000 - rev 498794
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1590582 - Have AsyncPanZoomController::ConvertToGecko return a Maybe rather than using a bool + outparam. r=tnikkel Depends on D50362 Differential Revision: https://phabricator.services.mozilla.com/D50363
fb87d73d3a4904d2055b3d254efba12526def3de: Bug 1590582 - Un-apply the fixed margins offset in ConvertToGecko as well. r=tnikkel
Botond Ballo <botond@mozilla.com> - Wed, 23 Oct 2019 22:24:18 +0000 - rev 498793
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1590582 - Un-apply the fixed margins offset in ConvertToGecko as well. r=tnikkel This ensures the coordinates of tap events are adjusted as well. Depends on D50361 Differential Revision: https://phabricator.services.mozilla.com/D50362
aa1e247e003858bfe12195b188823f9d0715d473: Bug 1590582 - Subtract the fixed margins offset when adjusting event coordinates during hit testing, rather than adding it. r=tnikkel
Botond Ballo <botond@mozilla.com> - Wed, 23 Oct 2019 22:11:35 +0000 - rev 498792
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1590582 - Subtract the fixed margins offset when adjusting event coordinates during hit testing, rather than adding it. r=tnikkel The offset is applied (added) during rendering; it needs to be un-applied (subtracted) during hit testing. Depends on D50360 Differential Revision: https://phabricator.services.mozilla.com/D50361
9094063323a365cb7a6a28c478a587581ae422f6: Bug 1590582 - Don't leave HitTestResult::mFixedPosSides uninitialized. r=tnikkel
Botond Ballo <botond@mozilla.com> - Wed, 23 Oct 2019 21:35:08 +0000 - rev 498791
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1590582 - Don't leave HitTestResult::mFixedPosSides uninitialized. r=tnikkel Likewise for APZCTreeManager::mFixedPosSidesForInputBlock. Differential Revision: https://phabricator.services.mozilla.com/D50360
dac50d796f639001743366716173ee20f6bfb8ac: Bug 1590426 - Cleanup signatures in nsImageFrame / nsImageControlFrame. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 22 Oct 2019 19:05:21 +0000 - rev 498690
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1590426 - Cleanup signatures in nsImageFrame / nsImageControlFrame. r=tnikkel In particular, remove useless virtual keywords per our style guide, and remove redundant argument names (argument names where the name is contained in the type of the argument), since they're somewhat useless. I think the result looks neater. Differential Revision: https://phabricator.services.mozilla.com/D50074
37538ab48bab7bc2a098fd320cc4178707278c37: Bug 1589955 - Sync-decode images when intrinsic ratio between the old and new source changes. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 23 Oct 2019 12:22:06 +0000 - rev 498689
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1589955 - Sync-decode images when intrinsic ratio between the old and new source changes. r=tnikkel This seems to fix it for me, and it's pretty straight-forward, but I _think_ we'd still paint the old image if the image is huge and it's loading, which may be counter-productive. Maybe we should guard the whole "paint mOldImage" with and if (!oldImageIsDifferent), wdyt? Differential Revision: https://phabricator.services.mozilla.com/D50062
92505a6990eed6019855c0593f09e8e3b1567933: Bug 1560770 - Use a method of getting the widget in UseMobileViewportManager() than does not require the frame tree to be constructed. r=tnikkel
Botond Ballo <botond@mozilla.com> - Tue, 22 Oct 2019 20:40:35 +0000 - rev 498623
Push 114159 by shindli@mozilla.com at Thu, 24 Oct 2019 09:49:00 +0000
Bug 1560770 - Use a method of getting the widget in UseMobileViewportManager() than does not require the frame tree to be constructed. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D50121
f96dbf78c2e184a1218bf47ab7d052054f39bf88: Bug 1560770 - Don't use MobileViewportManager if we're not using APZ. r=tnikkel
Botond Ballo <botond@mozilla.com> - Tue, 22 Oct 2019 04:20:07 +0000 - rev 498478
Push 114158 by ncsoregi@mozilla.com at Tue, 22 Oct 2019 09:53:30 +0000
Bug 1560770 - Don't use MobileViewportManager if we're not using APZ. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D50023
6506806b3bf81cf5093c9ce63aec76075bebf58e: Bug 1552608 - Handle fixed layer margins in ReceiveInputEvent. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:22:08 +0000 - rev 496692
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Handle fixed layer margins in ReceiveInputEvent. r=tnikkel Only touch events are handled at this time. Differential Revision: https://phabricator.services.mozilla.com/D48385
3c3c35952f7bab3c1ba896fe48636ba90c3ac0ae: Bug 1552608 - Include the fixed-position sides in HitTestResult. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:16:51 +0000 - rev 496691
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Include the fixed-position sides in HitTestResult. r=tnikkel Populating them is hooked up for non-WebRender, with a comment outlining possible implementation strategies for WebRender. Differential Revision: https://phabricator.services.mozilla.com/D48384
824e732b442a3aafe01f99972bb9f1eaff40de26: Bug 1552608 - Introduce a HitTestResult structure for grouping the results of APZ hit testing. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:47:17 +0000 - rev 496690
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Introduce a HitTestResult structure for grouping the results of APZ hit testing. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48383
5b65905fba5c3b3200ce8802682b010f930a046c: Bug 1552608 - Allow HitTestingTreeNodeAutoLock to be moved. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:04:23 +0000 - rev 496689
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Allow HitTestingTreeNodeAutoLock to be moved. r=tnikkel Also remove the MOZ_RAII annotation, as that inhibits moving. Differential Revision: https://phabricator.services.mozilla.com/D48382
5482a581ab190f9b18c06d517cce575892ba0df7: Bug 1552608 - Remove the unused function APZCTreeManager::HitTestAPZC(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:01:52 +0000 - rev 496688
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Remove the unused function APZCTreeManager::HitTestAPZC(). r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48381
50fb72cc4a73351db1d17bb4970f007f55348962: Bug 1552608 - Remove the 'aNode' parameter of GetAPZCAtPoint(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:00:59 +0000 - rev 496687
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Remove the 'aNode' parameter of GetAPZCAtPoint(). r=tnikkel This is a relic from earlier times when GetAPZCAtPoint() was recursive. Differential Revision: https://phabricator.services.mozilla.com/D48380
ff8e1fa23dea4b16e6b6b6925a723211d5488068: Bug 1552608 - Handle fixed layer margins in ComputeTransformForNode(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 22:00:31 +0000 - rev 496686
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Handle fixed layer margins in ComputeTransformForNode(). r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48379
7c1b57fdf9e2c50382494fd990d22bdb97dfd7ee: Bug 1552608 - Add a helper APZCTreeManager::IsFixedToRootContent(node). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:56:36 +0000 - rev 496685
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Add a helper APZCTreeManager::IsFixedToRootContent(node). r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48378
597c42949166fca5e3c0fb1feb5134bd3df113c3: Bug 1552608 - Make APZCTreeManager::GetTargetAPZC(layersId, scrollId) const. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:54:54 +0000 - rev 496684
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Make APZCTreeManager::GetTargetAPZC(layersId, scrollId) const. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48377
00824be6572bcedeb50559501811e170fb488595: Bug 1552608 - Factor out a helper AsyncCompositionManager::ComputeFixedMarginsOffset(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:52:49 +0000 - rev 496683
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Factor out a helper AsyncCompositionManager::ComputeFixedMarginsOffset(). r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48376
67c4bc35fc743489058d128eb4ee4df0d2a03312: Bug 1552608 - Propagate the fixed-position sides to the hit testing tree. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:41:20 +0000 - rev 496682
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Propagate the fixed-position sides to the hit testing tree. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48375
a8a4f618d3d3efa4225554cd14bbe7ad62399e3b: Bug 1552608 - Use SideBits rather than int32_t to represent fixed-position sides in the Layers API. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:03:52 +0000 - rev 496681
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Use SideBits rather than int32_t to represent fixed-position sides in the Layers API. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48374
0674fa5af30f334bdd09608f2af83f897ab53d43: Bug 1552608 - Define bitwise operators for SideBits. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:01:53 +0000 - rev 496680
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Define bitwise operators for SideBits. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48373
6310462ffafb3bce9278e21e91cb903c89bc9b2e: Bug 1552608 - Fix include-what-you-use violations in LayerAttributes.h. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 21:04:24 +0000 - rev 496679
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Fix include-what-you-use violations in LayerAttributes.h. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48372
daefc4088b5bb8121e2f6ff3f7ca046eb3f1feb8: Bug 1552608 - Store fixed layer margins in APZCTreeManager. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 20:44:29 +0000 - rev 496678
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Store fixed layer margins in APZCTreeManager. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48371
f8a125f2969e4fad5b3aecb5fdecaa990bed84a5: Bug 1552608 - Factor out a helper CompositorBridgeParent::SetFixedLayerMargins(). r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 20:43:23 +0000 - rev 496677
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Factor out a helper CompositorBridgeParent::SetFixedLayerMargins(). r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D48370
7bbff3ca9c531179101f3076a2d973d893dd5a93: Bug 1552608 - Define AsyncCompositionManager::mFixedLayerMargins and its getter/setter on all platforms. r=tnikkel
Botond Ballo <botond@mozilla.com> - Mon, 07 Oct 2019 20:37:09 +0000 - rev 496676
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1552608 - Define AsyncCompositionManager::mFixedLayerMargins and its getter/setter on all platforms. r=tnikkel It's still only used on Android, but keeping it and all code that interacts with it inside an #ifdef is too much trouble for a tiny amount of space saving. Differential Revision: https://phabricator.services.mozilla.com/D48369
7c80af009dee30518cc6eb0cfce311f2639400d1: Bug 1585858 - Remove nsChangeHint_SyncFrameView. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 07 Oct 2019 22:29:01 +0000 - rev 496670
Push 114145 by apavel@mozilla.com at Tue, 08 Oct 2019 11:00:56 +0000
Bug 1585858 - Remove nsChangeHint_SyncFrameView. r=tnikkel It's always used along with nsChangeHint_RepaintFrame, which does most of the work. It's only useful to skip calling SyncFrameViewProperties(). That call is really cheap if nothing actually changed, furthermore since only a handful of frames actually have views. So it doesn't seem like it serves any useful purpose. Differential Revision: https://phabricator.services.mozilla.com/D48003
db3e0362db018cd679e249b0ba220d80dd95ecd0: Bug 1583647 - Make the GIF decoder use the accelerated RGB unpacking. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Wed, 25 Sep 2019 23:31:40 +0000 - rev 496107
Push 114141 by rmaries@mozilla.com at Thu, 03 Oct 2019 09:42:28 +0000
Bug 1583647 - Make the GIF decoder use the accelerated RGB unpacking. r=tnikkel Differential Revision: https://phabricator.services.mozilla.com/D47063