Fix some warnings on Mac that are treated as errors:...

chrome/browser/sync/engine/syncer_unittest.cc

Index: chrome/browser/sync/engine/syncer_unittest.cc
===================================================================
--- chrome/browser/sync/engine/syncer_unittest.cc	(revision 29024)
+++ chrome/browser/sync/engine/syncer_unittest.cc	(working copy)
@@ -4344,8 +4344,6 @@
   }
 }
 
-namespace {

[Fred, 2009/10/15 00:21:05]

Instead of removing these namespace clauses, just include the TEST_F into the
anonymous namespace; that should work unless there's a FRIEND_TEST for these
tests.

[Munjal (Google), 2009/10/15 00:32:51]

I thought it's better to be consistent within the file rather than putting
maximum things in anonymous namespace. If I put some TEST_F inside anonymous and
some outside based on what has FRIEND_TEST and what doesn't it will become
inconsistent within the same file.

-
 class SyncerPositionUpdateTest : public SyncerTest {
  public:
   SyncerPositionUpdateTest() : next_update_id_(1), next_revision_(1) {}
@@ -4395,8 +4393,6 @@
   DISALLOW_COPY_AND_ASSIGN(SyncerPositionUpdateTest);
 };
 
-}  // namespace
-
 TEST_F(SyncerPositionUpdateTest, InOrderPositive) {
   // Add a bunch of items in increasing order, starting with just positive
   // position values.
@@ -4462,8 +4458,6 @@
   ExpectLocalItemsInServerOrder();
 }
 
-namespace {
-
 class SyncerPositionTiebreakingTest : public SyncerTest {
  public:
   SyncerPositionTiebreakingTest()
@@ -4518,8 +4512,6 @@
   DISALLOW_COPY_AND_ASSIGN(SyncerPositionTiebreakingTest);
 };
 
-}  // namespace
-
 TEST_F(SyncerPositionTiebreakingTest, LowMidHigh) {
   Add(low_id_);
   Add(mid_id_);