Keyboard APZ has landed on Inbound

2017-07-21 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
the right tracking bug? Doesn't seem related at a > glance. Whoops, the correct tracking bug is bug 1376525. Copied the wrong one. From: dev-platform on behalf of Emilio Cobos Álvarez Sent: Saturday, July 22, 2017 12:43:31 PM To: Ryan Hunt; dev-platfor

Re: Keyboard APZ has landed on Inbound

2017-07-23 Thread Ryan Hunt
rom: Ben Kelly 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 mailto:rh...@eqrion.net>> wrote: Keyboard APZ can't be used in every case. Currently it's d

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

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 m

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-14 Thread Ryan Hunt
tent? What is a > 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 ‐‐‐ > >

Re: Intent to Implement: css-scroll-anchoring

2018-11-17 Thread Ryan Hunt
2018 22:05: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.mozi

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 b

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: /* static

Re: C++ method definition comments

2019-01-28 Thread Ryan Hunt
own lines. > 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 >> t

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 > k

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 f

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 -> PBrowserBr

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 pat

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