base: Reverse the parameters passed to EXPECT_EQ in ObserverList test.

base/observer_list_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/observer_list.h"
 #include "base/observer_list_threadsafe.h"
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 183 matching lines @@
 
   observer_list.AddObserver(&evil);
   observer_list.AddObserver(&c);
   observer_list.AddObserver(&d);
 
   // Removing an observer not in the list should do nothing.
   observer_list.RemoveObserver(&e);
 
   FOR_EACH_OBSERVER(Foo, observer_list, Observe(10));
 
-  EXPECT_EQ(a.total, 20);
-  EXPECT_EQ(b.total, -20);
-  EXPECT_EQ(c.total, 0);
-  EXPECT_EQ(d.total, -10);
-  EXPECT_EQ(e.total, 0);
+  EXPECT_EQ(20, a.total);
+  EXPECT_EQ(-20, b.total);
+  EXPECT_EQ(0, c.total);
+  EXPECT_EQ(-10, d.total);
+  EXPECT_EQ(0, e.total);
 }
 
 TEST(ObserverListThreadSafeTest, BasicTest) {
   MessageLoop loop;
 
   scoped_refptr<ObserverListThreadSafe<Foo> > observer_list(
       new ObserverListThreadSafe<Foo>);
   Adder a(1);
   Adder b(-1);
   Adder c(1);
   Adder d(-1);
   ThreadSafeDisrupter evil(observer_list.get(), &c);
 
   observer_list->AddObserver(&a);
   observer_list->AddObserver(&b);
 
   observer_list->Notify(&Foo::Observe, 10);
   loop.RunAllPending();
 
   observer_list->AddObserver(&evil);
   observer_list->AddObserver(&c);
   observer_list->AddObserver(&d);
 
   observer_list->Notify(&Foo::Observe, 10);
   loop.RunAllPending();
 
-  EXPECT_EQ(a.total, 20);
-  EXPECT_EQ(b.total, -20);
-  EXPECT_EQ(c.total, 0);
-  EXPECT_EQ(d.total, -10);
+  EXPECT_EQ(20, a.total);
+  EXPECT_EQ(-20, b.total);
+  EXPECT_EQ(0, c.total);
+  EXPECT_EQ(-10, d.total);
 }
 
 TEST(ObserverListThreadSafeTest, RemoveObserver) {
   MessageLoop loop;
 
   scoped_refptr<ObserverListThreadSafe<Foo> > observer_list(
       new ObserverListThreadSafe<Foo>);
   Adder a(1), b(1);
 
   // A workaround for the compiler bug. See http://crbug.com/121960.
   EXPECT_NE(&a, &b);
 
   // Should do nothing.
   observer_list->RemoveObserver(&a);
   observer_list->RemoveObserver(&b);
 
   observer_list->Notify(&Foo::Observe, 10);
   loop.RunAllPending();
 
-  EXPECT_EQ(a.total, 0);
-  EXPECT_EQ(b.total, 0);
+  EXPECT_EQ(0, a.total);
+  EXPECT_EQ(0, b.total);
 
   observer_list->AddObserver(&a);
 
   // Should also do nothing.
   observer_list->RemoveObserver(&b);
 
   observer_list->Notify(&Foo::Observe, 10);
   loop.RunAllPending();
 
-  EXPECT_EQ(a.total, 10);
-  EXPECT_EQ(b.total, 0);
+  EXPECT_EQ(10, a.total);
+  EXPECT_EQ(0, b.total);
 }
 
 TEST(ObserverListThreadSafeTest, WithoutMessageLoop) {
   scoped_refptr<ObserverListThreadSafe<Foo> > observer_list(
       new ObserverListThreadSafe<Foo>);
 
   Adder a(1), b(1), c(1);
 
   // No MessageLoop, so these should not be added.
   observer_list->AddObserver(&a);
@@ skipping 259 matching lines @@
   ObserverList<Foo>* observer_list = new ObserverList<Foo>;
   ListDestructor a(observer_list);
   observer_list->AddObserver(&a);
 
   FOR_EACH_OBSERVER(Foo, *observer_list, Observe(0));
   // If this test fails, there'll be Valgrind errors when this function goes out
   // of scope.
 }
 
 }  // namespace