[Sync] Enable SyncBackendHostTest.FirstTimeSync on IOS

components/sync/driver/glue/sync_backend_host_impl_unittest.cc

 // Copyright 2013 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 "components/sync/driver/glue/sync_backend_host_impl.h"
 
 #include <cstddef>
 #include <utility>
 
 #include "base/files/file_util.h"
@@ skipping 276 matching lines @@
   InitializeBackend(true);
   EXPECT_EQ(ControlTypes(), fake_manager_->GetAndResetDownloadedTypes());
   EXPECT_EQ(ControlTypes(), fake_manager_->InitialSyncEndedTypes());
   EXPECT_TRUE(
       fake_manager_->GetTypesWithEmptyProgressMarkerToken(ControlTypes())
           .Empty());
 }
 
 // Test first time sync scenario. All types should be properly configured.
 
-#if defined(OS_IOS)
-// http://crbug.com/658619
-#define MAYBE_FirstTimeSync DISABLED_FirstTimeSync
-#else
 #define MAYBE_FirstTimeSync FirstTimeSync

[Nicolas Zea, 2016/11/23 00:49:21]

nit: just remove this line, and rename the test back to FirstTimeSync. No reason
to leave MAYBE_FirstTimeSync around.

[Gang Wu, 2016/11/23 17:34:11]

Done.

-#endif
 TEST_F(SyncBackendHostTest, MAYBE_FirstTimeSync) {
   InitializeBackend(true);
   EXPECT_EQ(ControlTypes(), fake_manager_->GetAndResetDownloadedTypes());
   EXPECT_EQ(ControlTypes(), fake_manager_->InitialSyncEndedTypes());
   EXPECT_TRUE(
       fake_manager_->GetTypesWithEmptyProgressMarkerToken(ControlTypes())
           .Empty());
 
   ModelTypeSet ready_types = ConfigureDataTypes(
       enabled_types_, Difference(ModelTypeSet::All(), enabled_types_),
@@ skipping 517 matching lines @@
   backend_->StopSyncingForShutdown();
   // Verify that call to DeactivateNonBlockingDataType doesn't assert.
   backend_->DeactivateNonBlockingDataType(AUTOFILL);
   backend_->Shutdown(STOP_SYNC);
   backend_.reset();
 }
 
 }  // namespace
 
 }  // namespace syncer