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

storage/browser/fileapi/obfuscated_file_util.cc

Index: storage/browser/fileapi/obfuscated_file_util.cc
diff --git a/storage/browser/fileapi/obfuscated_file_util.cc b/storage/browser/fileapi/obfuscated_file_util.cc
index 8f696c031f4900bd070627dc40c9a67c50775c5d..c9573fef28c6270dc793e522401f7e0b8a57d70a 100644
--- a/storage/browser/fileapi/obfuscated_file_util.cc
+++ b/storage/browser/fileapi/obfuscated_file_util.cc
@@ -898,6 +898,19 @@ bool ObfuscatedFileUtil::DeleteDirectoryForOriginAndType(
   return base::DeleteFile(origin_path, true /* recursive */);
 }
 
+void ObfuscatedFileUtil::CloseFileSystemForOriginAndType(
+    const GURL& origin,
+    const std::string& type_string) {
+  std::string key = GetDirectoryDatabaseKey(origin, type_string);
+  if (key.empty())
+    return;
+  DirectoryMap::iterator iter = directories_.find(key);

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

I don't know this class very well, but looking at
ObfuscatedFileUtil::DestroyDirectoryDatabase() it appears that multiple keys may
match. It does a loop and this function does not. Is that required?

[ryanackley, 2015/01/12 21:18:17]

I think there was a recent change because this code was originally taken from
that method.

+  if (iter == directories_.end())
+    return;
+  scoped_ptr<SandboxDirectoryDatabase> database(iter->second);
+  directories_.erase(iter);
+}
+
 ObfuscatedFileUtil::AbstractOriginEnumerator*
 ObfuscatedFileUtil::CreateOriginEnumerator() {
   std::vector<SandboxOriginDatabase::OriginRecord> origins;