On 2013-01-28 11:48 PM, Jet Villegas wrote: > Agreed! Looks like Fennec Team has a plan for PanZoomController.java: > https://bugzilla.mozilla.org/show_bug.cgi?id=745136 Last conversation (just yesterday) with Kats who owns one of the actionable bugs under this one, the "plan" is a strong word, in that it may not have firm dates associated with this.
Either way though, let's just coordinate who's driving this, and we can then sort out who's doing different parts. > I think we need to get past this big one to unify the other platforms: > https://bugzilla.mozilla.org/show_bug.cgi?id=825928 > > --Jet > > ----- Original Message ----- > From: "Anthony Jones" <ajo...@mozilla.com> > To: dev-platform@lists.mozilla.org > Sent: Monday, January 28, 2013 8:14:19 PM > Subject: Scrolling, Scrolling and more Scrolling > > I've spent several weeks fixing scrolling and zooming bugs on b2g. You > may have enjoyed bug 831973 (and it's duplicates) over the last week. In > bug 811950 it took -9 lines to introduce the bug and a further -27 lines > to fix it. Perhaps we have too much scrolling code: > > * AsyncPanZoomController.cpp > * BrowserElementScrolling.js > * PanZoomController.java > * nsFrameLoader.cpp > > There is a fair amount of duplication here. Ideally we should have a > single async implementation with a good set of tests. Removing the > duplication is going to require some collaboration. > > Are there other scrolling implementations that I've missed? > > Who is responsible for each of these implementations? > > What will it take to get to a single implementation? > > Anthony > > > > > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
