Fixed and added new tests in SyncIntegrationTest class.

chrome/test/functional/PYAUTO_TESTS

Index: chrome/test/functional/PYAUTO_TESTS
===================================================================
--- chrome/test/functional/PYAUTO_TESTS	(revision 110005)
+++ chrome/test/functional/PYAUTO_TESTS	(working copy)
@@ -134,7 +134,7 @@
     'win': [
       # Enable sync.SyncIntegrationTest only for windows CONTINUIOUS,
       # which contains the sync protobufs.
-      'sync.SyncIntegrationTest',
+      #'sync.SyncIntegrationTest',

[dennis_jeffrey, 2011/11/15 02:07:13]

Is this disabled temporarily or permanently?  If permanent, we should remove
lines 135-137.

[dyu1, 2011/11/15 02:56:08]

Temp, I added the the disable to lines 160,161. 

[dennis_jeffrey, 2011/11/15 18:05:31]

If we're disabling it down below, why do we need to comment out the current
line?  This is just another change we'll have to make later when we re-enable
the test.

 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.
@@ -157,14 +157,14 @@
       '-passwords.PasswordTest.testClearFetchedCredForNewUserName',
       # causes browser crash?  crbug.com/99865
       '-prefs.PrefsTest.testGeolocationPref',
+      # crbug.com/104227
+      '-sync.SyncIntegrationTest',
       # crbug.com/88679
       '-sync.SyncTest',
       # crbug.com/86949
       '-sync.SyncTest.testRestartBrowser',
       # crbug.com/87642
       '-sync.SyncTest.testDisableAndEnableDatatypes',
-      # crbug.com/104179
-      '-sync.SyncIntegrationTest.testAddBookmarkAndVerifySync',
       # crbug.com/95621
       '-youtube.YoutubeTest.testPlayerStatus',
       # crbug.com/98526
@@ -491,7 +491,7 @@
       '-enterprise.EnterpriseTest.testDisabledPlugins',
       '-enterprise.EnterpriseTest.testEnabledPlugins',
       # crbug.com/100893
-      '-enterprise.EnterpriseTestReverse.testEnableSPDY',      
+      '-enterprise.EnterpriseTestReverse.testEnableSPDY',
     ],
 
     'mac': [