More installer refactoring in the interest of fixing some bugs and cleaning t...

chrome/installer/util/chrome_browser_operations.h

+// Copyright (c) 2011 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_INSTALLER_UTIL_CHROME_BROWSER_OPERATIONS_H_
+#define CHROME_INSTALLER_UTIL_CHROME_BROWSER_OPERATIONS_H_
+#pragma once
+
+#include <set>
+#include <string>
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "chrome/installer/util/product_operations.h"
+
+namespace installer {
+
+class ChromeBrowserOperations : public ProductOperations {

[robertshield, 2011/01/20 23:36:34]

Perhaps a comment referring the reader to ProductOperations for documentation?

[grt (UTC plus 2), 2011/01/21 05:27:51]

Done.

+ public:
+  ChromeBrowserOperations() {}
+
+  virtual void ReadOptions(const MasterPreferences& prefs,
+                           std::set<std::wstring>* options) OVERRIDE;
+
+  virtual void ReadOptions(const CommandLine& uninstall_command,
+                           std::set<std::wstring>* options) OVERRIDE;
+
+  virtual void AddKeyFiles(const std::set<std::wstring>& options,
+                           std::vector<FilePath>* key_files) OVERRIDE;
+
+  virtual void AddComDllList(const std::set<std::wstring>& options,
+                             std::vector<FilePath>* com_dll_list) OVERRIDE;
+
+  virtual void AppendProductFlags(
+      const std::set<std::wstring>& options,
+      CommandLine* uninstall_command) OVERRIDE;
+
+  virtual bool SetChannelFlags(const std::set<std::wstring>& options,
+                               bool set,
+                               ChannelInfo* channel_info) const OVERRIDE;
+
+  virtual bool ShouldCreateUninstallEntry(
+      const std::set<std::wstring>& options) const OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ChromeBrowserOperations);
+};
+
+}  // namespace installer
+
+#endif  // CHROME_INSTALLER_UTIL_CHROME_BROWSER_OPERATIONS_H_