908934a90c5341bdfd16ad97f07477ab41f5dcd8: Bug 1472491: Part 5a - Add BrowserTabChild actor. r=felipe draft
Kris Maglione <maglione.k@gmail.com> - Sun, 29 Jul 2018 19:42:46 -0700 - rev 828436
Push 118680 by maglione.k@gmail.com at Fri, 10 Aug 2018 23:04:22 +0000
Bug 1472491: Part 5a - Add BrowserTabChild actor. r=felipe MozReview-Commit-ID: 38Y1xwkgxCx
ee2e4e295225beb11a843e49475e2191c0d7e676: Bug 1472491: Part 4b - Add lazy actor support to browser_all_files_referenced. r=florian draft
Kris Maglione <maglione.k@gmail.com> - Tue, 31 Jul 2018 14:56:02 -0700 - rev 828435
Push 118680 by maglione.k@gmail.com at Fri, 10 Aug 2018 23:04:22 +0000
Bug 1472491: Part 4b - Add lazy actor support to browser_all_files_referenced. r=florian ActorManagerParent.jsm handles loading JS IPC actors from JSMs using a base URL and some existing information. The browser_all_files_referenced test doesn't understand these fragmentary URLs, so this patch updates it to get the information directly from the actor registry. MozReview-Commit-ID: 6nRn3ZoXUsR
8baff73eb5a2b56a5eed7ad29b7d27ea7bbfa3b1: Bug 1472491: Part 4a - Add helper classes for lazily loading JS IPC actors. r?felipe draft
Kris Maglione <maglione.k@gmail.com> - Sun, 29 Jul 2018 19:51:13 -0700 - rev 828434
Push 118680 by maglione.k@gmail.com at Fri, 10 Aug 2018 23:04:22 +0000
Bug 1472491: Part 4a - Add helper classes for lazily loading JS IPC actors. r?felipe This adds the basic framework for defining IPC actors which are lazily instantiated for the appropriate frame loaders based on DOM events, message manager messages, and observers. Actual actors are defined in follow-up commits. MozReview-Commit-ID: Jb6CWWW7v3v
bc17120defb1e66071339f11a7e9fff3d47a95da: Bug 1480244: Part 5 - Run most framescripts in shared scope. r=mconley draft
Kris Maglione <maglione.k@gmail.com> - Fri, 10 Aug 2018 13:58:18 -0700 - rev 828433
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 5 - Run most framescripts in shared scope. r=mconley Creating non-shared scopes for frame scripts is fairly expensive. After these changes it's even more expensive. However, many frame scripts have no use for the shared scopes at all. Run-once scripts which execute in closures, for instance, make no use of them. And after bug 1472491, neither do most of our default frame scripts. MozReview-Commit-ID: 9PK7bYdQ0yh
be076323bc70bf5d80261efe1504b4ae54497f98: Bug 1480244: Part 4 - Make child message managers non-global objects. r=bz draft
Kris Maglione <maglione.k@gmail.com> - Fri, 10 Aug 2018 14:03:18 -0700 - rev 828432
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 4 - Make child message managers non-global objects. r=bz This is mostly self-explanatory. However, the patch also contains some minor changes to frame scripts which expect to be able to call message manager methods with a null target object, which stops working when they stop being global objects. MozReview-Commit-ID: HDT2RvK3F3L
93aa6a890d1a30dd044ad3d3c1acf69c82fcd9ce: Bug 1480244: Part 3c - Fix GC test with bad assumptions. r=aswan draft
Kris Maglione <maglione.k@gmail.com> - Fri, 03 Aug 2018 14:11:38 -0700 - rev 828431
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 3c - Fix GC test with bad assumptions. r=aswan This failure starts showing up after these changes by chance. Some aspect of the environment or other causes a map() call to be baseline-optimized, which causes its IC stubs to hold its function environemnt alive, which causes it to hold the context we're checking alive. Forcing a shrinking GC makes the IC stub go away sooner, and therefore allows the test to pass. MozReview-Commit-ID: LXp5mgMZeB1
2e65b3701fe59a10270663fd612b95cd6473522b: Bug 1480244: Part 3b - Fix tests which rely on bad scoping assumptions for frame scripts. r=aswan draft
Kris Maglione <maglione.k@gmail.com> - Fri, 03 Aug 2018 10:59:12 -0700 - rev 828430
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 3b - Fix tests which rely on bad scoping assumptions for frame scripts. r=aswan With the new loading model for frame scripts, lexical variables defined in a global frame script are not available to other frame scripts. Additionally, scripts loaded into a context object by the subscript loader should not depend on being able to access properties of the message manager as if they were globals. MozReview-Commit-ID: 6QEyA1sBVOV
daf637730cfe4f8210a9f3c3a9fcc2322decb842: Bug 1480244: Part 3a - Fix non-strict-mode test code which expects `this` to be bound to its global. r=aswan draft
Kris Maglione <maglione.k@gmail.com> - Fri, 03 Aug 2018 14:33:38 -0700 - rev 828429
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 3a - Fix non-strict-mode test code which expects `this` to be bound to its global. r=aswan A lot of the ad-hoc frame scripts we execute for tests does not run in strict mode, and therefore has its functions' `this` objects set to the global when they are called without a target object. At the moment, this gives them a MessageManager global. Once message managers become non-global objects, however, it will give them the shared JSM global, which is not what they expect. This patches changes scripts which rely on this to explicitly capture or set the appropriate `this` object for their calls. MozReview-Commit-ID: DY8DDb0xE1K
3c0bfe59abd1a357297cf8be3e7933cd372494d0: Bug 1480244: Part 2 - Replace ExecuteInGlobalAndReturnScope with ExecuteInScopeChainAndReturnNewScope. r?tcampbell draft
Kris Maglione <maglione.k@gmail.com> - Fri, 10 Aug 2018 13:54:23 -0700 - rev 828428
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 2 - Replace ExecuteInGlobalAndReturnScope with ExecuteInScopeChainAndReturnNewScope. r?tcampbell This patch series replaces message manager globals with ordinary JS objects which live in the shared JSM global. Once that happens, ExecuteInGlobalAndReturnScope will no longer have useful behavior for them, since the base global has none of the methods that they rely on, and it provides no way to insert another plain object into the scope chain. This patch changes the scope chain for frame scripts to instead look like: -+- Shared JSM global | +- LexicalEnvironment[this=global] | +- NonSyntacticVariablesObject | +- WithEnvironmentObject[target=messageManager] | +- LexicalEnvironment[this=messageManager] Where lexical assignments end up on the lexical scope, and both qualified and unqualified assignments wind up on the NSVO. This has some slight behavioral differences from the previous model, in that properties defined on the message manager can mask properties on the NSVO. But those differences are minor, and probably not worth worrying about, since frame scripts are being deprecated as part of the Fission project. MozReview-Commit-ID: ACEOY2hExco
e93e24a62aca9596f41e6d16045771177f38611c: Bug 1480244: Part 1c - Rename TabChildGlobal to TabChildMessageManager. r=bz draft
Kris Maglione <maglione.k@gmail.com> - Fri, 10 Aug 2018 14:04:23 -0700 - rev 828427
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 1c - Rename TabChildGlobal to TabChildMessageManager. r=bz MozReview-Commit-ID: ECtFjFV7dZa
02a79251d27d575cf034d117ff046d1873e667e2: Bug 1480244: Part 1b - Rename nsInProcessTabChildGlobal to InProcessTabChildMessageManager. r?bz draft
Kris Maglione <maglione.k@gmail.com> - Fri, 10 Aug 2018 14:08:07 -0700 - rev 828426
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 1b - Rename nsInProcessTabChildGlobal to InProcessTabChildMessageManager. r?bz MozReview-Commit-ID: GgleVt1tXTG
7fc0798d582c20ac12bc6fc70f8adf0293c18e4b: Bug 1480244: Part 1a - Rename ProcessGlobal to ContentProcessMessageManager. r=bz draft
Kris Maglione <maglione.k@gmail.com> - Thu, 02 Aug 2018 15:03:50 -0700 - rev 828425
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480244: Part 1a - Rename ProcessGlobal to ContentProcessMessageManager. r=bz After these patches, these objects will no longer be globals, which would make their current names misleading. Parts 1a-1c give more appropriate names to the bindings which will cease to be globals. MozReview-Commit-ID: L8GolQaHnO5
b4cc655e0ae8660046eca4200381c108b2e7923d: Bug 1480319: Split favicon loading code out of ContentLinkHandler.jsm. r?Mossop draft
Kris Maglione <maglione.k@gmail.com> - Wed, 01 Aug 2018 21:36:12 -0700 - rev 828424
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1480319: Split favicon loading code out of ContentLinkHandler.jsm. r?Mossop MozReview-Commit-ID: KyP42P5FJRq
afae556a7de72199dd9fbd4ca42292efd05cf229: Zoom. draft
Kris Maglione <maglione.k@gmail.com> - Tue, 31 Jul 2018 19:41:09 -0700 - rev 828423
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Zoom.
4cfc84c31a90db858791a4ebdd9edefd7fcbcbe2: rm-tab-content-Utils-import draft
Kris Maglione <maglione.k@gmail.com> - Thu, 02 Aug 2018 21:58:36 -0700 - rev 828422
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
rm-tab-content-Utils-import
dd1bf415ee6ccff41e7dcea90c7bee8c180ed61e: amend-WebProgressChild draft
Kris Maglione <maglione.k@gmail.com> - Wed, 01 Aug 2018 22:47:33 -0700 - rev 828421
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
amend-WebProgressChild
c2758ad962bd246bef6f58f4fed5b4095ca03ce2: Bug 1472491: Part 5μ - Add WebNavigationChild actor. r?felipe draft
Kris Maglione <maglione.k@gmail.com> - Mon, 30 Jul 2018 12:55:22 -0700 - rev 828420
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1472491: Part 5μ - Add WebNavigationChild actor. r?felipe MozReview-Commit-ID: 3AiCuNfMenb
7616c331f41edb50a1ef3db1056b84b41363d206: Bug 1472491: Part 5λ - Add Split RemoteFinder into FinderChild and FinderParent actors. r?gijs draft
Kris Maglione <maglione.k@gmail.com> - Mon, 30 Jul 2018 12:25:58 -0700 - rev 828419
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1472491: Part 5λ - Add Split RemoteFinder into FinderChild and FinderParent actors. r?gijs MozReview-Commit-ID: JAv8lh2gJoB
47c0892d67727c43975f347c53d3d989173aad4a: Bug 1472491: Part 5κ - Add ManifestMessagesChild actor. r?felipe draft
Kris Maglione <maglione.k@gmail.com> - Mon, 30 Jul 2018 11:39:52 -0700 - rev 828418
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1472491: Part 5κ - Add ManifestMessagesChild actor. r?felipe MozReview-Commit-ID: 20toouW9YzT
64302c5f0d2571abf1f5d8d0d12d08f9bcc775db: Bug 1472491: Part 5ι - Add LoginManagerChild actor. r?felipe draft
Kris Maglione <maglione.k@gmail.com> - Mon, 30 Jul 2018 11:36:26 -0700 - rev 828417
Push 118679 by maglione.k@gmail.com at Fri, 10 Aug 2018 21:19:41 +0000
Bug 1472491: Part 5ι - Add LoginManagerChild actor. r?felipe MozReview-Commit-ID: Baga8mM9DxD
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 tip