Migrates legacy packaged app data when it's upgraded to a platform app

chrome/browser/extensions/app_data_migrator.h

Index: chrome/browser/extensions/app_data_migrator.h
diff --git a/chrome/browser/extensions/app_data_migrator.h b/chrome/browser/extensions/app_data_migrator.h
new file mode 100644
index 0000000000000000000000000000000000000000..af72f9fc54307735aa996f044b34d6dfd3c17545
--- /dev/null
+++ b/chrome/browser/extensions/app_data_migrator.h
@@ -0,0 +1,39 @@
+// Copyright 2014 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_APP_DATA_MIGRATOR_H_
+#define CHROME_BROWSER_EXTENSIONS_APP_DATA_MIGRATOR_H_
+
+#include "base/callback_forward.h"
+#include "base/macros.h"
+
+class Profile;
+
+namespace extensions {
+class Extension;
+class ExtensionRegistry;
+
+// This class migrates legacy packaged app data in the general storage
+// partition to an isolated storage partition. This happens when a legacy
+// packaged app is upgraded to a platform app. See http://crbug.com/302577
+class AppDataMigrator {
+ public:
+  AppDataMigrator(Profile* profile, ExtensionRegistry* registry);
+
+  bool NeedsMigration(const Extension* old, const Extension* extension);

[cmumford, 2015/01/06 15:30:26]

The name suggests this can be a const method - can it? Actually - can't it just
be a static method?

+
+  void DoMigrationAndReply(const Extension* old,
+                           const Extension* extension,
+                           const base::Closure& reply);
+
+ private:
+  ExtensionRegistry* registry_;
+
+  Profile* profile_;
+
+  DISALLOW_COPY_AND_ASSIGN(AppDataMigrator);
+};
+
+}  // namespace extensions
+#endif  // CHROME_BROWSER_EXTENSIONS_APP_DATA_MIGRATOR_H_