Bug 1314350 - Port TestTXMgr to gtest. This removes the framework for testing object destruction order because that was already non-deterministic due to the cycle collector, r?masayuki draft
authorBenjamin Smedberg <benjamin@smedbergs.us>
Tue, 15 Nov 2016 13:06:05 -0500
changeset 439289 7d5e89ce1ac0c63f58b4abafff4c9d4a49b08a63
parent 439288 4691d53e529cdaa93684c24ddbdfc559c253f0cc
child 439290 c94626b185f430e93f0ee2f163f71dff3f975166
push id35959
push userbsmedberg@mozilla.com
push dateTue, 15 Nov 2016 19:26:33 +0000
reviewersmasayuki
bugs1314350
milestone53.0a1
Bug 1314350 - Port TestTXMgr to gtest. This removes the framework for testing object destruction order because that was already non-deterministic due to the cycle collector, r?masayuki MozReview-Commit-ID: vK4iYmytgt
editor/txmgr/tests/TestTXMgr.cpp
editor/txmgr/tests/moz.build
testing/cppunittest.ini
--- a/editor/txmgr/tests/TestTXMgr.cpp
+++ b/editor/txmgr/tests/TestTXMgr.cpp
@@ -1,42 +1,27 @@
 /* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-#include "TestHarness.h"
+#include "gtest/gtest.h"
 
 #include "nsITransactionManager.h"
 #include "nsComponentManagerUtils.h"
 #include "mozilla/Likely.h"
 
 static int32_t sConstructorCount     = 0;
-static int32_t sDestructorCount      = 0;
-static int32_t *sDestructorOrderArr  = 0;
 static int32_t sDoCount              = 0;
 static int32_t *sDoOrderArr          = 0;
 static int32_t sUndoCount            = 0;
 static int32_t *sUndoOrderArr        = 0;
 static int32_t sRedoCount            = 0;
 static int32_t *sRedoOrderArr        = 0;
 
-// #define ENABLE_DEBUG_PRINTFS 1
-
-int32_t sSimpleTestDestructorOrderArr[] = {
-          2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,  15,
-         16,  17,  18,  19,  20,  21,   1,  22,  23,  24,  25,  26,  27,  28,
-         29,  30,  31,  42,  43,  44,  45,  46,  47,  48,  49,  50,  51,  52,
-         53,  54,  55,  56,  57,  58,  59,  60,  61,  41,  40,  62,  39,  38,
-         37,  36,  35,  34,  33,  32,  68,  63,  64,  65,  66,  67,  69,  71,
-         70,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
-         85,  86,  87,  88,  89,  90,  91,  92,  93,  94,  95,  96,  97,  98,
-         99, 100, 101, 111, 110, 109, 108, 107, 106, 105, 104, 103, 102, 131,
-        130, 129, 128, 127, 126, 125, 124, 123, 122, 121, 120, 119, 118, 117,
-        116, 115, 114, 113, 112 };
 
 int32_t sSimpleTestDoOrderArr[] = {
           1,   2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,
          15,  16,  17,  18,  19,  20,  21,  22,  23,  24,  25,  26,  27,  28,
          29,  30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  40,  41,  42,
          43,  44,  45,  46,  47,  48,  49,  50,  51,  52,  53,  54,  55,  56,
          57,  58,  59,  60,  61,  62,  63,  64,  65,  66,  67,  68,  69,  70,
          71,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
@@ -48,84 +33,16 @@ int32_t sSimpleTestDoOrderArr[] = {
 int32_t sSimpleTestUndoOrderArr[] = {
          41,  40,  39,  38,  62,  39,  38,  37,  69,  71,  70, 111, 110, 109,
         108, 107, 106, 105, 104, 103, 102, 131, 130, 129, 128, 127, 126, 125,
         124, 123, 122 };
 
 static int32_t sSimpleTestRedoOrderArr[] = {
          38,  39,  70 };
 
-int32_t sAggregateTestDestructorOrderArr[] = {
-         14,  13,  12,  11,  10,   9,   8,  21,  20,  19,  18,  17,  16,  15,
-         28,  27,  26,  25,  24,  23,  22,  35,  34,  33,  32,  31,  30,  29,
-         42,  41,  40,  39,  38,  37,  36,  49,  48,  47,  46,  45,  44,  43,
-         56,  55,  54,  53,  52,  51,  50,  63,  62,  61,  60,  59,  58,  57,
-         70,  69,  68,  67,  66,  65,  64,  77,  76,  75,  74,  73,  72,  71,
-         84,  83,  82,  81,  80,  79,  78,  91,  90,  89,  88,  87,  86,  85,
-         98,  97,  96,  95,  94,  93,  92, 105, 104, 103, 102, 101, 100,  99,
-        112, 111, 110, 109, 108, 107, 106, 119, 118, 117, 116, 115, 114, 113,
-        126, 125, 124, 123, 122, 121, 120, 133, 132, 131, 130, 129, 128, 127,
-        140, 139, 138, 137, 136, 135, 134, 147, 146, 145, 144, 143, 142, 141,
-          7,   6,   5,   4,   3,   2,   1, 154, 153, 152, 151, 150, 149, 148,
-        161, 160, 159, 158, 157, 156, 155, 168, 167, 166, 165, 164, 163, 162,
-        175, 174, 173, 172, 171, 170, 169, 182, 181, 180, 179, 178, 177, 176,
-        189, 188, 187, 186, 185, 184, 183, 196, 195, 194, 193, 192, 191, 190,
-        203, 202, 201, 200, 199, 198, 197, 210, 209, 208, 207, 206, 205, 204,
-        217, 216, 215, 214, 213, 212, 211, 294, 293, 292, 291, 290, 289, 288,
-        301, 300, 299, 298, 297, 296, 295, 308, 307, 306, 305, 304, 303, 302,
-        315, 314, 313, 312, 311, 310, 309, 322, 321, 320, 319, 318, 317, 316,
-        329, 328, 327, 326, 325, 324, 323, 336, 335, 334, 333, 332, 331, 330,
-        343, 342, 341, 340, 339, 338, 337, 350, 349, 348, 347, 346, 345, 344,
-        357, 356, 355, 354, 353, 352, 351, 364, 363, 362, 361, 360, 359, 358,
-        371, 370, 369, 368, 367, 366, 365, 378, 377, 376, 375, 374, 373, 372,
-        385, 384, 383, 382, 381, 380, 379, 392, 391, 390, 389, 388, 387, 386,
-        399, 398, 397, 396, 395, 394, 393, 406, 405, 404, 403, 402, 401, 400,
-        413, 412, 411, 410, 409, 408, 407, 420, 419, 418, 417, 416, 415, 414,
-        427, 426, 425, 424, 423, 422, 421, 287, 286, 285, 284, 283, 282, 281,
-        280, 279, 278, 277, 276, 275, 274, 434, 433, 432, 431, 430, 429, 428,
-        273, 272, 271, 270, 269, 268, 267, 266, 265, 264, 263, 262, 261, 260,
-        259, 258, 257, 256, 255, 254, 253, 252, 251, 250, 249, 248, 247, 246,
-        245, 244, 243, 242, 241, 240, 239, 238, 237, 236, 235, 234, 233, 232,
-        231, 230, 229, 228, 227, 226, 225, 224, 223, 222, 221, 220, 219, 218,
-        472, 471, 470, 441, 440, 439, 438, 437, 436, 435, 448, 447, 446, 445,
-        444, 443, 442, 455, 454, 453, 452, 451, 450, 449, 462, 461, 460, 459,
-        458, 457, 456, 469, 468, 467, 466, 465, 464, 463, 479, 478, 477, 476,
-        475, 474, 473, 493, 492, 491, 490, 489, 488, 487, 486, 485, 484, 483,
-        482, 481, 480, 496, 497, 495, 499, 500, 498, 494, 503, 504, 502, 506,
-        507, 505, 501, 510, 511, 509, 513, 514, 512, 508, 517, 518, 516, 520,
-        521, 519, 515, 524, 525, 523, 527, 528, 526, 522, 531, 532, 530, 534,
-        535, 533, 529, 538, 539, 537, 541, 542, 540, 536, 545, 546, 544, 548,
-        549, 547, 543, 552, 553, 551, 555, 556, 554, 550, 559, 560, 558, 562,
-        563, 561, 557, 566, 567, 565, 569, 570, 568, 564, 573, 574, 572, 576,
-        577, 575, 571, 580, 581, 579, 583, 584, 582, 578, 587, 588, 586, 590,
-        591, 589, 585, 594, 595, 593, 597, 598, 596, 592, 601, 602, 600, 604,
-        605, 603, 599, 608, 609, 607, 611, 612, 610, 606, 615, 616, 614, 618,
-        619, 617, 613, 622, 623, 621, 625, 626, 624, 620, 629, 630, 628, 632,
-        633, 631, 627, 640, 639, 638, 637, 636, 635, 634, 647, 646, 645, 644,
-        643, 642, 641, 654, 653, 652, 651, 650, 649, 648, 661, 660, 659, 658,
-        657, 656, 655, 668, 667, 666, 665, 664, 663, 662, 675, 674, 673, 672,
-        671, 670, 669, 682, 681, 680, 679, 678, 677, 676, 689, 688, 687, 686,
-        685, 684, 683, 696, 695, 694, 693, 692, 691, 690, 703, 702, 701, 700,
-        699, 698, 697, 773, 772, 771, 770, 769, 768, 767, 766, 765, 764, 763,
-        762, 761, 760, 759, 758, 757, 756, 755, 754, 753, 752, 751, 750, 749,
-        748, 747, 746, 745, 744, 743, 742, 741, 740, 739, 738, 737, 736, 735,
-        734, 733, 732, 731, 730, 729, 728, 727, 726, 725, 724, 723, 722, 721,
-        720, 719, 718, 717, 716, 715, 714, 713, 712, 711, 710, 709, 708, 707,
-        706, 705, 704, 913, 912, 911, 910, 909, 908, 907, 906, 905, 904, 903,
-        902, 901, 900, 899, 898, 897, 896, 895, 894, 893, 892, 891, 890, 889,
-        888, 887, 886, 885, 884, 883, 882, 881, 880, 879, 878, 877, 876, 875,
-        874, 873, 872, 871, 870, 869, 868, 867, 866, 865, 864, 863, 862, 861,
-        860, 859, 858, 857, 856, 855, 854, 853, 852, 851, 850, 849, 848, 847,
-        846, 845, 844, 843, 842, 841, 840, 839, 838, 837, 836, 835, 834, 833,
-        832, 831, 830, 829, 828, 827, 826, 825, 824, 823, 822, 821, 820, 819,
-        818, 817, 816, 815, 814, 813, 812, 811, 810, 809, 808, 807, 806, 805,
-        804, 803, 802, 801, 800, 799, 798, 797, 796, 795, 794, 793, 792, 791,
-        790, 789, 788, 787, 786, 785, 784, 783, 782, 781, 780, 779, 778, 777,
-        776, 775, 774 };
-
 int32_t sAggregateTestDoOrderArr[] = {
           1,   2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,
          15,  16,  17,  18,  19,  20,  21,  22,  23,  24,  25,  26,  27,  28,
          29,  30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  40,  41,  42,
          43,  44,  45,  46,  47,  48,  49,  50,  51,  52,  53,  54,  55,  56,
          57,  58,  59,  60,  61,  62,  63,  64,  65,  66,  67,  68,  69,  70,
          71,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
          85,  86,  87,  88,  89,  90,  91,  92,  93,  94,  95,  96,  97,  98,
@@ -206,26 +123,16 @@ int32_t sAggregateTestUndoOrderArr[] = {
         882, 881, 880, 879, 878, 877, 876, 875, 874, 873, 872, 871, 870, 869,
         868, 867, 866, 865, 864, 863, 862, 861, 860, 859, 858, 857, 856, 855,
         854, 853, 852, 851, 850, 849, 848, 847, 846, 845, 844 };
 
 int32_t sAggregateTestRedoOrderArr[] = {
         260, 261, 262, 263, 264, 265, 266, 267, 268, 269, 270, 271, 272, 273,
         476, 477, 478, 479, 480, 481, 482, 483, 484, 485, 486 };
 
-int32_t sSimpleBatchTestDestructorOrderArr[] = {
-         21,  22,  23,  24,  25,  26,  27,  28,  29,  30,  31,  32,  33,  34,
-         35,  36,  37,  38,  39,  40,  43,  42,  41,  64,  63,  62,  61,  60,
-         59,  58,  57,  56,  55,  54,  53,  52,  51,  50,  49,  48,  47,  46,
-         45,  44,  20,  19,  18,  17,  16,  15,  14,  13,  12,  11,  10,   9,
-          8,   7,   6,   5,   4,   3,   2,   1,  65,  67,  66,  68,  69,  70,
-         71,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
-         85,  86,  87, 107, 106, 105, 104, 103, 102, 101, 100,  99,  98,  97,
-         96,  95,  94,  93,  92,  91,  90,  89,  88 };
-
 int32_t sSimpleBatchTestDoOrderArr[] = {
           1,   2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,
          15,  16,  17,  18,  19,  20,  21,  22,  23,  24,  25,  26,  27,  28,
          29,  30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  40,  41,  42,
          43,  44,  45,  46,  47,  48,  49,  50,  51,  52,  53,  54,  55,  56,
          57,  58,  59,  60,  61,  62,  63,  64,  65,  66,  67,  68,  69,  70,
          71,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
          85,  86,  87,  88,  89,  90,  91,  92,  93,  94,  95,  96,  97,  98,
@@ -237,72 +144,16 @@ int32_t sSimpleBatchTestUndoOrderArr[] =
          61,  60,  59,  58,  57,  56,  55,  54,  53,  52,  51,  50,  49,  48,
          47,  46,  45,  44,  65,  67,  66, 107, 106, 105, 104, 103, 102, 101,
         100,  99,  98 };
 
 int32_t sSimpleBatchTestRedoOrderArr[] = {
           1,   2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,
          15,  16,  17,  18,  19,  20,  41,  42,  43,  66 };
 
-int32_t sAggregateBatchTestDestructorOrderArr[] = {
-        147, 146, 145, 144, 143, 142, 141, 154, 153, 152, 151, 150, 149, 148,
-        161, 160, 159, 158, 157, 156, 155, 168, 167, 166, 165, 164, 163, 162,
-        175, 174, 173, 172, 171, 170, 169, 182, 181, 180, 179, 178, 177, 176,
-        189, 188, 187, 186, 185, 184, 183, 196, 195, 194, 193, 192, 191, 190,
-        203, 202, 201, 200, 199, 198, 197, 210, 209, 208, 207, 206, 205, 204,
-        217, 216, 215, 214, 213, 212, 211, 224, 223, 222, 221, 220, 219, 218,
-        231, 230, 229, 228, 227, 226, 225, 238, 237, 236, 235, 234, 233, 232,
-        245, 244, 243, 242, 241, 240, 239, 252, 251, 250, 249, 248, 247, 246,
-        259, 258, 257, 256, 255, 254, 253, 266, 265, 264, 263, 262, 261, 260,
-        273, 272, 271, 270, 269, 268, 267, 280, 279, 278, 277, 276, 275, 274,
-        301, 300, 299, 298, 297, 296, 295, 294, 293, 292, 291, 290, 289, 288,
-        287, 286, 285, 284, 283, 282, 281, 444, 443, 442, 441, 440, 439, 438,
-        437, 436, 435, 434, 433, 432, 431, 430, 429, 428, 427, 426, 425, 424,
-        423, 422, 421, 420, 419, 418, 417, 416, 415, 414, 413, 412, 411, 410,
-        409, 408, 407, 406, 405, 404, 403, 402, 401, 400, 399, 398, 397, 396,
-        395, 394, 393, 392, 391, 390, 389, 388, 387, 386, 385, 384, 383, 382,
-        381, 380, 379, 378, 377, 376, 375, 374, 373, 372, 371, 370, 369, 368,
-        367, 366, 365, 364, 363, 362, 361, 360, 359, 358, 357, 356, 355, 354,
-        353, 352, 351, 350, 349, 348, 347, 346, 345, 344, 343, 342, 341, 340,
-        339, 338, 337, 336, 335, 334, 333, 332, 331, 330, 329, 328, 327, 326,
-        325, 324, 323, 322, 321, 320, 319, 318, 317, 316, 315, 314, 313, 312,
-        311, 310, 309, 308, 307, 306, 305, 304, 303, 302, 140, 139, 138, 137,
-        136, 135, 134, 133, 132, 131, 130, 129, 128, 127, 126, 125, 124, 123,
-        122, 121, 120, 119, 118, 117, 116, 115, 114, 113, 112, 111, 110, 109,
-        108, 107, 106, 105, 104, 103, 102, 101, 100,  99,  98,  97,  96,  95,
-         94,  93,  92,  91,  90,  89,  88,  87,  86,  85,  84,  83,  82,  81,
-         80,  79,  78,  77,  76,  75,  74,  73,  72,  71,  70,  69,  68,  67,
-         66,  65,  64,  63,  62,  61,  60,  59,  58,  57,  56,  55,  54,  53,
-         52,  51,  50,  49,  48,  47,  46,  45,  44,  43,  42,  41,  40,  39,
-         38,  37,  36,  35,  34,  33,  32,  31,  30,  29,  28,  27,  26,  25,
-         24,  23,  22,  21,  20,  19,  18,  17,  16,  15,  14,  13,  12,  11,
-         10,   9,   8,   7,   6,   5,   4,   3,   2,   1, 451, 450, 449, 448,
-        447, 446, 445, 465, 464, 463, 462, 461, 460, 459, 458, 457, 456, 455,
-        454, 453, 452, 468, 469, 467, 471, 472, 470, 466, 475, 476, 474, 478,
-        479, 477, 473, 482, 483, 481, 485, 486, 484, 480, 489, 490, 488, 492,
-        493, 491, 487, 496, 497, 495, 499, 500, 498, 494, 503, 504, 502, 506,
-        507, 505, 501, 510, 511, 509, 513, 514, 512, 508, 517, 518, 516, 520,
-        521, 519, 515, 524, 525, 523, 527, 528, 526, 522, 531, 532, 530, 534,
-        535, 533, 529, 538, 539, 537, 541, 542, 540, 536, 545, 546, 544, 548,
-        549, 547, 543, 552, 553, 551, 555, 556, 554, 550, 559, 560, 558, 562,
-        563, 561, 557, 566, 567, 565, 569, 570, 568, 564, 573, 574, 572, 576,
-        577, 575, 571, 580, 581, 579, 583, 584, 582, 578, 587, 588, 586, 590,
-        591, 589, 585, 594, 595, 593, 597, 598, 596, 592, 601, 602, 600, 604,
-        605, 603, 599, 745, 744, 743, 742, 741, 740, 739, 738, 737, 736, 735,
-        734, 733, 732, 731, 730, 729, 728, 727, 726, 725, 724, 723, 722, 721,
-        720, 719, 718, 717, 716, 715, 714, 713, 712, 711, 710, 709, 708, 707,
-        706, 705, 704, 703, 702, 701, 700, 699, 698, 697, 696, 695, 694, 693,
-        692, 691, 690, 689, 688, 687, 686, 685, 684, 683, 682, 681, 680, 679,
-        678, 677, 676, 675, 674, 673, 672, 671, 670, 669, 668, 667, 666, 665,
-        664, 663, 662, 661, 660, 659, 658, 657, 656, 655, 654, 653, 652, 651,
-        650, 649, 648, 647, 646, 645, 644, 643, 642, 641, 640, 639, 638, 637,
-        636, 635, 634, 633, 632, 631, 630, 629, 628, 627, 626, 625, 624, 623,
-        622, 621, 620, 619, 618, 617, 616, 615, 614, 613, 612, 611, 610, 609,
-        608, 607, 606 };
-
 int32_t sAggregateBatchTestDoOrderArr[] = {
           1,   2,   3,   4,   5,   6,   7,   8,   9,  10,  11,  12,  13,  14,
          15,  16,  17,  18,  19,  20,  21,  22,  23,  24,  25,  26,  27,  28,
          29,  30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  40,  41,  42,
          43,  44,  45,  46,  47,  48,  49,  50,  51,  52,  53,  54,  55,  56,
          57,  58,  59,  60,  61,  62,  63,  64,  65,  66,  67,  68,  69,  70,
          71,  72,  73,  74,  75,  76,  77,  78,  79,  80,  81,  82,  83,  84,
          85,  86,  87,  88,  89,  90,  91,  92,  93,  94,  95,  96,  97,  98,
@@ -396,18 +247,16 @@ int32_t sAggregateBatchTestRedoOrderArr[
          85,  86,  87,  88,  89,  90,  91,  92,  93,  94,  95,  96,  97,  98,
          99, 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112,
         113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126,
         127, 128, 129, 130, 131, 132, 133, 134, 135, 136, 137, 138, 139, 140,
         281, 282, 283, 284, 285, 286, 287, 288, 289, 290, 291, 292, 293, 294,
         295, 296, 297, 298, 299, 300, 301, 448, 449, 450, 451, 452, 453, 454,
         455, 456, 457, 458 };
 
-#define TEST_TXMGR_IF_RELEASE(tx) if (tx) tx->Release(); // Release but don't clear pointer!
-
 class TestTransaction : public nsITransaction
 {
 protected:
   virtual ~TestTransaction() {}
 
 public:
 
   TestTransaction() {}
@@ -436,106 +285,66 @@ protected:
 public:
 
   explicit SimpleTransaction(int32_t aFlags=NONE_FLAG)
     : mVal(++sConstructorCount), mFlags(aFlags)
   {}
 
   virtual ~SimpleTransaction()
   {
-    //
-    // Make sure transactions are being destroyed in the order we expect!
-    // Notice that we don't check to see if we go past the end of the array.
-    // This is done on purpose since we want to crash if the order array is out
-    // of date.
-    //
-    /* Disabled because the current cycle collector doesn't delete
-       cycle collectable objects synchronously, nor doesn't guarantee any order.
-    if (sDestructorOrderArr && mVal != sDestructorOrderArr[sDestructorCount]) {
-      fail("~SimpleTransaction expected %d got %d.\n",
-           mVal, sDestructorOrderArr[sDestructorCount]);
-      exit(-1);
-    }
-    */
-
-    ++sDestructorCount;
-
-#ifdef ENABLE_DEBUG_PRINTFS
-    printf("\n~SimpleTransaction: %d - 0x%.8x\n", mVal, (int32_t)this);
-#endif // ENABLE_DEBUG_PRINTFS
-
-    mVal = -1;
   }
 
   NS_IMETHOD DoTransaction()
   {
     //
     // Make sure DoTransaction() is called in the order we expect!
     // Notice that we don't check to see if we go past the end of the array.
     // This is done on purpose since we want to crash if the order array is out
     // of date.
     //
-    if (sDoOrderArr && mVal != sDoOrderArr[sDoCount]) {
-      fail("DoTransaction expected %d got %d.\n",
-           mVal, sDoOrderArr[sDoCount]);
-      exit(-1);
+    if (sDoOrderArr) {
+      EXPECT_EQ(mVal, sDoOrderArr[sDoCount]);
     }
 
     ++sDoCount;
 
-#ifdef ENABLE_DEBUG_PRINTFS
-    printf("\nSimpleTransaction.DoTransaction: %d - 0x%.8x\n", mVal, (int32_t)this);
-#endif // ENABLE_DEBUG_PRINTFS
-
     return (mFlags & THROWS_DO_ERROR_FLAG) ? NS_ERROR_FAILURE : NS_OK;
   }
 
   NS_IMETHOD UndoTransaction()
   {
     //
     // Make sure UndoTransaction() is called in the order we expect!
     // Notice that we don't check to see if we go past the end of the array.
     // This is done on purpose since we want to crash if the order array is out
     // of date.
     //
-    if (sUndoOrderArr && mVal != sUndoOrderArr[sUndoCount]) {
-      fail("UndoTransaction expected %d got %d.\n",
-           mVal, sUndoOrderArr[sUndoCount]);
-      exit(-1);
+    if (sUndoOrderArr) {
+      EXPECT_EQ(mVal, sUndoOrderArr[sUndoCount]);
     }
 
     ++sUndoCount;
 
-#ifdef ENABLE_DEBUG_PRINTFS
-    printf("\nSimpleTransaction.Undo: %d - 0x%.8x\n", mVal, (int32_t)this);
-#endif // ENABLE_DEBUG_PRINTFS
-
     return (mFlags & THROWS_UNDO_ERROR_FLAG) ? NS_ERROR_FAILURE : NS_OK;
   }
 
   NS_IMETHOD RedoTransaction()
   {
     //
     // Make sure RedoTransaction() is called in the order we expect!
     // Notice that we don't check to see if we go past the end of the array.
     // This is done on purpose since we want to crash if the order array is out
     // of date.
     //
-    if (sRedoOrderArr && mVal != sRedoOrderArr[sRedoCount]) {
-      fail("RedoTransaction expected %d got %d.\n",
-           mVal, sRedoOrderArr[sRedoCount]);
-      exit(-1);
+    if (sRedoOrderArr) {
+      EXPECT_EQ(mVal, sRedoOrderArr[sRedoCount]);
     }
 
     ++sRedoCount;
 
-#ifdef ENABLE_DEBUG_PRINTFS
-    printf("\nSimpleTransaction.Redo: %d - 0x%.8x\n", mVal, (int32_t)this);
-#endif // ENABLE_DEBUG_PRINTFS
-
     return (mFlags & THROWS_REDO_ERROR_FLAG) ? NS_ERROR_FAILURE : NS_OK;
   }
 
   NS_IMETHOD GetIsTransient(bool *aIsTransient)
   {
     if (aIsTransient) {
       *aIsTransient = (mFlags & TRANSIENT_FLAG) ? true : false;
     }
@@ -591,17 +400,16 @@ public:
     mErrorFlags         = aFlags & ALL_ERROR_FLAGS;
     mTXMgr              = aTXMgr;
     mMaxLevel           = aMaxLevel;
     mNumChildrenPerNode = aNumChildrenPerNode;
   }
 
   virtual ~AggregateTransaction()
   {
-    // printf("~AggregateTransaction(0x%.8x) - %3d (%3d)\n", this, mLevel, mVal);
   }
 
   NS_IMETHOD DoTransaction()
   {
     if (mLevel >= mMaxLevel) {
       // Only leaf nodes can throw errors!
       mFlags |= mErrorFlags;
     }
@@ -636,54 +444,27 @@ public:
       } else if ((mErrorFlags & THROWS_UNDO_ERROR_FLAG) && i == 1) {
         // Make the leftmost leaf transaction throw the error!
         flags = THROWS_UNDO_ERROR_FLAG;
         mErrorFlags = mErrorFlags & (~THROWS_UNDO_ERROR_FLAG);
       }
 
       flags |= mFlags & BATCH_FLAG;
 
-      AggregateTransaction *tximpl =
+      RefPtr<AggregateTransaction> tximpl =
               new AggregateTransaction(mTXMgr, cLevel, i, mMaxLevel,
                                        mNumChildrenPerNode, flags);
 
-      if (!tximpl) {
-        fail("Failed to allocate AggregateTransaction %d, level %d. (%d)\n",
-             i, mLevel, rv);
-
-        if (mFlags & BATCH_FLAG) {
-          mTXMgr->EndBatch(false);
-        }
-
-        return NS_ERROR_OUT_OF_MEMORY;
-      }
-
-      nsITransaction *tx = 0;
-      rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
+      rv = mTXMgr->DoTransaction(tximpl);
       if (NS_FAILED(rv)) {
-        fail("QueryInterface() failed for transaction %d, level %d. (%d)\n",
-             i, mLevel, rv);
-
         if (mFlags & BATCH_FLAG) {
           mTXMgr->EndBatch(false);
         }
         return rv;
       }
-
-      rv = mTXMgr->DoTransaction(tx);
-      if (NS_FAILED(rv)) {
-        // fail("Failed to execute transaction %d, level %d. (%d)\n",
-        //      i, mLevel, rv);
-        tx->Release();
-        if (mFlags & BATCH_FLAG) {
-          mTXMgr->EndBatch(false);
-        }
-        return rv;
-      }
-      tx->Release();
     }
 
     if (mFlags & BATCH_FLAG) {
       mTXMgr->EndBatch(false);
     }
     return rv;
   }
 };
