Support asynchronous patching operations in the component updater.

chrome/browser/component_updater/component_patcher.cc

Index: chrome/browser/component_updater/component_patcher.cc
diff --git a/chrome/browser/component_updater/component_patcher.cc b/chrome/browser/component_updater/component_patcher.cc
index de082a74296bc9b10196dfc11706fbfc246d465a..2d327381422990e0200eb973ce75543ebfd2b319 100644
--- a/chrome/browser/component_updater/component_patcher.cc
+++ b/chrome/browser/component_updater/component_patcher.cc
@@ -32,6 +32,7 @@ base::ListValue* ReadCommands(const base::FilePath& unpack_path) {
 
 }  // namespace
 
+namespace component_updater {
 
 // The patching support is not cross-platform at the moment.
 ComponentPatcherCrossPlatform::ComponentPatcherCrossPlatform() {}
@@ -49,33 +50,42 @@ ComponentUnpacker::Error ComponentPatcherCrossPlatform::Patch(
 // Takes the contents of a differential component update in input_dir
 // and produces the contents of a full component update in unpack_dir
 // using input_abs_path_ files that the installer knows about.
-ComponentUnpacker::Error DifferentialUpdatePatch(
+void DifferentialUpdatePatch(
     const base::FilePath& input_dir,
     const base::FilePath& unpack_dir,
     ComponentPatcher* patcher,
     ComponentInstaller* installer,
-    int* error) {
-  *error = 0;
+    base::Callback<void(ComponentUnpacker::Error, int)> callback) {
+  int error = 0;
   scoped_ptr<base::ListValue> commands(ReadCommands(input_dir));
-  if (!commands.get())
-    return ComponentUnpacker::kDeltaBadCommands;
+  if (!commands.get()) {
+    callback.Run(ComponentUnpacker::kDeltaBadCommands, error);
+    return;
+  }
 
   for (base::ValueVector::const_iterator command = commands->begin(),
       end = commands->end(); command != end; command++) {
-    if (!(*command)->IsType(base::Value::TYPE_DICTIONARY))
-      return ComponentUnpacker::kDeltaBadCommands;
+    if (!(*command)->IsType(base::Value::TYPE_DICTIONARY)) {
+      callback.Run(ComponentUnpacker::kDeltaBadCommands, error);
+      return;
+    }
     base::DictionaryValue* command_args =
         static_cast<base::DictionaryValue*>(*command);
     scoped_ptr<DeltaUpdateOp> operation(CreateDeltaUpdateOp(command_args));
-    if (!operation)
-      return ComponentUnpacker::kDeltaUnsupportedCommand;
+    if (!operation) {
+      callback.Run(ComponentUnpacker::kDeltaUnsupportedCommand, error);
+      return;
+    }
 
     ComponentUnpacker::Error result = operation->Run(
-        command_args, input_dir, unpack_dir, patcher, installer, error);
-    if (result != ComponentUnpacker::kNone)
-      return result;
+        command_args, input_dir, unpack_dir, patcher, installer, &error);
+    if (result != ComponentUnpacker::kNone) {
+      callback.Run(result, error);
+      return;
+    }
   }
 
-  return ComponentUnpacker::kNone;
+  callback.Run(ComponentUnpacker::kNone, error);
 }
 
+}  // namespace component_updater