Bug 1318089: Follow-up: Re-add a lazy getter trigger that looked like a no-op. r?rhelmer draft
authorKris Maglione <maglione.k@gmail.com>
Thu, 17 Nov 2016 13:39:56 -0800
changeset 440594 7a2ef5f4d6490c4a04631156946196425cd2d633
parent 439934 f5521b22fd95004b01f6151ab8c5145e848d0a20
child 537415 f73e5f6bdbabc720488925a232d2cb31d7c6af15
push id36271
push usermaglione.k@gmail.com
push dateThu, 17 Nov 2016 21:40:49 +0000
reviewersrhelmer
bugs1318089
milestone53.0a1
Bug 1318089: Follow-up: Re-add a lazy getter trigger that looked like a no-op. r?rhelmer MozReview-Commit-ID: 65HmvAzTNal
toolkit/mozapps/extensions/test/xpcshell/test_provider_shutdown.js
--- a/toolkit/mozapps/extensions/test/xpcshell/test_provider_shutdown.js
+++ b/toolkit/mozapps/extensions/test/xpcshell/test_provider_shutdown.js
@@ -53,16 +53,18 @@ function findInStatus(aStatus, aName) {
 
 /*
  * Make sure we report correctly when an add-on provider or AddonRepository block shutdown
  */
 add_task(function* blockRepoShutdown() {
   // Reach into the AddonManager scope and inject our mock AddonRepository
   // the mock provider behaves enough like AddonRepository for the purpose of this test
   let mockRepo = mockAddonProvider("Mock repo");
+  // Trigger the lazy getter so that we can assign a new value to it:
+  void AMscope.AddonRepository;
   AMscope.AddonRepository = mockRepo;
 
   let mockProvider = mockAddonProvider("Mock provider");
 
   startupManager();
   AddonManagerPrivate.registerProvider(mockProvider);
 
   // Start shutting the manager down