@@ -729,3695 +510,1564 @@ public:
   }
 };
 
 void
 reset_globals()
 {
   sConstructorCount   = 0;
 
-  sDestructorCount    = 0;
-  sDestructorOrderArr = 0;
-
   sDoCount            = 0;
   sDoOrderArr         = 0;
 
   sUndoCount          = 0;
   sUndoOrderArr       = 0;
 
   sRedoCount          = 0;
   sRedoOrderArr       = 0;
 }
 
 /**
  * Test behaviors in non-batch mode.
  **/
-nsresult
+void
 quick_test(TestTransactionFactory *factory)
 {
   /*******************************************************************
    *
    * Create a transaction manager implementation:
    *
    *******************************************************************/
 
   nsresult rv;
   nsCOMPtr<nsITransactionManager> mgr =
     do_CreateInstance(NS_TRANSACTIONMANAGER_CONTRACTID, &rv);
-  if (NS_FAILED(rv) || !mgr) {
-    fail("Failed to create Transaction Manager instance.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  passed("Create transaction manager instance");
+  ASSERT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call DoTransaction() with a null transaction:
    *
    *******************************************************************/
 
   rv = mgr->DoTransaction(0);
-  if (rv != NS_ERROR_NULL_POINTER) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call DoTransaction() with null transaction");
+  EXPECT_EQ(rv, NS_ERROR_NULL_POINTER);
 
   /*******************************************************************
    *
    * Call UndoTransaction() with an empty undo stack:
    *
    *******************************************************************/
 
   rv = mgr->UndoTransaction();
-  if (NS_FAILED(rv)) {
-    fail("Undo on empty undo stack failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call UndoTransaction() with empty undo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call RedoTransaction() with an empty redo stack:
    *
    *******************************************************************/
 
   rv = mgr->RedoTransaction();
-  if (NS_FAILED(rv)) {
-    fail("Redo on empty redo stack failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call RedoTransaction() with empty redo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call SetMaxTransactionCount(-1) with empty undo and redo stacks:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(-1);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(-1) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call SetMaxTransactionCount(-1) with empty undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call SetMaxTransactionCount(0) with empty undo and redo stacks:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(0);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(0) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call SetMaxTransactionCount(0) with empty undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call SetMaxTransactionCount(10) with empty undo and redo stacks:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(10);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(10) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call SetMaxTransactionCount(10) with empty undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call Clear() with empty undo and redo stacks:
    *
    *******************************************************************/
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear on empty undo and redo stack failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call Clear() with empty undo and redo stack");
-
-  int32_t numitems;
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Call GetNumberOfUndoItems() with an empty undo stack:
    *
    *******************************************************************/
 
+  int32_t numitems;
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Call GetNumberOfUndoItems() with empty undo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Call GetNumberOfRedoItems() with an empty redo stack:
    *
    *******************************************************************/
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Call GetNumberOfRedoItems() with empty redo stack");
-
-  nsITransaction *tx;
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Call PeekUndoStack() with an empty undo stack:
    *
    *******************************************************************/
 
-  tx = 0;
-  rv = mgr->PeekUndoStack(&tx);
-
-  TEST_TXMGR_IF_RELEASE(tx); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("PeekUndoStack() on empty undo stack failed. (%d)\n", rv);
-    return rv;
+  {
+    nsCOMPtr<nsITransaction> tx;
+    rv = mgr->PeekUndoStack(getter_AddRefs(tx));
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(tx, nullptr);
   }
 
-  if (tx) {
-    fail("PeekUndoStack() on empty undo stack failed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Call PeekUndoStack() with empty undo stack");
-
   /*******************************************************************
    *
    * Call PeekRedoStack() with an empty undo stack:
    *
    *******************************************************************/
 
-  tx = 0;
-  rv = mgr->PeekRedoStack(&tx);
-
-  TEST_TXMGR_IF_RELEASE(tx); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("PeekRedoStack() on empty redo stack failed. (%d)\n", rv);
-    return rv;
+  {
+    nsCOMPtr<nsITransaction> tx;
+    rv = mgr->PeekRedoStack(getter_AddRefs(tx));
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(tx, nullptr);
   }
 
-  if (tx) {
-    fail("PeekRedoStack() on empty redo stack failed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Call PeekRedoStack() with empty undo stack");
-
   /*******************************************************************
    *
    * Call AddListener() with a null listener pointer:
    *
    *******************************************************************/
 
-  rv = mgr->AddListener(0);
-  if (rv != NS_ERROR_NULL_POINTER) {
-    fail("AddListener() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call AddListener() with null listener");
+  rv = mgr->AddListener(nullptr);
+  EXPECT_EQ(rv, NS_ERROR_NULL_POINTER);
 
   /*******************************************************************
    *
    * Call RemoveListener() with a null listener pointer:
    *
    *******************************************************************/
 
-  rv = mgr->RemoveListener(0);
-  if (rv != NS_ERROR_NULL_POINTER) {
-    fail("RemoveListener() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Call RemoveListener() with null listener");
-
-  int32_t i;
-  TestTransaction *tximpl;
-  nsITransaction *u1, *u2;
-  nsITransaction *r1, *r2;
+  rv = mgr->RemoveListener(nullptr);
+  EXPECT_EQ(rv, NS_ERROR_NULL_POINTER);
 
   /*******************************************************************
    *
    * Test coalescing by executing a transaction that can merge any
    * command into itself. Then execute 20 transaction. Afterwards,
    * we should still have the first transaction sitting on the undo
    * stack. Then clear the undo and redo stacks.
    *
    *******************************************************************/
 
+  int32_t i;
+  RefPtr<TestTransaction> tximpl;
+  nsCOMPtr<nsITransaction> u1, u2, r1, r2;
+
   rv = mgr->SetMaxTransactionCount(10);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(10) failed. (%d)\n", rv);
-    return rv;
-  }
-
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   tximpl = factory->create(mgr, MERGE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate initial transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for initial transaction. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("Failed to execute initial transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, tximpl);
 
-  if (u1 != tx) {
-    fail("Top of undo stack is different!. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Test coalescing of transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   /*******************************************************************
    *
    * Execute 20 transactions. Afterwards, we should have 10
    * transactions on the undo stack:
    *
    *******************************************************************/
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Execute 20 transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Execute 20 transient transactions. Afterwards, we should still
    * have the same 10 transactions on the undo stack:
    *
    *******************************************************************/
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, TRANSIENT_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Execute 20 transient transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Undo 4 transactions. Afterwards, we should have 6 transactions
    * on the undo stack, and 4 on the redo stack:
    *
    *******************************************************************/
 
   for (i = 1; i <= 4; i++) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 6 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 6) {
-    fail("GetNumberOfUndoItems() expected 6 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 6);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 4 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 4) {
-    fail("GetNumberOfRedoItems() expected 4 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Undo 4 transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 4);
 
   /*******************************************************************
    *
    * Redo 2 transactions. Afterwards, we should have 8 transactions
    * on the undo stack, and 2 on the redo stack:
    *
    *******************************************************************/
 
   for (i = 1; i <= 2; ++i) {
     rv = mgr->RedoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to redo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 8 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 8) {
-    fail("GetNumberOfUndoItems() expected 8 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 8);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfRedoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Redo 2 transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   /*******************************************************************
    *
    * Execute a new transaction. The redo stack should get pruned!
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("Failed to execute transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 9 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 9) {
-    fail("GetNumberOfUndoItems() expected 9 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 9);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Check if new transactions prune the redo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Undo 4 transactions then clear the undo and redo stacks.
    *
    *******************************************************************/
 
   for (i = 1; i <= 4; ++i) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 5 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 5) {
-    fail("GetNumberOfUndoItems() expected 5 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 5);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 4 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 4) {
-    fail("GetNumberOfRedoItems() expected 4 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 4);
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on cleared undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on cleared redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Undo 4 transactions then clear the undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Execute 5 transactions.
    *
    *******************************************************************/
 
   for (i = 1; i <= 5; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 5 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 5) {
-    fail("GetNumberOfUndoItems() expected 5 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 5);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Execute 5 transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Test transaction DoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_DO_ERROR_FLAG);
 
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  u1 = u2 = r1 = r2 = 0;
+  u1 = u2 = r1 = r2 = nullptr;
 
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->DoTransaction(tx);
-  if (rv != NS_ERROR_FAILURE) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 5 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 5) {
-    fail("GetNumberOfUndoItems() expected 5 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 5);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction DoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Test transaction UndoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_UNDO_ERROR_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
+  u1 = u2 = r1 = r2 = nullptr;
 
-  tx->Release();
-
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->UndoTransaction();
-  if (rv != NS_ERROR_FAILURE) {
-    fail("UndoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 6 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 6) {
-    fail("GetNumberOfUndoItems() expected 6 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 6);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction UndoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Test transaction RedoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_REDO_ERROR_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for RedoErrorTransaction. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   //
   // Execute a normal transaction to be used in a later test:
   //
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   //
   // Undo the 2 transactions just executed.
   //
 
   for (i = 1; i <= 2; ++i) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   //
   // The RedoErrorTransaction should now be at the top of the redo stack!
   //
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->RedoTransaction();
-  if (rv != NS_ERROR_FAILURE) {
-    fail("RedoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 6 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 6) {
-    fail("GetNumberOfUndoItems() expected 6 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 6);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfRedoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction RedoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   /*******************************************************************
    *
    * Make sure that setting the transaction manager's max transaction
    * count to zero, clears both the undo and redo stacks, and executes
    * all new commands without pushing them on the undo stack!
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(0);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(0) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->GetNumberOfUndoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
 
     rv = mgr->GetNumberOfRedoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
   }
 
-  passed("Test max transaction count of zero");
-
   /*******************************************************************
    *
    * Make sure that setting the transaction manager's max transaction
    * count to something greater than the number of transactions on
    * both the undo and redo stacks causes no pruning of the stacks:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(-1);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(-1) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   // Push 20 transactions on the undo stack:
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->GetNumberOfUndoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfUndoItems() on undo stack with %d items failed. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    if (numitems != i) {
-      fail("GetNumberOfUndoItems() expected %d got %d. (%d)\n",
-           i, numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, i);
 
     rv = mgr->GetNumberOfRedoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
   }
 
   for (i = 1; i <= 10; i++) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
-  if (numitems != 10) {
-    fail("GetNumberOfRedoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
+  u1 = u2 = r1 = r2 = nullptr;
 
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->SetMaxTransactionCount(25);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(25) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfRedoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test SetMaxTransactionCount() greater than num stack items");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   /*******************************************************************
    *
    * Test undo stack pruning by setting the transaction
    * manager's max transaction count to a number lower than the
    * number of transactions on both the undo and redo stacks:
    *
    *******************************************************************/
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->SetMaxTransactionCount(15);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(15) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 5 items failed. (%d)\n",
-           rv);
-    return rv;
-  }
-
-  if (numitems != 5) {
-    fail("GetNumberOfUndoItems() expected 5 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 5);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfRedoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test SetMaxTransactionCount() pruning undo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   /*******************************************************************
    *
    * Test redo stack pruning by setting the transaction
    * manager's max transaction count to a number lower than the
    * number of transactions on both the undo and redo stacks:
    *
    *******************************************************************/
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->SetMaxTransactionCount(5);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(5) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (u2) {
-    fail("Unexpected item at top of undo stack. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_FALSE(u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 0) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 5 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 5) {
-    fail("GetNumberOfRedoItems() expected 5 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test SetMaxTransactionCount() pruning redo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 5);
 
   /*******************************************************************
    *
    * Release the transaction manager. Any transactions on the undo
    * and redo stack should automatically be released:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(-1);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(-1) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   // Push 20 transactions on the undo stack:
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->GetNumberOfUndoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfUndoItems() on undo stack with %d items failed. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    if (numitems != i) {
-      fail("GetNumberOfUndoItems() expected %d got %d. (%d)\n",
-           i, numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, i);
 
     rv = mgr->GetNumberOfRedoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
   }
 
   for (i = 1; i <= 10; i++) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfRedoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Release the transaction manager");
-
-  /*******************************************************************
-   *
-   * Make sure number of transactions created matches number of
-   * transactions destroyed!
-   *
-   *******************************************************************/
-
-  /* Disabled because the current cycle collector doesn't delete
-     cycle collectable objects synchronously.
-  if (sConstructorCount != sDestructorCount) {
-    fail("Transaction constructor count (%d) != destructor count (%d).\n",
-         sConstructorCount, sDestructorCount);
-    return NS_ERROR_FAILURE;
-  }*/
-
-  passed("Number of transactions created and destroyed match");
-  passed("%d transactions processed during quick test", sConstructorCount);
-
-  return NS_OK;
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 }
 
-nsresult
-simple_test()
+TEST(TestTXMgr, SimpleTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
   reset_globals();
-  sDestructorOrderArr = sSimpleTestDestructorOrderArr;
   sDoOrderArr         = sSimpleTestDoOrderArr;
   sUndoOrderArr       = sSimpleTestUndoOrderArr;
   sRedoOrderArr       = sSimpleTestRedoOrderArr;
 
   /*******************************************************************
    *
    * Run the quick test.
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Begin Simple Transaction Test:\n");
-  printf("-----------------------------------------------------\n");
-
   SimpleTransactionFactory factory;
 
-  return quick_test(&factory);
+  quick_test(&factory);
 }
 
-nsresult
-aggregation_test()
+TEST(TestTXMgr, AggregationTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
 
   reset_globals();
-  sDestructorOrderArr = sAggregateTestDestructorOrderArr;
   sDoOrderArr         = sAggregateTestDoOrderArr;
   sUndoOrderArr       = sAggregateTestUndoOrderArr;
   sRedoOrderArr       = sAggregateTestRedoOrderArr;
 
   /*******************************************************************
    *
    * Run the quick test.
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Begin Aggregate Transaction Test:\n");
-  printf("-----------------------------------------------------\n");
-
   AggregateTransactionFactory factory(3, 2);
 
-  return quick_test(&factory);
+  quick_test(&factory);
 }
 
 /**
  * Test behaviors in batch mode.
  **/
-nsresult
+void
 quick_batch_test(TestTransactionFactory *factory)
 {
   /*******************************************************************
    *
    * Create a transaction manager implementation:
    *
    *******************************************************************/
 
   nsresult rv;
   nsCOMPtr<nsITransactionManager> mgr =
     do_CreateInstance(NS_TRANSACTIONMANAGER_CONTRACTID, &rv);
-  if (NS_FAILED(rv) || !mgr) {
-    fail("Failed to create Transaction Manager instance.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  passed("Create transaction manager instance");
+  ASSERT_TRUE(mgr);
+  ASSERT_TRUE(NS_SUCCEEDED(rv));
 
   int32_t numitems;
 
   /*******************************************************************
    *
    * Make sure an unbalanced call to EndBatch(false) with empty undo stack
    * throws an error!
    *
    *******************************************************************/
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->EndBatch(false);
-  if (rv != NS_ERROR_FAILURE) {
-    fail("EndBatch(false) returned unexpected status. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test unbalanced EndBatch(false) with empty undo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Make sure that an empty batch is not added to the undo stack
    * when it is closed.
    *
    *******************************************************************/
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test empty batch");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   int32_t i;
-  TestTransaction *tximpl;
-  nsITransaction *tx;
+  RefPtr<TestTransaction> tximpl;
 
   /*******************************************************************
    *
    * Execute 20 transactions. Afterwards, we should have 1
    * transaction on the undo stack:
    *
    *******************************************************************/
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 0) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Execute 20 batched transactions");
-
-  nsITransaction *u1, *u2;
-  nsITransaction *r1, *r2;
+  nsCOMPtr<nsITransaction> u1, u2, r1, r2;
 
   /*******************************************************************
    *
    * Execute 20 transient transactions. Afterwards, we should still
    * have the same transaction on the undo stack:
    *
    *******************************************************************/
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, TRANSIENT_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Execute 20 batched transient transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Test nested batching. Afterwards, we should have 2 transactions
    * on the undo stack:
    *
    *******************************************************************/
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("Failed to execute transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("Failed to execute transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx = 0;
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("Failed to execute transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfUndoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test nested batched transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   /*******************************************************************
    *
    * Undo 2 batch transactions. Afterwards, we should have 0
    * transactions on the undo stack and 2 on the redo stack.
    *
    *******************************************************************/
 
   for (i = 1; i <= 2; ++i) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfRedoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Undo 2 batch transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   /*******************************************************************
    *
    * Redo 2 batch transactions. Afterwards, we should have 2
    * transactions on the undo stack and 0 on the redo stack.
    *
    *******************************************************************/
 
   for (i = 1; i <= 2; ++i) {
     rv = mgr->RedoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfUndoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Redo 2 batch transactions");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Call undo. Afterwards, we should have 1 transaction
    * on the undo stack, and 1 on the redo stack:
    *
    *******************************************************************/
 
   rv = mgr->UndoTransaction();
-  if (NS_FAILED(rv)) {
-    fail("Failed to undo transaction. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Undo a batched transaction that was redone");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   /*******************************************************************
    *
    * Make sure an unbalanced call to EndBatch(false) throws an error and
    * doesn't affect the undo and redo stacks!
    *
    *******************************************************************/
 
   rv = mgr->EndBatch(false);
-  if (rv != NS_ERROR_FAILURE) {
-    fail("EndBatch(false) returned unexpected status. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test effect of unbalanced EndBatch(false) on undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   /*******************************************************************
    *
    * Make sure that an empty batch is not added to the undo stack
    * when it is closed, and that it does not affect the undo and redo
    * stacks.
    *
    *******************************************************************/
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test effect of empty batch on undo and redo stacks");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   /*******************************************************************
    *
    * Execute a new transaction. The redo stack should get pruned!
    *
    *******************************************************************/
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
-
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfUndoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Check if new batched transactions prune the redo stack");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Call undo.
    *
    *******************************************************************/
 
   // Move a transaction over to the redo stack, so that we have one
   // transaction on the undo stack, and one on the redo stack!
 
   rv = mgr->UndoTransaction();
-  if (NS_FAILED(rv)) {
-    fail("Failed to undo transaction. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Call undo");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   /*******************************************************************
    *
    * Test transaction DoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_DO_ERROR_FLAG);
 
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx     = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  u1 = u2 = r1 = r2 = 0;
+  u1 = u2 = r1 = r2 = nullptr;
 
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->PeekRedoStack(&r1);
-
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->DoTransaction(tx);
-  if (rv != NS_ERROR_FAILURE) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfUndoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 1 item failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 1) {
-    fail("GetNumberOfRedoItems() expected 1 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction DoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 1);
 
   /*******************************************************************
    *
    * Test transaction UndoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_UNDO_ERROR_FLAG);
 
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx     = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->BeginBatch(nullptr);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->UndoTransaction();
-  if (rv != NS_ERROR_FAILURE) {
-    fail("UndoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfUndoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction UndoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   /*******************************************************************
    *
    * Test transaction RedoTransaction() error:
    *
    *******************************************************************/
 
   tximpl = factory->create(mgr, THROWS_REDO_ERROR_FLAG);
 
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx     = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for RedoErrorTransaction. (%d)\n",
-         rv);
-    return rv;
-  }
+  rv = mgr->BeginBatch(nullptr);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  rv = mgr->BeginBatch(nullptr);
-  if (NS_FAILED(rv)) {
-    fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->EndBatch(false);
-  if (NS_FAILED(rv)) {
-    fail("EndBatch(false) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   //
   // Execute a normal transaction to be used in a later test:
   //
 
   tximpl = factory->create(mgr, NONE_FLAG);
-
-  if (!tximpl) {
-    fail("Failed to allocate transaction.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
-
-  tx     = 0;
-
-  rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-  if (NS_FAILED(rv)) {
-    fail("QueryInterface() failed for transaction. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->DoTransaction(tx);
-  if (NS_FAILED(rv)) {
-    fail("DoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  tx->Release();
+  rv = mgr->DoTransaction(tximpl);
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   //
   // Undo the 2 transactions just executed.
   //
 
   for (i = 1; i <= 2; ++i) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
 
   //
   // The RedoErrorTransaction should now be at the top of the redo stack!
   //
 
-  u1 = u2 = r1 = r2 = 0;
-
-  rv = mgr->PeekUndoStack(&u1);
-
-  TEST_TXMGR_IF_RELEASE(u1); // Don't hold onto any references!
+  u1 = u2 = r1 = r2 = nullptr;
 
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekRedoStack(&r1);
+  rv = mgr->PeekUndoStack(getter_AddRefs(u1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-  TEST_TXMGR_IF_RELEASE(r1); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Initial PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r1));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->RedoTransaction();
-  if (rv != NS_ERROR_FAILURE) {
-    fail("RedoTransaction() returned unexpected error. (%d)\n", rv);
-    return rv;
-  }
-
-  rv = mgr->PeekUndoStack(&u2);
-
-  TEST_TXMGR_IF_RELEASE(u2); // Don't hold onto any references!
-
-  if (NS_FAILED(rv)) {
-    fail("Second PeekUndoStack() failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_EQ(rv, NS_ERROR_FAILURE);
 
-  if (u1 != u2) {
-    fail("Top of undo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  rv = mgr->PeekRedoStack(&r2);
-
-  TEST_TXMGR_IF_RELEASE(r2); // Don't hold onto any references!
+  rv = mgr->PeekUndoStack(getter_AddRefs(u2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(u1, u2);
 
-  if (NS_FAILED(rv)) {
-    fail("Second PeekRedoStack() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  if (r1 != r2) {
-    fail("Top of redo stack changed. (%d)\n", rv);
-    return NS_ERROR_FAILURE;
-  }
+  rv = mgr->PeekRedoStack(getter_AddRefs(r2));
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(r1, r2);
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on undo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfUndoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 2 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 2) {
-    fail("GetNumberOfRedoItems() expected 2 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
-
-  passed("Test transaction RedoTransaction() error");
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 2);
 
   /*******************************************************************
    *
    * Make sure that setting the transaction manager's max transaction
    * count to zero, clears both the undo and redo stacks, and executes
    * all new commands without pushing them on the undo stack!
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(0);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(0) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems) {
-    fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 0);
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
 
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
+    rv = mgr->BeginBatch(nullptr);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-    rv = mgr->BeginBatch(nullptr);
-    if (NS_FAILED(rv)) {
-      fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->EndBatch(false);
-    if (NS_FAILED(rv)) {
-      fail("EndBatch(false) failed. (%d)\n", rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->GetNumberOfUndoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfUndoItems() on empty undo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfUndoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
 
     rv = mgr->GetNumberOfRedoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
   }
 
-  passed("Test max transaction count of zero");
-
   /*******************************************************************
    *
    * Release the transaction manager. Any transactions on the undo
    * and redo stack should automatically be released:
    *
    *******************************************************************/
 
   rv = mgr->SetMaxTransactionCount(-1);
-  if (NS_FAILED(rv)) {
-    fail("SetMaxTransactionCount(0) failed. (%d)\n", rv);
-    return rv;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 
   // Push 20 transactions on the undo stack:
 
   for (i = 1; i <= 20; i++) {
     tximpl = factory->create(mgr, NONE_FLAG);
 
-    if (!tximpl) {
-      fail("Failed to allocate transaction %d.\n", i);
-      return NS_ERROR_OUT_OF_MEMORY;
-    }
-
-    tx = 0;
-    rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-    if (NS_FAILED(rv)) {
-      fail("QueryInterface() failed for transaction %d. (%d)\n",
-           i, rv);
-      return rv;
-    }
+    rv = mgr->BeginBatch(nullptr);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
-    rv = mgr->BeginBatch(nullptr);
-    if (NS_FAILED(rv)) {
-      fail("BeginBatch(nullptr) failed. (%d)\n", rv);
-      return rv;
-    }
-
-    rv = mgr->DoTransaction(tx);
-    if (NS_FAILED(rv)) {
-      fail("Failed to execute transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
-
-    tx->Release();
+    rv = mgr->DoTransaction(tximpl);
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->EndBatch(false);
-    if (NS_FAILED(rv)) {
-      fail("EndBatch(false) failed. (%d)\n", rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
 
     rv = mgr->GetNumberOfUndoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfUndoItems() on undo stack with %d items failed. (%d)\n",
-           i, rv);
-      return rv;
-    }
-
-    if (numitems != i) {
-      fail("GetNumberOfUndoItems() expected %d got %d. (%d)\n",
-           i, numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, i);
 
     rv = mgr->GetNumberOfRedoItems(&numitems);
-    if (NS_FAILED(rv)) {
-      fail("GetNumberOfRedoItems() on empty redo stack failed. (%d)\n",
-           rv);
-      return rv;
-    }
-
-    if (numitems) {
-      fail("GetNumberOfRedoItems() expected 0 got %d. (%d)\n",
-           numitems, rv);
-      return NS_ERROR_FAILURE;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
+    EXPECT_EQ(numitems, 0);
   }
 
   for (i = 1; i <= 10; i++) {
     rv = mgr->UndoTransaction();
-    if (NS_FAILED(rv)) {
-      fail("Failed to undo transaction %d. (%d)\n", i, rv);
-      return rv;
-    }
+    EXPECT_TRUE(NS_SUCCEEDED(rv));
   }
-
   rv = mgr->GetNumberOfUndoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfUndoItems() on empty undo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfUndoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->GetNumberOfRedoItems(&numitems);
-  if (NS_FAILED(rv)) {
-    fail("GetNumberOfRedoItems() on redo stack with 10 items failed. (%d)\n",
-         rv);
-    return rv;
-  }
-
-  if (numitems != 10) {
-    fail("GetNumberOfRedoItems() expected 10 got %d. (%d)\n",
-         numitems, rv);
-    return NS_ERROR_FAILURE;
-  }
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
+  EXPECT_EQ(numitems, 10);
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  passed("Release the transaction manager");
-
-  /*******************************************************************
-   *
-   * Make sure number of transactions created matches number of
-   * transactions destroyed!
-   *
-   *******************************************************************/
-
-  /* Disabled because the current cycle collector doesn't delete
-     cycle collectable objects synchronously.
-  if (sConstructorCount != sDestructorCount) {
-    fail("Transaction constructor count (%d) != destructor count (%d).\n",
-         sConstructorCount, sDestructorCount);
-    return NS_ERROR_FAILURE;
-  }*/
-
-  passed("Number of transactions created and destroyed match");
-  passed("%d transactions processed during quick batch test",
-         sConstructorCount);
-
-  return NS_OK;
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 }
 
-nsresult
-simple_batch_test()
+TEST(TestTXMgr, SimpleBatchTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
   reset_globals();
-  sDestructorOrderArr = sSimpleBatchTestDestructorOrderArr;
   sDoOrderArr         = sSimpleBatchTestDoOrderArr;
   sUndoOrderArr       = sSimpleBatchTestUndoOrderArr;
   sRedoOrderArr       = sSimpleBatchTestRedoOrderArr;
 
   /*******************************************************************
    *
    * Run the quick batch test.
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Begin Batch Transaction Test:\n");
-  printf("-----------------------------------------------------\n");
-
   SimpleTransactionFactory factory;
-
-  return quick_batch_test(&factory);
+  quick_batch_test(&factory);
 }
 
-nsresult
-aggregation_batch_test()
+TEST(TestTXMgr, AggregationBatchTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
 
   reset_globals();
-  sDestructorOrderArr = sAggregateBatchTestDestructorOrderArr;
   sDoOrderArr         = sAggregateBatchTestDoOrderArr;
   sUndoOrderArr       = sAggregateBatchTestUndoOrderArr;
   sRedoOrderArr       = sAggregateBatchTestRedoOrderArr;
 
   /*******************************************************************
    *
    * Run the quick batch test.
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Begin Batch Aggregate Transaction Test:\n");
-  printf("-----------------------------------------------------\n");
-
   AggregateTransactionFactory factory(3, 2, BATCH_FLAG);
 
-  return quick_batch_test(&factory);
+  quick_batch_test(&factory);
 }
 
 /**
  * Create 'iterations * (iterations + 1) / 2' transactions;
  * do/undo/redo/undo them.
  **/
-nsresult
+void
 stress_test(TestTransactionFactory *factory, int32_t iterations)
 {
-  printf("Stress test of %i iterations (may take a while) ... ", iterations);
-  fflush(stdout);
-
   /*******************************************************************
    *
    * Create a transaction manager:
    *
    *******************************************************************/
 
   nsresult rv;
   nsCOMPtr<nsITransactionManager> mgr =
     do_CreateInstance(NS_TRANSACTIONMANAGER_CONTRACTID, &rv);
-  if (NS_FAILED(rv) || !mgr) {
-    fail("Failed to create Transaction Manager instance.\n");
-    return NS_ERROR_OUT_OF_MEMORY;
-  }
+  ASSERT_TRUE(NS_SUCCEEDED(rv));
+  ASSERT_TRUE(mgr);
 
   int32_t i, j;
-  nsITransaction *tx;
 
   for (i = 1; i <= iterations; i++) {
     /*******************************************************************
      *
      * Create and execute a bunch of transactions:
      *
      *******************************************************************/
 
     for (j = 1; j <= i; j++) {
-      TestTransaction *tximpl = factory->create(mgr, NONE_FLAG);
-
-      if (!tximpl) {
-        fail("Failed to allocate transaction %d-%d.\n", i, j);
-        return NS_ERROR_OUT_OF_MEMORY;
-      }
-
-      tx = 0;
-      rv = tximpl->QueryInterface(NS_GET_IID(nsITransaction), (void **)&tx);
-      if (NS_FAILED(rv)) {
-        fail("QueryInterface() failed for transaction %d-%d. (%d)\n",
-             i, j, rv);
-        return rv;
-      }
-
-      rv = mgr->DoTransaction(tx);
-      if (NS_FAILED(rv)) {
-        fail("Failed to execute transaction %d-%d. (%d)\n",
-             i, j, rv);
-        return rv;
-      }
-
-      tx->Release();
+      RefPtr<TestTransaction> tximpl = factory->create(mgr, NONE_FLAG);
+      rv = mgr->DoTransaction(tximpl);
+      EXPECT_TRUE(NS_SUCCEEDED(rv));
     }
 
     /*******************************************************************
      *
      * Undo all the transactions:
      *
      *******************************************************************/
 
     for (j = 1; j <= i; j++) {
       rv = mgr->UndoTransaction();
-      if (NS_FAILED(rv)) {
-        fail("Failed to undo transaction %d-%d. (%d)\n", i, j, rv);
-        return rv;
-      }
+      EXPECT_TRUE(NS_SUCCEEDED(rv));
     }
 
     /*******************************************************************
      *
      * Redo all the transactions:
      *
      *******************************************************************/
 
     for (j = 1; j <= i; j++) {
       rv = mgr->RedoTransaction();
-      if (NS_FAILED(rv)) {
-        fail("Failed to redo transaction %d-%d. (%d)\n", i, j, rv);
-        return rv;
-      }
+      EXPECT_TRUE(NS_SUCCEEDED(rv));
     }
 
     /*******************************************************************
      *
      * Undo all the transactions again so that they all end up on
      * the redo stack for pruning the next time we execute a new
      * transaction
      *
      *******************************************************************/
 
     for (j = 1; j <= i; j++) {
       rv = mgr->UndoTransaction();
-      if (NS_FAILED(rv)) {
-        fail("Failed to undo transaction %d-%d. (%d)\n", i, j, rv);
-        return rv;
-      }
+      EXPECT_TRUE(NS_SUCCEEDED(rv));
     }
-
-    // Trivial feedback not to let the user think the test is stuck.
-    if (MOZ_UNLIKELY(j % 100 == 0)) {
-      printf("%i ", j);
-    }
-  } // for, iterations.
-
-  printf("passed\n");
+  }
 
   rv = mgr->Clear();
-  if (NS_FAILED(rv)) {
-    fail("Clear() failed. (%d)\n", rv);
-    return rv;
-  }
-
-  /* Disabled because the current cycle collector doesn't delete
-     cycle collectable objects synchronously.
-  if (sConstructorCount != sDestructorCount) {
-    fail("Transaction constructor count (%d) != destructor count (%d).\n",
-         sConstructorCount, sDestructorCount);
-    return NS_ERROR_FAILURE;
-  }*/
-
-  passed("%d transactions processed during stress test", sConstructorCount);
-
-  return NS_OK;
+  EXPECT_TRUE(NS_SUCCEEDED(rv));
 }
 
-nsresult
-simple_stress_test()
+TEST(TestTXMgr, SimpleStressTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
 
   reset_globals();
 
   /*******************************************************************
    *
    * Do the stress test:
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Simple Transaction Stress Test:\n");
-  printf("-----------------------------------------------------\n");
-
   SimpleTransactionFactory factory;
 
   int32_t iterations =
 #ifdef DEBUG
   10
 #else
   //
   // 1500 iterations sends 1,125,750 transactions through the system!!
   //
   1500
 #endif
   ;
-  return stress_test(&factory, iterations);
+  stress_test(&factory, iterations);
 }
 
-nsresult
-aggregation_stress_test()
+TEST(TestTXMgr, AggregationStressTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
 
   reset_globals();
 
   /*******************************************************************
    *
    * Do the stress test:
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Aggregate Transaction Stress Test:\n");
-  printf("-----------------------------------------------------\n");
-
   AggregateTransactionFactory factory(3, 4);
 
   int32_t iterations =
 #ifdef DEBUG
   10
 #else
   //
   // 500 iterations sends 2,630,250 transactions through the system!!
   //
   500
 #endif
   ;
-  return stress_test(&factory, iterations);
+  stress_test(&factory, iterations);
 }
 
-nsresult
-aggregation_batch_stress_test()
+TEST(TestTXMgr, AggregationBatchStressTest)
 {
   /*******************************************************************
    *
    * Initialize globals for test.
    *
    *******************************************************************/
 
   reset_globals();
 
   /*******************************************************************
    *
    * Do the stress test:
    *
    *******************************************************************/
 
-  printf("\n-----------------------------------------------------\n");
-  printf("- Aggregate Batch Transaction Stress Test:\n");
-  printf("-----------------------------------------------------\n");
-
   AggregateTransactionFactory factory(3, 4, BATCH_FLAG);
 
   int32_t iterations =
 #ifdef DEBUG
   10
 #else
 #if defined(MOZ_ASAN) || defined(MOZ_WIDGET_ANDROID)
   // See Bug 929985: 500 is too many for ASAN and Android, 100 is safe.
@@ -4425,54 +2075,10 @@ aggregation_batch_stress_test()
 #else
   //
   // 500 iterations sends 2,630,250 transactions through the system!!
   //
   500
 #endif
 #endif
   ;
-  return stress_test(&factory, iterations);
+  stress_test(&factory, iterations);
 }
-
-int
-main (int argc, char *argv[])
-{
-  ScopedXPCOM xpcom("nsITransactionManager");
-  if (xpcom.failed()) {
-    return 1;
-  }
-
-  //
-  // quick_test() part:
-  //
-
-  nsresult rv = simple_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  rv = aggregation_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  //
-  // quick_batch_test() part:
-  //
-
-  rv = simple_batch_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  rv = aggregation_batch_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  //
-  // stress_test() part:
-  //
-
-  rv = simple_stress_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  rv = aggregation_stress_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  rv = aggregation_batch_stress_test();
-  NS_ENSURE_SUCCESS(rv, 1);
-
-  return 0;
-}
--- a/editor/txmgr/tests/moz.build
+++ b/editor/txmgr/tests/moz.build
@@ -1,9 +1,10 @@
 # -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
 # vim: set filetype=python:
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
-GeckoCppUnitTests([
-    'TestTXMgr',
-])
+SOURCES += [
+    'TestTXMgr.cpp',
+]
+FINAL_LIBRARY = 'xul-gtest'
--- a/testing/cppunittest.ini
+++ b/testing/cppunittest.ini
@@ -35,18 +35,16 @@ skip-if = os == 'android' # Bug 1147630
 [TestRefPtr]
 [TestRollingMean]
 [TestScopeExit]
 [TestSegmentedVector]
 [TestSHA1]
 [TestSaturate]
 [TestSplayTree]
 [TestSyncRunnable]
-[TestTXMgr]
-skip-if = os == 'b2g'  #Bug 919595
 [TestTemplateLib]
 [TestTuple]
 [TestTypeTraits]
 [TestTypedEnum]
 [TestUniquePtr]
 [TestVariant]
 [TestVector]
 [TestVolatileBuffer]