Include v2 packaged apps in sync.

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

Index: chrome/browser/sync/test/integration/apps_helper.cc
diff --git a/chrome/browser/sync/test/integration/apps_helper.cc b/chrome/browser/sync/test/integration/apps_helper.cc
index faf0adde2e75095164eb95377c928f31c6554eb0..4ff62b79fa434acb82c4b119c9a317d2668becc9 100644
--- a/chrome/browser/sync/test/integration/apps_helper.cc
+++ b/chrome/browser/sync/test/integration/apps_helper.cc
@@ -47,6 +47,13 @@ std::string InstallApp(Profile* profile, int index) {
       extensions::Extension::TYPE_HOSTED_APP);
 }
 
+std::string InstallPlatformApp(Profile* profile, int index) {
+  return SyncExtensionHelper::GetInstance()->InstallExtension(
+      profile,
+      CreateFakeAppName(index),

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

can you add a CreateFakePlatformAppName() function?  Otherwise bad things might
happen if for some reason a fake app and a fake platform app for the same index
are installed.

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

Done.

[Marijn Kruisselbrink, 2012/11/19 18:43:01]

Actually, I decided to undo this again (since this change was what caused the
test to not pass). There are just too many places in apps_helper where it
assumes it can determine the name from just an index (the alternative would be
to have 2 copies of almost every method or pass an extension type to almost
every method).

+      extensions::Extension::TYPE_PLATFORM_APP);
+}
+
 std::string InstallAppForAllProfiles(int index) {
   for (int i = 0; i < test()->num_clients(); ++i)
     InstallApp(test()->GetProfile(i), index);