e158708d4ca309e32095175626d856c7861c43e5: Bug 1193201 - Incorrect version displayed (should be 43 and not 42) r=bhearsum
Sylvestre Ledru <sledru@mozilla.com> - Tue, 11 Aug 2015 15:36:14 +0200 - rev 257248
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1193201 - Incorrect version displayed (should be 43 and not 42) r=bhearsum
cd84db0152b02a85e005058d19fe02a25cb1b012: No bug: Make ./mach clobber work on windows without winrm.exe. r=me
Kyle Huey <khuey@kylehuey.com> - Tue, 11 Aug 2015 06:10:46 -0700 - rev 257247
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
No bug: Make ./mach clobber work on windows without winrm.exe. r=me
a13c1f26e351dd6251da641fe7a9eb53790fc2d0: Bug 1179909: Refactor stable state handling. r=smaug
Kyle Huey <khuey@kylehuey.com> - Tue, 11 Aug 2015 06:10:46 -0700 - rev 257246
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1179909: Refactor stable state handling. r=smaug This is motivated by three separate but related problems: 1. Our concept of recursion depth is broken for things that run from AfterProcessNextEvent observers (e.g. Promises). We decrement the recursionDepth counter before firing observers, so a Promise callback running at the lowest event loop depth has a recursion depth of 0 (whereas a regular nsIRunnable would be 1). This is a problem because it's impossible to distinguish a Promise running after a sync XHR's onreadystatechange handler from a top-level event (since the former runs with depth 2 - 1 = 1, and the latter runs with just 1). 2. The nsIThreadObserver mechanism that is used by a lot of code to run "after" the current event is a poor fit for anything that runs script. First, the order the observers fire in is the order they were added, not anything fixed by spec. Additionally, running script can cause the event loop to spin, which is a big source of pain here (bholley has some nasty bug caused by this). 3. We run Promises from different points in the code for workers and main thread. The latter runs from XPConnect's nsIThreadObserver callbacks, while the former runs from a hardcoded call to run Promises in the worker event loop. What workers do is particularly problematic because it means we can't get the right recursion depth no matter what we do to nsThread. The solve this, this patch does the following: 1. Consolidate some handling of microtasks and all handling of stable state from appshell and WorkerPrivate into CycleCollectedJSRuntime. 2. Make the recursionDepth counter only available to CycleCollectedJSRuntime (and its consumers) and remove it from the nsIThreadInternal and nsIThreadObserver APIs. 3. Adjust the recursionDepth counter so that microtasks run with the recursionDepth of the task they are associated with. 4. Introduce the concept of metastable state to replace appshell's RunBeforeNextEvent. Metastable state is reached after every microtask or task is completed. This provides the semantics that bent and I want for IndexedDB, where transactions autocommit at the end of a microtask and do not "spill" from one microtask into a subsequent microtask. This differs from appshell's RunBeforeNextEvent in two ways: a) It fires between microtasks, which was the motivation for starting this. b) It no longer ensures that we're at the same event loop depth in the native event queue. bent decided we don't care about this. 5. Reorder stable state to happen after microtasks such as Promises, per HTML. Right now we call the regular thread observers, including appshell, before the main thread observer (XPConnect), so stable state tasks happen before microtasks.
f78347b683ee4be6d28adb49b94ea1335750e87d: Bug 1190495 - Hoist TaskQueue into xpcom. r=froydnj
Bobby Holley <bobbyholley@gmail.com> - Fri, 07 Aug 2015 17:03:47 -0700 - rev 257245
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1190495 - Hoist TaskQueue into xpcom. r=froydnj
030f756097026386cce95b67b6110f51985b2ef1: Bug 1190495 - Switch TaskQueue entirely to nsRefPtr. r=cpearce
Bobby Holley <bobbyholley@gmail.com> - Fri, 07 Aug 2015 17:01:55 -0700 - rev 257244
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1190495 - Switch TaskQueue entirely to nsRefPtr. r=cpearce
9f162f396c26b23cf09b88ccdbc2a21bc5ae0200: Bug 1190495 - Remove TaskQueue::SyncDispatch. r=cpearce
Bobby Holley <bobbyholley@gmail.com> - Fri, 07 Aug 2015 16:59:54 -0700 - rev 257243
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1190495 - Remove TaskQueue::SyncDispatch. r=cpearce Looks like the last consumers of this went away. \o/
446acb1f222c8c3172d5656c43b877bcc750b750: Bug 1190495 - Separate FlushableTaskQueue into its own file. r=cpearce
Bobby Holley <bobbyholley@gmail.com> - Fri, 07 Aug 2015 16:58:43 -0700 - rev 257242
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1190495 - Separate FlushableTaskQueue into its own file. r=cpearce This thing is garbage, and I don't want to hoist it into XPCOM.
390bedd839297e15113624aea5e9516250d92a4b: Bug 1190495 - Fix include. r=me
Bobby Holley <bobbyholley@gmail.com> - Fri, 07 Aug 2015 17:36:52 -0700 - rev 257241
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1190495 - Fix include. r=me Not sure how this slipped by before.
d4e36668617e18e79b4048de12a88587e2be9435: Bug 1179255 - (fatal) Assertion failure: aRequest == mCancelable, at netwerk/protocol/websocket/WebSocketChannel.cpp:2766, r=mcmanus
Michal Novotny <michal.novotny@gmail.com> - Tue, 11 Aug 2015 14:50:08 +0200 - rev 257240
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1179255 - (fatal) Assertion failure: aRequest == mCancelable, at netwerk/protocol/websocket/WebSocketChannel.cpp:2766, r=mcmanus
7c2524bbcb40fd2a65cff877c20e9a42b0b9f764: Bug 1192277 follow-up - delete the macro invocation from js/src/configure.in; r=me
Nathan Froyd <froydnj@mozilla.com> - Tue, 11 Aug 2015 16:22:33 -0400 - rev 257239
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1192277 follow-up - delete the macro invocation from js/src/configure.in; r=me
497b3313e006ec26eccc0ef1bf67f0462d89ff8a: Bug 1193005 - delete unused function nsRefPtr::begin_assignment; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Tue, 11 Aug 2015 00:23:33 -0400 - rev 257238
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1193005 - delete unused function nsRefPtr::begin_assignment; r=erahm We don't use it; the getter_AddRefs overload for nsRefPtr uses StartAssignment, which is the same thing.
683f439ef93cf06b79a7e6442143d561b7a696cc: Bug 1192277 - remove check for LLVM PR 8927; r=glandium
Nathan Froyd <froydnj@mozilla.com> - Fri, 07 Aug 2015 19:35:43 -0400 - rev 257237
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1192277 - remove check for LLVM PR 8927; r=glandium The PR was fixed in early 2011. clang 3.3, the oldest version of clang that we build with, was released in mid-2013. It's safe to say that all versions of clang now have this fix, and we can delete the check.
1955a6afd3538649bf42662e244480d68100d705: Bug 1192233 - delete check for sys/cdefs.h from configure; r=mshal
Nathan Froyd <froydnj@mozilla.com> - Fri, 07 Aug 2015 17:38:14 -0400 - rev 257236
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1192233 - delete check for sys/cdefs.h from configure; r=mshal We never consult HAVE_SYS_CDEFS_H anywhere. All the places we do include <sys/cdefs.h> do it unconditionally. So let's get rid of this useless check.
1a728b32c041cf5d7cf0dcaa39499b4348a394dd: Bug 1192245 - Fix tests that fail with incremental zeal r=terrence
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 11 Aug 2015 12:41:14 +0100 - rev 257235
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1192245 - Fix tests that fail with incremental zeal r=terrence
a7d7b4d91def8d06e87b546d73a75011c2fc9b4c: Bug 1191482 - Improve WeakCache sweep code and add assertions r=terence
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 11 Aug 2015 12:41:14 +0100 - rev 257234
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1191482 - Improve WeakCache sweep code and add assertions r=terence
633fb9a5405e4b39e0aa022e7d41a5dd1163418f: Bug 1192694: remove mDecodingFrozenAtStateDecoding so decoder can leave dormant normally. r=jwwang
Alfredo Yang <ayang@mozilla.com> - Tue, 11 Aug 2015 21:23:03 +1000 - rev 257233
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1192694: remove mDecodingFrozenAtStateDecoding so decoder can leave dormant normally. r=jwwang
ecebeba2582c0261448239671d07b9a1cab78ff4: Bug 1008146 - Split up test_runtest.html; rs=Ms2ger
Aryeh Gregor <ayg@aryeh.name> - Tue, 11 Aug 2015 14:02:33 +0300 - rev 257232
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1008146 - Split up test_runtest.html; rs=Ms2ger
4aca9861fa7b4b73b9f2d8734d218ec61c5fb200: Bug 1191177 - Kill staticLevel. (r=efaust)
Shu-yu Guo <shu@rfrn.org> - Tue, 11 Aug 2015 03:01:25 -0700 - rev 257231
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1191177 - Kill staticLevel. (r=efaust)
cdfe078bc9d6cf4790acdaa115a62f403be6ebdd: Bug 1191177 - Kill UpvarCookie. (r=efaust)
Shu-yu Guo <shu@rfrn.org> - Tue, 11 Aug 2015 03:01:24 -0700 - rev 257230
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1191177 - Kill UpvarCookie. (r=efaust)
c7891ce06d734f6706b8206be0a491057327292b: Bug 1179063 - Simplify enclosingStaticScope and rename to innermostStaticScope in BCE. (r=efaust)
Shu-yu Guo <shu@rfrn.org> - Tue, 11 Aug 2015 03:01:24 -0700 - rev 257229
Push 29210 by kwierso@gmail.com at Tue, 11 Aug 2015 22:35:38 +0000
Bug 1179063 - Simplify enclosingStaticScope and rename to innermostStaticScope in BCE. (r=efaust)
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip