Re: Intent to ship: MouseEvent.offsetX/Y

2015-03-01 Thread Boris Zbarsky
On 3/2/15 1:00 AM, Chris Peterson wrote: Gecko's offsetX/Y could return doubles, for compatibility with the spec, that are always snapped to whole numbers, for compatibility with other browsers. How, precisely, is this different from returning integers? Remember, this is JS we're talking abou

Re: Intent to ship: MouseEvent.offsetX/Y

2015-03-01 Thread Chris Peterson
On 2/27/15 7:25 PM, Robert O'Callahan wrote: On Sat, Feb 28, 2015 at 8:30 AM, Jeff Muizelaar wrote: >On Fri, Feb 27, 2015 at 2:21 PM, Robert O'Callahan >wrote: > >Oh, another issue is that I've followed the spec and made offsetX/Y > >doubles, whereas Blink is integers, which introduces a small

Re: Syncing 2 scrolling operations

2015-03-01 Thread Paul Rouget
(forgot to cc dev.platform in my reply) On Sat, Feb 28, 2015 at 3:47 AM, Paul Rouget wrote: >> So in order to implement the behaviour you're looking for we would >> definitely need to make some changes in gecko. > > How much work is that? Is it just a matter of "piping" things together > to expo

Re: Syncing 2 scrolling operations

2015-03-01 Thread Paul Rouget
(forgot to cc dev.platform in my reply) On Fri, Feb 27, 2015 at 11:57 AM, Paul Rouget wrote: > Hi Kats. Thanks for taking a look at this. > > On Thu, Feb 26, 2015 at 10:21 PM, wrote: >> On Thursday, February 26, 2015 at 1:06:15 PM UTC-5, Paul Rouget wrote: >>> I need a pretty picture to explain

dev-platform@lists.mozilla.org

2015-03-01 Thread smaug
On 03/02/2015 01:11 AM, Xidorn Quan wrote: On Mon, Mar 2, 2015 at 9:50 AM, Boris Zbarsky wrote: On 3/1/15 5:04 PM, Xidorn Quan wrote: Hence I think we should remove this method. All callees should use either AssignLiteral(MOZ_UTF16("some string")), or, if don't want to bloat the binary, expl

Re: Intent to ship: MouseEvent.offsetX/Y

2015-03-01 Thread smaug
On 02/28/2015 05:25 AM, Robert O'Callahan wrote: On Sat, Feb 28, 2015 at 8:30 AM, Jeff Muizelaar wrote: On Fri, Feb 27, 2015 at 2:21 PM, Robert O'Callahan wrote: Oh, another issue is that I've followed the spec and made offsetX/Y doubles, whereas Blink is integers, which introduces a small a

dev-platform@lists.mozilla.org

2015-03-01 Thread Xidorn Quan
On Mon, Mar 2, 2015 at 9:50 AM, Boris Zbarsky wrote: > On 3/1/15 5:04 PM, Xidorn Quan wrote: > >> Hence I think we should remove this method. All callees should use either >> AssignLiteral(MOZ_UTF16("some string")), or, if don't want to bloat the >> binary, explicitly use AssignASCII("some string

dev-platform@lists.mozilla.org

2015-03-01 Thread Boris Zbarsky
On 3/1/15 5:04 PM, Xidorn Quan wrote: Hence I think we should remove this method. All callees should use either AssignLiteral(MOZ_UTF16("some string")), or, if don't want to bloat the binary, explicitly use AssignASCII("some string"). The latter requires an strlen() that AssignLiteral optimizes

Re: Jemalloc 3 is now on by default

2015-03-01 Thread Mike Hommey
On Tue, Jan 13, 2015 at 12:44:19PM +0900, Mike Hommey wrote: > On Tue, Jan 13, 2015 at 10:13:34AM +0900, Mike Hommey wrote: > > Hi, > > > > I just landed bug 762449 on inbound, which enables Jemalloc 3 on the > > nightly channel. This puts us a step forward to removing mozjemalloc, > > our highly

dev-platform@lists.mozilla.org

2015-03-01 Thread Xidorn Quan
I think we should remove the method nsAString::AssignLiteral(const char (&aStr)[N]) because its name is misleading. The method name AssignLiteral can easily make people at the callee side think it makes the string point to a piece of static data, which has no runtime penalty. But this is false. T