Use UtilityProcessHost to patch files.

chrome/browser/component_updater/component_patcher.h

 // Copyright 2013 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.
 
 // Component updates can be either differential updates or full updates.
 // Full updates come in CRX format; differential updates come in CRX-style
 // archives, but have a different magic number. They contain "commands.json", a
 // list of commands for the patcher to follow. The patcher uses these commands,
 // the other files in the archive, and the files from the existing installation
 // of the component to create the contents of a full update, which is then
@@ skipping 10 matching lines @@
 // and allowed to, and fall back to a full update if it fails.
 //
 // After installation (diff or full), the component updater records "fp", the
 // fingerprint of the installed files, to later identify the existing files to
 // the server so that a proper differential update can be provided next cycle.
 
 
 #ifndef CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_PATCHER_H_
 #define CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_PATCHER_H_
 
-#include "base/basictypes.h"
 #include "base/callback_forward.h"
-#include "base/compiler_specific.h"
 #include "chrome/browser/component_updater/component_unpacker.h"
 
 namespace base {
 class FilePath;
 }
 
 namespace component_updater {
 
 class ComponentInstaller;
+class DeltaUpdateOp;
 
-// Applies a delta patch to a single file. Specifically, creates a file at
-// |output_file| using |input_file| patched according to the algorithm
-// specified by |patch_type| using |patch_file|. Sets the value of error to
-// the error code of the failing patch operation, if there is such a failure.
+// The type of a patch file.
+enum PatchType {
+  kPatchTypeUnknown,
+  kPatchTypeCourgette,
+  kPatchTypeBsdiff,
+};
+
+// Applies a delta patch to a set of files.
+// The constructor takes an unpacked differential CRX (|input_dir|) and a

[Sorin Jianu, 2014/02/03 20:57:57]

This comment appears to be a function comment instead of a class comment.

[waffles, 2014/02/07 01:00:59]

Done.

+// component installer, and sets up the class to create a new (non-differential)
+// unpacked CRX.
+// The non-differential files are written into the |unpack_dir| directory.
+// When finished, calls |callback|, passing error codes if any errors were
+// encountered.

[Sorin Jianu, 2014/02/03 20:57:57]

Calls |callback| when finished, with the error codes if any errors were
encountered.

[waffles, 2014/02/07 01:00:59]

Done.

 class ComponentPatcher {
  public:
-  // The type of a patch file.
-  enum PatchType {
-    kPatchTypeUnknown,
-    kPatchTypeCourgette,
-    kPatchTypeBsdiff,
-  };
+  ComponentPatcher(const base::FilePath& input_dir,
+                   const base::FilePath& unpack_dir,
+                   ComponentInstaller* installer,
+                   bool in_process,

[Sorin Jianu, 2014/02/03 20:57:57]

might want to document |in_process|.

[waffles, 2014/02/07 01:00:59]

Done.

+                   scoped_refptr<base::SequencedTaskRunner> task_runner);
 
-  virtual ComponentUnpacker::Error Patch(PatchType patch_type,
-                                         const base::FilePath& input_file,
-                                         const base::FilePath& patch_file,
-                                         const base::FilePath& output_file,
-                                         int* error) = 0;
-  virtual ~ComponentPatcher() {}
+  virtual ~ComponentPatcher();
+
+  // Starts patching files. This method returns immediately, after posting a

[Sorin Jianu, 2014/02/03 20:57:57]

Technically, in C++ only virtual member functions are called "methods" although
people are loose with the terminology in many cases.

[waffles, 2014/02/07 01:00:59]

Could you provide me with a reference on this? All explanations I've found so
far equate member functions with methods, whether or not they are virtual.

A quick search of a (working draft) of the standard seems to indicate that the
standard doesn't mention "method", instead preferring "member function".

[Sorin Jianu, 2014/02/27 20:53:57]

http://books.google.com/books?id=0klsAQAAQBAJ&printsec=frontcover&source=gbs_...

Look on the definition on the page 586.

+  // task to do the patching.
+  void Start(
+      const base::Callback<void(ComponentUnpacker::Error, int)>& callback);
+
+  // Returns a weak pointer to this object.
+  base::WeakPtr<ComponentPatcher> GetWeakPtr();
+
+ private:
+  void PatchNextFile();
+
+  void DonePatchingFile(ComponentUnpacker::Error error, int extended_error);
+
+  base::FilePath input_dir_;

[Sorin Jianu, 2014/02/03 20:57:57]

Can some of these members be declared const? The ones initialized in the ctor?

[waffles, 2014/02/07 01:00:59]

Done.

+  base::FilePath unpack_dir_;
+  ComponentInstaller* installer_;
+  bool in_process_;
+  base::Callback<void(ComponentUnpacker::Error, int)> callback_;
+  scoped_ptr<base::ListValue> commands_;
+  base::ValueVector::const_iterator next_command_;
+  scoped_ptr<DeltaUpdateOp> current_operation_;
+  base::WeakPtrFactory<ComponentPatcher> ptr_factory_;
+  scoped_refptr<base::SequencedTaskRunner> task_runner_;
+
+  DISALLOW_COPY_AND_ASSIGN(ComponentPatcher);
 };
 
-class ComponentPatcherCrossPlatform : public ComponentPatcher {
- public:
-  ComponentPatcherCrossPlatform();
-  virtual ComponentUnpacker::Error Patch(PatchType patch_type,
-                                         const base::FilePath& input_file,
-                                         const base::FilePath& patch_file,
-                                         const base::FilePath& output_file,
-                                         int* error) OVERRIDE;
- private:
-  DISALLOW_COPY_AND_ASSIGN(ComponentPatcherCrossPlatform);
-};
-
-// This function takes an unpacked differential CRX (|input_dir|) and a
-// component installer, and creates a new (non-differential) unpacked CRX, which
-// is then installed normally.
-// The non-differential files are written into the |unpack_dir| directory.
-// When finished, calls the callback, passing error codes if any errors were
-// encountered.
-void DifferentialUpdatePatch(
-    const base::FilePath& input_dir,
-    const base::FilePath& unpack_dir,
-    ComponentPatcher* component_patcher,
-    ComponentInstaller* installer,
-    base::Callback<void(ComponentUnpacker::Error, int)> callback);
-
 }  // namespace component_updater
 
 #endif  // CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_PATCHER_H_