On 2/12/2013 3:08 PM, Ed Morley wrote:
On 12 February 2013 22:11:12, Stephen Pohl wrote:
I wanted to give a heads up that we're in the process of finalizing
the patch for bug 678392 which will give us history swipe animations
on Mac OSX 10.7+. Since we will be taking snapshots of the 20
most-recently visited pages, this will undoubtedly lead to an increase
in memory utilization on these platforms.
To save everyone having to look at the graph - the initial landing
showed a consistent 20% regression in trace malloc maxheap. If this were
a 1-5% regression, then I think it would be worth discussing the
trade-off. At 20%, I really don't see how we can take this, sorry! :-(
Ed
I don't see how we can *not* take this. Of course it's going to mean
using more memory. If it doesn't leak, and it doesn't put us over some
magic limit where a significant portion of our users end up paging or
something like that, then I don't see how we can reject it.
Without context, 1-5% or 20% growth are just meaningless numbers. The
context here is not some accidental regression or a feature doing
something horribly wrong with memory. This is simply a memory-expensive
feature and it's a feature we *must* land.
I'm all for smart people looking for ways to get this memory usage as
low as it can be without undermining the value of the feature, but if we
cannot find those wins, we should land this as it is.
- A
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
