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..2215ec2740a19e425499d8e37f4807a2d5de5d6f
--- /dev/null
+++ b/chrome/browser/extensions/app_data_migrator.h
@@ -0,0 +1,39 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[benwells, 2015/01/14 03:34:28]

Nit: now it is 2015...

+// 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);
+
+  static bool NeedsMigration(const Extension* old, const Extension* extension);
+
+  void DoMigrationAndReply(const Extension* old,
+                           const Extension* extension,
+                           const base::Closure& reply);
+
+ private:
+  ExtensionRegistry* registry_;

[benwells, 2015/01/14 03:34:28]

Nit: you can remove the blank line between the variable declarations.

+
+  Profile* profile_;
+
+  DISALLOW_COPY_AND_ASSIGN(AppDataMigrator);
+};
+
+}  // namespace extensions

[benwells, 2015/01/14 03:34:28]

Nit: blank line after namespace.

+#endif  // CHROME_BROWSER_EXTENSIONS_APP_DATA_MIGRATOR_H_