Include v2 packaged apps in sync.

chrome/browser/sync/test/integration/single_client_apps_sync_test.cc

Index: chrome/browser/sync/test/integration/single_client_apps_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/single_client_apps_sync_test.cc b/chrome/browser/sync/test/integration/single_client_apps_sync_test.cc
index cff03fa8d271d4919fa1bbd9d7c9b1393370c462..62deed2dd92612d86e484967e86ac7418f9918e7 100644
--- a/chrome/browser/sync/test/integration/single_client_apps_sync_test.cc
+++ b/chrome/browser/sync/test/integration/single_client_apps_sync_test.cc
@@ -9,6 +9,7 @@
 
 using apps_helper::AllProfilesHaveSameAppsAsVerifier;
 using apps_helper::InstallApp;
+using apps_helper::InstallPlatformApp;
 
 class SingleClientAppsSyncTest : public SyncTest {
  public:
@@ -40,6 +41,20 @@ IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, StartWithSomeApps) {
   ASSERT_TRUE(AllProfilesHaveSameAppsAsVerifier());
 }
 
+IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, StartWithSomePlatformApps) {
+  ASSERT_TRUE(SetupClients());
+
+  const int kNumApps = 5;
+  for (int i = 0; i < kNumApps; ++i) {
+    InstallPlatformApp(GetProfile(0), i);
+    InstallPlatformApp(verifier(), i);
+  }
+
+  ASSERT_TRUE(SetupSync());
+
+  ASSERT_TRUE(AllProfilesHaveSameAppsAsVerifier());

[akalin, 2012/11/16 18:38:53]

Doesn't AllProfileHaveSameAppsAsVerifier() have to be modified to look for
platform apps? (It may be that the answer is 'no' if it just looks at the sync
type and not the extension type)

(Or perhaps have an AllProfilesHaveSamePlatformAppsAsVerifier() function)

[Marijn Kruisselbrink, 2012/11/16 18:54:30]

As far as I could tell AllProfileHaveSameAppsAsVerifier indeed only looks at the
sync type (so the tests I added would have passed also without the
Extension::GetSyncType change, since then it would just not include them in the
verification either...).

[akalin, 2012/11/16 20:09:06]

Okay, I think you're right.

Can you update the comment to AllProfilesHaveSameAppsAsVerifier() to mention
platform apps?

+}
+
 IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, InstallSomeApps) {

[akalin, 2012/11/16 18:38:53]

can you add an InstallSomePlatformApps test also?

perhaps also another test that installs a mixture of both.

[Marijn Kruisselbrink, 2012/11/16 18:54:30]

Done.

   ASSERT_TRUE(SetupSync());