Comment on attachment 8425857 [details] [diff] [review] patch v1 https://tbpl.mozilla.org/?tree=Try&rev=c4662b4e6ae2 push contains also patches from bug 1013333 and bug 1013587

Comment on attachment 8425857 [details] [diff] [review] patch v1 Review of attachment 8425857 [details] [diff] [review]: ----------------------------------------------------------------- Try is green and I didn't find any problems locally (today). Let's wait here how the mobile memory regression evolves first, I don't want to put to many change in there one-by-one.

So, since it's just a week to the merge, I think we should land this potentially dangerous patch after the merge. It's actually blocking only a perf optimization, otherwise the code is stable despite this omission.

Michal, can you please land this patch? I can do it for you if you can't and land it together with bug 1013587.

Going to land (confirmed on IRC).

Comment on attachment 8425857 [details] [diff] [review] patch v1 [Approval Request Comment] Bug caused by (feature/regressing bug #): 913807 User impact if declined: blocks significant cache load optimization (bug 1013587) Testing completed (on m-c, etc.): more than 10 days on m-c Risk to taking this patch (and alternatives if risky): low String or IDL/UUID changes made by this patch: none

Comment on attachment 8425857 [details] [diff] [review] patch v1 Dropping the request until bug 1026951 cause is known (can be related).