searching for reviewer(kats)
367d47729181b6a4c3562368e5f787a2a6dbc2fa: Bug 1647156 - Improve the displayport snapping logic. r=kats, a=jcristau
Nicolas Silva <nsilva@mozilla.com> - Thu, 02 Jul 2020 16:14:51 +0000 - rev 601914
Push 13350 by jcristau@mozilla.com at Fri, 03 Jul 2020 15:09:58 +0000
Bug 1647156 - Improve the displayport snapping logic. r=kats, a=jcristau Aligning the displayport position in large increments can cause issues when the displayport is small enough to be smaller than the alignment itself. The risk is higher for small frames with no displayport margin as the alignment might have previously be rounded up to 512. Luckily not having margins is an indication that we aren't trying hard to benefit from APZ and a large alignment isn't benefiting us. This patch takes that into consideration and defaults to small alignements when there is no margins. The alignment is also rounded to a multiple of 256 instead of 512 to further reduce the likelihood of running into issues with small displayports. Differential Revision: https://phabricator.services.mozilla.com/D80614
d6deac041d1a6046d680bee085a95c91f4adacc2: Bug 1647474 - Wait for frame callbacks on the right window. r=kats
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 25 Jun 2020 16:00:43 +0000 - rev 601520
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1647474 - Wait for frame callbacks on the right window. r=kats Quite obvious in hindsight :( Differential Revision: https://phabricator.services.mozilla.com/D81128
8bde5b74dd1b69c2929294514ef9c4211c3104a8: Bug 1647871 - Remove nsIWidget::GetCompositionSize(). r=kats
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 24 Jun 2020 13:30:45 +0000 - rev 601272
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1647871 - Remove nsIWidget::GetCompositionSize(). r=kats No longer called. This was done as an optimization for OOP iframes, but it affects the scrollport so it's clearly not sound (the visible rect shouldn't affect the layout scroll port). If very tall OOP iframes are a problem somehow, it's something that we need to deal with in another place. It was, in fact, removed for top-level remote iframes because of bug 1554861 and other regressions. Depends on D80731 Differential Revision: https://phabricator.services.mozilla.com/D80732
345be10ae3f6fa33b5297e80217feaf40bb87a11: Bug 1647871 - Remove AutoResizeReflowSquasher. r=kats
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 24 Jun 2020 13:29:18 +0000 - rev 601271
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1647871 - Remove AutoResizeReflowSquasher. r=kats After the above, I don't think this is needed anymore, because we shouldn't be looking at the widget size from layout. It also shouldn't cause more reflows on desktop at least, because of the early out in ResizeReflowIgnoreOverride before calling SimpleResizeReflow(). Differential Revision: https://phabricator.services.mozilla.com/D80731
730dac40b4ba32e0f225b4bb5b3b380407ed8e69: Bug 1647871 - Simplify content viewer composition size code. r=kats,hiro
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 24 Jun 2020 14:46:40 +0000 - rev 601269
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1647871 - Simplify content viewer composition size code. r=kats,hiro This should be equivalent because: * Content viewer size and widget size should eventually be the same always (modulo mid-resize, see bug 1645954). * Only the root content document has a widget on android (the puppet widget). * Only the root content document has a dynamic toolbar. * We were only calling this on the root document, which has no parent pres context (at least with e10s). Without e10s we could have a chrome prescontext (not sure it that ends up being a parent in the pres context chain), but we should never change the resolution of that one, so I think aScaleContentViewerSize is just useless. This avoids having two sources of truth by always checking the content viewer size without subtracting the dynamic toolbar max size. Differential Revision: https://phabricator.services.mozilla.com/D80723
9a8d28cd15ba4b9c292f24de45907480352f5490: Bug 1646243 - Guard against null state in UiCompositorControllerParent::Initialize(). r=kats
Jamie Nicol <jnicol@mozilla.com> - Thu, 18 Jun 2020 16:17:34 +0000 - rev 600534
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646243 - Guard against null state in UiCompositorControllerParent::Initialize(). r=kats Differential Revision: https://phabricator.services.mozilla.com/D80205
3613d0c141e8250a1684947ea9b15a970c9e47dc: Bug 1645817 - Use overlay scrollbars win browser resolution test. r=kats
Eitan Isaacson <eitan@monotonous.org> - Thu, 18 Jun 2020 11:21:41 +0000 - rev 600415
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1645817 - Use overlay scrollbars win browser resolution test. r=kats Using overlay scrollbars will avoid the need to account for the trimmed space that is introduced after an overflowing zoom caused by scrollbars. Differential Revision: https://phabricator.services.mozilla.com/D80102
bd42396d45d7ec68adb9d248f88aad70d67f6738: Bug 1467765 - Sample off-main-thread animations on the render backend thread in case of WebRender. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 23:47:20 +0000 - rev 600326
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1467765 - Sample off-main-thread animations on the render backend thread in case of WebRender. r=kats Now CompositorAnimationStorage::GetAnimatedValue() and SetAnimatedValue()s are called on the sampler thread in case of WebRender, are called on the compositor thread in case of non WebRender, so we drop assertions of IsInCompositorThread check there. A mLock.AssertCurrentThreadOwns call in each function should make sure the function gets called on the sampler/compositor thread with acquiring the lock. One caveat in this change is that in case we try to get an animation value via nsIDOMWindowUtils.getOMTAStyle(), we do sample animations on the _compositor_ thread and we never call UpdateDynamicProperties, which means if it gets called in testing refresh driver mode, visual results will differ from what the value returned by the getOMTAStyle should look like. But it should be fine because we disallow using any chrome priviledge APIs in reftests and also we will never use the testing refresh driver mode in the reftest harness. Also in mochitests the visual results' differences might make people confusing if the person can notice it, but in principal getOMTAStyle() is designed to get computed animating values so that it doesn't matter what the visual result is. Differential Revision: https://phabricator.services.mozilla.com/D79982
37cf2bf562f8273bf9ec9bb9df265bad9d61d7de: Bug 1467765 - Factor out resetting previous time stamp stuff. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 23:32:22 +0000 - rev 600325
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1467765 - Factor out resetting previous time stamp stuff. r=kats |mPreviousFrameTimeStamp| will be moved into the sampler for OTMA in the next commit though. Differential Revision: https://phabricator.services.mozilla.com/D79981
b43c4bccbe3d4a43af65b8b2158d765f2403bf7e: Bug 1467765 - Factor out clear animation resources stuff. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 23:32:10 +0000 - rev 600324
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1467765 - Factor out clear animation resources stuff. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79980
298a43997d22c2804a31bb3c6c0027fd28a0a0f3: Bug 1646263 - Introduce a lock in CompositorAnimationStorage. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:39:50 +0000 - rev 600291
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Introduce a lock in CompositorAnimationStorage. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79949
8dc5e1f8a7324d4ba783dcb393fcd2fe40b2f34c: Bug 1646263 - Make internal-only methods of CompositorAnimationStorage private. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:39:42 +0000 - rev 600290
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Make internal-only methods of CompositorAnimationStorage private. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79948
68a63d9b0f1b44f88a64392050024eb5b581c3f2: Bug 1646263 - Move the non WebRender version of SampleAnimations into CompositorAnimationStorage. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:39:40 +0000 - rev 600289
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Move the non WebRender version of SampleAnimations into CompositorAnimationStorage. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79947
a42890da3bb98a8d51ee995c6fab0e4b6e2e3ac6: Bug 1646263 - Add CompositorAnimationStorage::CollectWebRenderAnimations. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:39:35 +0000 - rev 600288
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Add CompositorAnimationStorage::CollectWebRenderAnimations. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79946
a7100d508ee2e89ceeca9c754a19274bf72f43de: Bug 1646263 - Move the WebRender version of AnimationHelper::SampleAnimations into CompositorAnimationStorage. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:39:33 +0000 - rev 600287
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Move the WebRender version of AnimationHelper::SampleAnimations into CompositorAnimationStorage. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79945
e141e68a3f81cf4f35ec01538bad6df0939c1323: Bug 1646263 - Separate CompositorAnimationStorage from AnimationHelper.[cpp,h]. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:38:30 +0000 - rev 600286
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Separate CompositorAnimationStorage from AnimationHelper.[cpp,h]. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79944
c270926a8abd6341c39da8c8124e1ed1c636a2bb: Bug 1646263 - Use a proper name for NS_NewRunnableFunction in APZSampler. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:38:13 +0000 - rev 600285
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Use a proper name for NS_NewRunnableFunction in APZSampler. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79943
5af581c74d4b81694445ad92854b9406bb99ef28: Bug 1646263 - Drop nsTArray.h from APZSampler.h. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 17 Jun 2020 22:38:00 +0000 - rev 600284
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646263 - Drop nsTArray.h from APZSampler.h. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79942
7d080876c7e29a86d025b3b5c73108f27dc4f6d6: Bug 1635472 - Move the displayport in larger increments with WebRender. r=kats
Nicolas Silva <nsilva@mozilla.com> - Tue, 16 Jun 2020 17:41:56 +0000 - rev 599986
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1635472 - Move the displayport in larger increments with WebRender. r=kats With this change the displayport moves by screen (or a scroll-frame) up to a quarter of the displayport size instead of every 128 pixels when WebRender is enabled. Also separate the alignment of the position and size to allow moving the displayport in much larger increments without increasing the displayport size which regresses tscrollx and generally increases webrender's CPU load. Differential Revision: https://phabricator.services.mozilla.com/D77870
f719a8ab74c6cb79b1ce477a09f83b21aedef4a9: Bug 1572508 - Convert `ui.touch.*` and `ui.mouse.*` VarCache prefs to static prefs. r=KrisWright,kats,geckoview-reviewers,agi.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 15 Jun 2020 06:23:48 +0000 - rev 599714
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1572508 - Convert `ui.touch.*` and `ui.mouse.*` VarCache prefs to static prefs. r=KrisWright,kats,geckoview-reviewers,agi. This is a weird conversion because the existing VarCache mirror prefs are in a two static structs, one for touch prefs and one for the equivalent mouse prefs, and we dynamically select the appropriate struct in `GetPrefsFor()`. (But note that `ui.mouse.radius.reposition` and `ui.mouse.radius.inputSource.touchOnly` do not have corresponding `touch` prefs! For touch events they are always considered to be false.) The commit make the following changes. - It makes the prefs into static prefs. - It moves the code in `GetPrefsFor()` into a constructor for `EventRadiusPrefs`, and copies the pref values into a temporary object of that type. Those copied values are then used by the subsequent code. - It removes the two static `EventRadiusPrefs` structs. - It renames some fields in `EventRadiusPrefs` to make them more closely match the pref names. Differential Revision: https://phabricator.services.mozilla.com/D79405
b21ea1dcb21caf8287cbd0f0cceaf8becf995166: Bug 1644254 - Keep a reference to the CompositorThreadHolder until gfx shutdown tasks have run. r=kats
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 13 Jun 2020 12:30:27 +0000 - rev 599673
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1644254 - Keep a reference to the CompositorThreadHolder until gfx shutdown tasks have run. r=kats Differential Revision: https://phabricator.services.mozilla.com/D79596
ab8da9fe39fe8e086a642bc967695be0bed3c6d3: Bug 1635472 - Move the displayport in largerer increments with WebRender. r=kats
Nicolas Silva <nsilva@mozilla.com> - Fri, 12 Jun 2020 08:24:16 +0000 - rev 599367
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1635472 - Move the displayport in largerer increments with WebRender. r=kats Also separate the alignment of the position and size to allow moving the displayport in much larger increments without increasing the displayport size which regresses tscrollx and generally increases webrender's CPU load. Differential Revision: https://phabricator.services.mozilla.com/D77870
4e04aefa055870b8cebf7b1d22685515dce9ae8f: Bug 1644638. ScrollFrameHelper::ReflowFinished should use GetVisualScrollRange instead of trying to calculate it itself. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Fri, 12 Jun 2020 08:12:30 +0000 - rev 599365
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1644638. ScrollFrameHelper::ReflowFinished should use GetVisualScrollRange instead of trying to calculate it itself. r=kats This avoids a bug where the scroll range can be negative. Differential Revision: https://phabricator.services.mozilla.com/D79029
d4c8b8c748ee8548278d1ebce543381691df9da3: Bug 1643604 - Move the check for a resized layout viewport in NotifylayersUpdated to the non-first-paint case. r=kats
Botond Ballo <botond@mozilla.com> - Thu, 11 Jun 2020 21:05:09 +0000 - rev 599260
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1643604 - Move the check for a resized layout viewport in NotifylayersUpdated to the non-first-paint case. r=kats In the first-paint case, we are accepting the incoming scroll metadata wholesale, so most of the consequences of this code block will happen anyways -- except for the extra repaint, which is not actually needed and in fact undesirable in the first-paint case. Differential Revision: https://phabricator.services.mozilla.com/D79195
867913a162a1b3826f92f6582c9b368822f38ab4: Bug 1644381. Remove listener in nsSliderFrame. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Tue, 09 Jun 2020 13:40:27 +0000 - rev 598890
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1644381. Remove listener in nsSliderFrame. r=kats Differential Revision: https://phabricator.services.mozilla.com/D78882
5b53507e5a9b008c17080652649002cd81238a31: Bug 1643504 - Upgrade webrender and wgpu docker images to Debian buster. r=kats
Mike Hommey <mh+mozilla@glandium.org> - Tue, 09 Jun 2020 13:35:40 +0000 - rev 598844
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1643504 - Upgrade webrender and wgpu docker images to Debian buster. r=kats Differential Revision: https://phabricator.services.mozilla.com/D78419
e2d0d6675bb54f2148f42128bd95994c4f408ed6: Bug 1642884 - Protect all access to Axis::mVelocity with a mutex. r=kats
Botond Ballo <botond@mozilla.com> - Mon, 08 Jun 2020 16:14:29 +0000 - rev 598517
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642884 - Protect all access to Axis::mVelocity with a mutex. r=kats Differential Revision: https://phabricator.services.mozilla.com/D78556
f6eee91e620a3724559192c9d2c83b32e34e022c: Bug 1643504 - Upgrade Windows searchfox task to Debian buster. r=kats
Mike Hommey <mh+mozilla@glandium.org> - Sun, 07 Jun 2020 22:29:07 +0000 - rev 598412
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1643504 - Upgrade Windows searchfox task to Debian buster. r=kats Differential Revision: https://phabricator.services.mozilla.com/D78431
7966e39bfe36563bb94867fa97095024803d1876: Bug 1643461. Fix calculation of deltaY when creating a widget wheel event from pinch input data on Windows. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Fri, 05 Jun 2020 19:45:20 +0000 - rev 598296
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1643461. Fix calculation of deltaY when creating a widget wheel event from pinch input data on Windows. r=kats The original code made the mistake of thinking that [event magnification] was a multiplicative quantity (ie if the scale changed from a to b then [event magnification] would be b/a). But it is an additive quantity (so b-a). If we then apply this knowledge to how we calculate the value from the Windows code we get the new formula. The macOS and non-macOS code could be merged because mCurrentSpan is always 100.0 on macOS, but I'm not going to do that just yet because the Windows specific code isn't mature yet, so I'm not sure if we will need to tweak it more and my goal with this change was to just get the signs right to fix the worst bug. The scale can be addressed later if needed. Differential Revision: https://phabricator.services.mozilla.com/D78479
a934a2435d2ab0e29b781bbf08021829801b9980: Bug 1642869. Check if the visual viewport differs from the layout viewport in ScrollFrameHelper::GetPageScrollAmount. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Thu, 04 Jun 2020 02:01:06 +0000 - rev 597938
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642869. Check if the visual viewport differs from the layout viewport in ScrollFrameHelper::GetPageScrollAmount. r=kats So we can preserve the scrolling behaviour that takes into account fixed "headers" and "footers". Differential Revision: https://phabricator.services.mozilla.com/D77985
02f8daecd98e4aaeb0eb2101725b49bd515764f9: Bug 1642884 - Ensure calls to Axis::CancelGesture() are protected by the APZC lock. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 03 Jun 2020 16:56:45 +0000 - rev 597812
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642884 - Ensure calls to Axis::CancelGesture() are protected by the APZC lock. r=kats Differential Revision: https://phabricator.services.mozilla.com/D78086
702cb5d63666fc667d12265e0fb1028230bfdc1c: Bug 1642788 - Ensure APZ gtests set mTimeStamp correctly on events where needed. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 03 Jun 2020 16:39:01 +0000 - rev 597808
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642788 - Ensure APZ gtests set mTimeStamp correctly on events where needed. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77981
67ce8c8ddafef497d0599d7bb353219ff176103d: Bug 1642788 - Call StartTouch() when pinch locking is activated. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 03 Jun 2020 16:38:53 +0000 - rev 597807
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642788 - Call StartTouch() when pinch locking is activated. r=kats Activating pinch locking means we are locked into a two-finger pan (no scaling), which can trigger a fling animation at the end. We track velocity via UpdateWithTouchAtDevicePoint() while pinch locked. We need to call StartTouch() when we start tracking velocity. Differential Revision: https://phabricator.services.mozilla.com/D77980
7534525df6afb4507418da9b33a9872fa51aea52: Bug 1642788 - Use mTimeStamp rather than mTime for APZ velocity computations. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 03 Jun 2020 16:38:38 +0000 - rev 597806
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642788 - Use mTimeStamp rather than mTime for APZ velocity computations. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77979
124e127d05f467bed582507cc3e3db8d3100b7c7: Bug 1642817 - Avoid accessing sIndirectLayerTreesLock during shutdown. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 03 Jun 2020 13:25:41 +0000 - rev 597796
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1642817 - Avoid accessing sIndirectLayerTreesLock during shutdown. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77957
00e9f231805c249beafe57ada73c2484c6bc3a0c: Bug 1641224 - Add webrender-perf try preset. r=kats
Nicolas Silva <nsilva@mozilla.com> - Thu, 28 May 2020 15:13:26 +0000 - rev 597412
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1641224 - Add webrender-perf try preset. r=kats ./mach try --preset webrender-perf runs the following tasks: - test-linux64-shippable-qr/opt-raptor-motionmark-animometer-firefox-e10s - test-linux64-shippable-qr/opt-talos-g1-e10s - test-linux64-shippable-qr/opt-talos-g4-e10s - test-linux64-shippable-qr/opt-talos-svgr-e10s - test-linux64-shippable-qr/opt-talos-tp5o-e10s - test-linux64-shippable-qr/opt-talos-webgl-e10s - test-windows10-64-shippable-qr/opt-raptor-motionmark-animometer-firefox-e10s - test-windows10-64-shippable-qr/opt-talos-g1-e10s - test-windows10-64-shippable-qr/opt-talos-g4-e10s - test-windows10-64-shippable-qr/opt-talos-svgr-e10s - test-windows10-64-shippable-qr/opt-talos-tp5o-e10s - test-windows10-64-shippable-qr/opt-talos-webgl-e10s Differential Revision: https://phabricator.services.mozilla.com/D77265
2e62d9ec618119846e6bb09153c083ed1c49cff1: Bug 1644254 - Keep a reference to the CompositorThreadHolder until gfx shutdown tasks have run. r=kats, a=jcristau
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 13 Jun 2020 12:30:27 +0000 - rev 597148
Push 13285 by jcristau@mozilla.com at Mon, 15 Jun 2020 14:38:09 +0000
Bug 1644254 - Keep a reference to the CompositorThreadHolder until gfx shutdown tasks have run. r=kats, a=jcristau Differential Revision: https://phabricator.services.mozilla.com/D79596
e7a11bf72d1e762c5b4fe2c16dbc04b76c32b7db: Bug 1642088. Use a ResolutionChangeOrigin for when the resolution is changing by tests that overides and sets the restore resolution. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Sat, 30 May 2020 21:34:39 +0000 - rev 596890
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1642088. Use a ResolutionChangeOrigin for when the resolution is changing by tests that overides and sets the restore resolution. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77582
65c72f5bd06f60162d22e5f290c04395a3cbc85e: Bug 1641996. Put the focus point of pinch events from Direct Manipulation in widget relative coordinates. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Sat, 30 May 2020 08:33:24 +0000 - rev 596865
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1641996. Put the focus point of pinch events from Direct Manipulation in widget relative coordinates. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77576
ac8ebdcda25ba6f6bbc62c27dffc63c1e4219a12: Bug 1641996. Put the focus point of pinch events from Direct Manipulation in widget relative coordinates. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Sat, 30 May 2020 06:58:47 +0000 - rev 596861
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1641996. Put the focus point of pinch events from Direct Manipulation in widget relative coordinates. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77576
1e34d7c16a15937412017f9f56be5131a6d55d04: Bug 1641752. Don't access dead frame in AutoScrollbarRepaintSuppression. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Fri, 29 May 2020 11:19:59 +0000 - rev 596814
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1641752. Don't access dead frame in AutoScrollbarRepaintSuppression. r=kats Differential Revision: https://phabricator.services.mozilla.com/D77403
7ced9547a2b33307774e5292aa9f787109633267: Bug 1625693 - Fix mouse movement position tracking in examples. r=kats
Gabriel D'Alimonte <gabriel.dalimonte@gmail.com> - Mon, 25 May 2020 15:38:34 +0000 - rev 595797
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1625693 - Fix mouse movement position tracking in examples. r=kats [import_pr] From https://github.com/servo/webrender/pull/3961 Differential Revision: https://phabricator.services.mozilla.com/D76716
6191dcf6f130dd30fe25b93836231b0d122bc3e9: Bug 1636273. Fix calculation of scroll position in AsyncPanZoomController::HandleDragEvent. r=kats,botond
Timothy Nikkel <tnikkel@gmail.com> - Thu, 21 May 2020 02:41:41 +0000 - rev 595550
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1636273. Fix calculation of scroll position in AsyncPanZoomController::HandleDragEvent. r=kats,botond The scroll position and scrollable rect are inside the zoom boundary so we need to use the composisted size inside the zoom boundary as well. Differential Revision: https://phabricator.services.mozilla.com/D76089
2d00a1a6495c9b0350c4a7d601df64b49355e701: Bug 1639448. When a visual viewport is set the page scroll amount for a scroll frame is incorrect. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Wed, 20 May 2020 13:35:13 +0000 - rev 595129
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1639448. When a visual viewport is set the page scroll amount for a scroll frame is incorrect. r=kats We want to use a page scroll amount based on the visual viewport size. The extra complications we have for not scrolling too far if there are fixed/sticky position things are quite hard to do properly with a visual viewport (I tried). I think there is far less user expectation of it working if a visual viewport is set. If we get complaints we can look into it. Differential Revision: https://phabricator.services.mozilla.com/D76091
0aace53849df2a45a937aa41676617a5f2cbded5: Bug 1639076 - Drop PostTranslate call for transform animations on WebRender. r=kats
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Tue, 19 May 2020 21:06:30 +0000 - rev 594636
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1639076 - Drop PostTranslate call for transform animations on WebRender. r=kats On WebRender |transformData.origin()| value is always zero, we don't need to explicitly call it at all. [1] https://searchfox.org/mozilla-central/rev/9193635dca8cfdcb68f114306194ffc860456044/gfx/layers/AnimationInfo.cpp#678-680 Differential Revision: https://phabricator.services.mozilla.com/D75883
797e4e07bcdfc6ba9f55bda8923af009711b2b9b: Bug 1636911. Add mochitest. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Mon, 18 May 2020 10:26:28 +0000 - rev 594322
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1636911. Add mochitest. r=kats Differential Revision: https://phabricator.services.mozilla.com/D75665
2b12448f0d622b08dc952c0eae20c35b8bcc5b27: Bug 1636911. In ScrollFrameHelper::RestrictToRootDisplayPort don't adjust the root display port by the root scroll frame apz callback transform because it already factors that in. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Mon, 18 May 2020 10:26:16 +0000 - rev 594321
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1636911. In ScrollFrameHelper::RestrictToRootDisplayPort don't adjust the root display port by the root scroll frame apz callback transform because it already factors that in. r=kats Differential Revision: https://phabricator.services.mozilla.com/D75526
37e68bff6c62be9e13f75685252445bece3f421c: Bug 1636911. Add mochitest. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Mon, 18 May 2020 06:32:34 +0000 - rev 594281
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1636911. Add mochitest. r=kats Differential Revision: https://phabricator.services.mozilla.com/D75665
74a83c2492318f6b1a48f7d771b8c5020195de89: Bug 1636911. In ScrollFrameHelper::RestrictToRootDisplayPort don't adjust the root display port by the root scroll frame apz callback transform because it already factors that in. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Mon, 18 May 2020 06:32:34 +0000 - rev 594280
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1636911. In ScrollFrameHelper::RestrictToRootDisplayPort don't adjust the root display port by the root scroll frame apz callback transform because it already factors that in. r=kats Differential Revision: https://phabricator.services.mozilla.com/D75526
ecb10a20343f79a0c0f2e364372d0c5934cb2f0f: Bug 1638535. APZCTreeManager::SetTargetAPZC needs to make sure it uses a zoomable apzc if it's processing pinch gesture input. r=kats
Timothy Nikkel <tnikkel@gmail.com> - Sun, 17 May 2020 11:36:12 +0000 - rev 594253
Push 13186 by ffxbld-merge at Mon, 01 Jun 2020 09:52:46 +0000
Bug 1638535. APZCTreeManager::SetTargetAPZC needs to make sure it uses a zoomable apzc if it's processing pinch gesture input. r=kats Differential Revision: https://phabricator.services.mozilla.com/D75670