Cleanup: Remove unneeded forward declarations in chrome/installer.

chrome/installer/util/product.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_PRODUCT_H_
 #define CHROME_INSTALLER_UTIL_PRODUCT_H_
 #pragma once
 
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/installer/util/browser_distribution.h"
 
 class CommandLine;
 
 namespace installer {
 
+class ChannelInfo;

[grt (UTC plus 2), 2011/11/11 14:48:42]

This is needed for Product::SetChannelFlags(bool, ChannelInfo*), no?

[Lei Zhang, 2011/11/11 19:50:53]

Yes, that's why I'm adding it.

[grt (UTC plus 2), 2011/11/12 01:47:32]

Vision fail.

 class MasterPreferences;
 class Product;
 class ProductOperations;
 
 // Represents an installation of a specific product which has a one-to-one
 // relation to a BrowserDistribution.  A product has registry settings, related
 // installation/uninstallation actions and exactly one Package that represents
 // the files on disk.  The Package may be shared with other Product instances,
 // so only the last Product to be uninstalled should remove the package.
 // Right now there are no classes that derive from Product, but in
@@ skipping 90 matching lines @@
   scoped_ptr<ProductOperations> operations_;
   std::set<std::wstring> options_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Product);
 };
 
 }  // namespace installer
 
 #endif  // CHROME_INSTALLER_UTIL_PRODUCT_H_