Bug 1092627 - Followup fix for bug 1091077. _testLogger has to be renamed to _mailnewsTestLogger. r=mkmelin, a=mkmelin
authorHiroyuki Ikezoe <hiikezoe@mozilla-japan.org>
Sat, 01 Nov 2014 17:03:00 +0200
changeset 16988 1cbb106a7954ec30c8215c67f19351515c6009f5
parent 16987 34173b81407fda2cb069ca4cae59f7c40b0499e9
child 16989 ed705dfbfef160cb05f86475fcdacde67525c694
push id10550
push usermkmelin@iki.fi
push dateSun, 02 Nov 2014 17:02:51 +0000
treeherdercomm-central@1cbb106a7954 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmkmelin, mkmelin
bugs1092627, 1091077
Bug 1092627 - Followup fix for bug 1091077. _testLogger has to be renamed to _mailnewsTestLogger. r=mkmelin, a=mkmelin
mail/test/mozmill/shared-modules/test-folder-display-helpers.js
--- a/mail/test/mozmill/shared-modules/test-folder-display-helpers.js
+++ b/mail/test/mozmill/shared-modules/test-folder-display-helpers.js
@@ -118,19 +118,19 @@ function setupModule() {
 
   // The xpcshell test resources assume they are loaded into a single global
   //  namespace, so we need to help them out to maintain their delusion.
   load_via_src_path('logHelper.js', testHelperModule);
   mark_action = testHelperModule.mark_action;
   mark_failure = testHelperModule.mark_failure;
 
   // Remove the dump appender that got appended; it just adds noise.
-  testHelperModule._testLogger.removeAppender(
-    testHelperModule._testLogger.ownAppenders[
-      testHelperModule._testLogger.ownAppenders.length - 1]);
+  testHelperModule._mailnewsTestLogger.removeAppender(
+    testHelperModule._mailnewsTestLogger.ownAppenders[
+      testHelperModule._mailnewsTestLogger.ownAppenders.length - 1]);
 
   // Add a bucketing appender to the root.
   let rootLogger = Log4Moz.repository.rootLogger;
   let bucketAppender = new Log4Moz.TimeAwareMemoryBucketAppender();
   bucketAppender.level = Log4Moz.Level.All;
   rootLogger.addAppender(bucketAppender);
 
   // Indicate to any fancy helpers (just folderEventLogHelper right now) that