Talos Improvement! Trace Malloc MaxHeap decrease 0.499% on CentOS release 5 (Final) Mozilla-Inbound

[Nobody]

Improvement! Trace Malloc MaxHeap decrease 0.499% on CentOS release 5 (Final) Mozilla-Inbound
---------------------------------------------------------------------------------------------
Previous: avg 26698593.333 stddev 62026.585 of 30 runs up to revision 0601f686e1b8
New : avg 26565460.000 stddev 918.150 of 5 runs since revision a8115574617b
Change : -133133.333 (0.499% / z=2.146)
Graph : http://mzl.la/zn9yX9

Changeset range: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=0601f686e1b8&tochange=a8115574617b

Changesets:
* http://hg.mozilla.org/integration/mozilla-inbound/rev/79c6b50a95c5
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Indicate free bits in nsIXPCScriptable.idl. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/9592334d4aff
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Introduce a flag for global natives and assert that we have it. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/cf7d53530360
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Remove unnecessary aIID parameter from nsIXPConnect::InitClassesWithNewWrappedGlobal. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/2880b8cc9551
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Stop passing aExtraPtr to InitClassesWithNewWrapedGlobal. r=mrbkap
We only ever pass null here anyway, and this whole infrastructure is going away real soon now.
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/e7228f6767ae
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Stop passing isGlobal everywhere and use the nsIXPCScriptable flags instead. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/0fd1e730ebd3
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Move the AutoMarkingWrappedNativePtr in XPCWrappedNative::{GetNewOrUsed,Morph} closer to where we need it. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/36df6e53c4a1
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Use nsRefPtr instead of manual AddRef/Release in XPCWrappedNative::{GetNewOrUsed,Morph}. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/951150c4758a
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Modify the semantics of XPCWrappedNative::FinishCreate() so that it doesn't muck around with the refcounts of its inparam. r=mrbkap
The current semantics involve the caller transfering ownership of the input wrapper to FinishCreate(), which is complicated and error-prone. Now that we're using nsRefPtr at the callsites, we can get rid of it.
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/32ae32597625
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Make JSID class initialization happen on demand. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/3ebefca0bf40
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Manually resolve |Object| on the global in XPCWrappedNativeScope::SetGlobal() to avoid calling into the XPConnect resolve hook too early. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/a17c3b23a664
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Accept an explicit native global during XPCWrappedNativeScope creation. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/62b27d043ec8
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Factor out the call to PostCreatePrototype into a helper method. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/d6e1981c7d3e
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Make the call to PostCreatePrototype optional. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/b5abccf004e2
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Call TraceDOMPrototypes via TraceXPCGlobal, and avoid the verification process for wrapped natives. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/592a35a4e03d
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Introduce XPCWrappedNative::WrapNewGlobal. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/8277b919772d
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Kill temporary globals. r=mrbkap
This patch breaks the "every commit is a 100% correct tree" invariant a little bit, because constructors and prototypes for DOM globals are broken (PostCreatePrototype never gets called). This is fixed in the next patch. Doing it this way makes for a cleaner series of commits, and the browser still builds and runs fine.
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/119787ed1514
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Do the last bit of wrapped global initialization once the dust has settled. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/56c192395dd3
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Assert against creating globals in xpc_NewSystemInheritingJSObject. r=mrbkap
With this patch, the only call to JS_NewGlobalObject in XPConnect happens in xpc_CreateGlobalObject. \o/
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

* http://hg.mozilla.org/integration/mozilla-inbound/rev/a8115574617b
: Bobby Holley <bobby...@gmail.com> - Bug 720580 - Assert that we have the global jsclass flag if and only if we're a global. r=mrbkap
: http://bugzilla.mozilla.org/show_bug.cgi?id=720580

Bugs:
* http://bugzilla.mozilla.org/show_bug.cgi?id=720580

[Nobody]

Improvement! Trace Malloc MaxHeap decrease 0.329% on CentOS (x86_64) release 5 (Final) Mozilla-Inbound
------------------------------------------------------------------------------------------------------
Previous: avg 36623816.667 stddev 70040.555 of 30 runs up to revision 0601f686e1b8
New : avg 36503500.000 stddev 1242.980 of 5 runs since revision a8115574617b
Change : -120316.667 (0.329% / z=1.718)
Graph : http://mzl.la/AoxayT

[Nobody]

Regression :( Dromaeo (jslib) decrease 2.43% on Win7 Mozilla-Inbound-Non-PGO
----------------------------------------------------------------------------
Previous: avg 145.708 stddev 1.351 of 30 runs up to revision 0601f686e1b8
New : avg 142.165 stddev 0.597 of 5 runs since revision a8115574617b
Change : -3.543 (2.43% / z=2.622)
Graph : http://mzl.la/AoGbEm