Keyboard APZ has landed on Inbound

2017-07-22 Thread Ryan Hunt
The patch to enable async keyboard scrolling in nightly (for all platforms except Android) has landed on inbound. Once this is merged to central, key scrolling will be done by the compositor instead of on the main thread in most cases. This should bring the responsiveness of key scrolling in line

Re: Keyboard APZ has landed on Inbound

2017-07-22 Thread Ryan Hunt
al.io> Sent: Saturday, July 22, 2017 12:43:31 PM To: Ryan Hunt; dev-platform@lists.mozilla.org Subject: Re: Keyboard APZ has landed on Inbound On 07/22/2017 08:05 AM, Ryan Hunt wrote: > Keyboard APZ can't be used in every case. Currently it's disabled in the > presense of key event listener

Re: Keyboard APZ has landed on Inbound

2017-07-24 Thread Ryan Hunt
> We will not do keyboard APZ, even if the event listener is marked passive. I filed a bug 1383900 about this with a more in depth explanation and possible ways we can get around this to use keyboard APZ more often. [1] [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1383900 > Should that be

Re: Keyboard APZ has landed on Inbound

2017-07-23 Thread Ryan Hunt
bke...@mozilla.com> Sent: Sunday, July 23, 2017 12:26 PM To: Ryan Hunt Cc: dev-platform@lists.mozilla.org Subject: Re: Keyboard APZ has landed on Inbound On Sat, Jul 22, 2017 at 2:05 AM, Ryan Hunt <rh...@eqrion.net<mailto:rh...@eqrion.net>> wrote: Keyboard APZ can't be used in every ca

OMTP enabled in nightly on OSX

2017-12-08 Thread Ryan Hunt
​Just a heads up, OMTP (or Off-Main Thread Painting) [1] has been enabled on OSX in nightly. Please file any bugs against it blocking bug 1369541. Thanks, Ryan [1] https://mozillagfx.wordpress.com/2017/12/05/off-main-thread-painting/ ___ dev-platform

Re: Intent to Implement: css-scroll-anchoring

2018-11-14 Thread Ryan Hunt
> good example website to see the effect of scroll anchoring? > > On 2018-11-14 1:09 PM, Ryan Hunt wrote: > > > Apologies. The target release is 66, while Chrome released this feature in > > M56. > > ‐‐‐ Original Message ‐‐‐ > > On Wednesday, N

Intent to Implement: css-scroll-anchoring

2018-11-14 Thread Ryan Hunt
Summary: Scroll anchoring aims to prevent user experience disruptions from content loading outside the viewport and causing the page to jump around. Bug: Bug 1305957 Link to standard: https://drafts.csswg.org/css-scroll-anchoring/ Platform coverage: All platforms Estimated or target release:

Re: Intent to Implement: css-scroll-anchoring

2018-11-14 Thread Ryan Hunt
Apologies. The target release is 66, while Chrome released this feature in M56. ‐‐‐ Original Message ‐‐‐ On Wednesday, November 14, 2018 3:05 PM, Ryan Hunt wrote: > Summary: > > Scroll anchoring aims to prevent user experience disruptions from content > loading outside

Re: Intent to Implement: css-scroll-anchoring

2018-11-17 Thread Ryan Hunt
:18 UTC+1 schrieb Ryan Hunt: > > > Summary: > > Scroll anchoring aims to prevent user experience disruptions from content > > loading outside the viewport and causing the page to jump around. > > So this will fix stone-old bugs like > https://bugzilla.mozilla.org/sh

PSA: Scroll anchoring in nightly

2019-01-10 Thread Ryan Hunt
Hi all, Scroll anchoring (bug 1305957) is on inbound and should make it into nightly soon. If you're not familiar, take a look at the intent-to-implement post [1]. This change has the potential for web compatibility issues. If you notice a strange change in scrolling behavior, please file a bug

C++ method definition comments

2019-01-25 Thread Ryan Hunt
Hi all, Quick C++ style question. A common pattern in Gecko is for method definitions to have a comment with the 'static' or 'virtual' qualification. Before the reformat, the comment would be on it's own separate line [1]. Now it's on the main line of the definition [2]. For example: /*

Re: C++ method definition comments

2019-03-01 Thread Ryan Hunt
they were all formatted to be on the same line. [1] https://searchfox.org/mozilla-central/diff/265e6721798a455604328ed5262f430cfcc37c2f/layout/base/nsLayoutUtils.cpp#1098 Thanks, Ryan ‐‐‐ Original Message ‐‐‐ On Wednesday, January 30, 2019 10:17 AM, Ryan Hunt wrote: > I've filed

Re: C++ method definition comments

2019-01-28 Thread Ryan Hunt
. > Would you be OK with that? > > Thanks, > Ehsan > > On Fri, Jan 25, 2019 at 11:49 PM Ryan Hunt wrote: > >> Hi all, >> >> Quick C++ style question. >> >> A common pattern in Gecko is for method definitions to have a comment with >> the >> 'static'

Re: C++ method definition comments

2019-01-30 Thread Ryan Hunt
I've filed bug 1523969 to consider making this change. (https://bugzilla.mozilla.org/show_bug.cgi?id=1523969) ‐‐‐ Original Message ‐‐‐ On Monday, January 28, 2019 6:27 PM, Ryan Hunt wrote: > Yeah, personally I have found them be useful and don't have an issue with > keeping >

DOM IPC actor renames for Fission

2019-04-26 Thread Ryan Hunt
Hi all, Over the last month I have slowly been landing patches to standardize the naming of our DOM IPC actors in the wake of changes made for Fission. The final major commits landed this week, so here's the summary: TabParent -> BrowserParent TabChild -> BrowserChild PRemoteFrame ->

Re: What to do about scroll anchoring?

2019-09-27 Thread Ryan Hunt
Hi, I was the one who originally implemented scroll anchoring in Firefox and just want to say that I agree with Emilio here. Scroll anchoring is a cool feature and would be great to have, but these difficulties that Emilio has mentioned have been here since day-1. It would be great to find a

Intent to ship: WebAssembly bulk-memory-operations and reference-types proposals

2020-06-24 Thread Ryan Hunt
Hi all, In bug 1528294 and bug 1637884 I intend to ship the WebAssembly bulk-memory-operations and reference-types proposals. The bulk-memory-operations proposal adds new instructions for common data manipulation operations such as 'memcpy' and 'memset'. WebAssembly code using these instructions