Componentize VersionInfo.

chrome/common/chrome_version_info.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_COMMON_CHROME_VERSION_INFO_H_
 #define CHROME_COMMON_CHROME_VERSION_INFO_H_
 
 #include <string>
 
-#include "base/basictypes.h"
+#include "components/version_info/version_info.h"
 
 namespace chrome {
 
 // An instance of chrome::VersionInfo has information about the
 // current running build of Chrome.
 class VersionInfo {

[brettw, 2015/07/27 19:42:31]

I think this class should be deleted, or maybe renamed ChannelInfo with just the
channel function in it.

It wasn't clear from the CL description if you plan to do this. I agree doing
that rename at the same time of this patch isn't desirable, but I think we
should follow-up.

Can you add a note about what you should use instead with a TODO and a bug link,
and then follow up on that after this lands?

[sdefresne, 2015/07/28 08:58:26]

It makes sense. Created
https://code.google.com/p/chromium/issues/detail?id=514562 as the CL is already
large enough.

  public:
-  // The possible channels for an installation, from most fun to most stable.
-  enum Channel {
-    CHANNEL_UNKNOWN = 0,  // Probably blue
-    CHANNEL_CANARY,       // Yellow
-    CHANNEL_DEV,          // Technicolor
-    CHANNEL_BETA,         // Rainbow
-    CHANNEL_STABLE        // Full-spectrum
-  };
-
   VersionInfo();
   ~VersionInfo();
 
   // E.g. "Chrome/a.b.c.d"
-  std::string ProductNameAndVersionForUserAgent() const;
+  static std::string ProductNameAndVersionForUserAgent();
 
   // E.g. "Chromium" or "Google Chrome".
-  std::string Name() const;
+  static std::string Name();
 
   // Version number, e.g. "6.0.490.1".
-  std::string Version() const;
+  static std::string Version();
 
   // The SVN revision of this release.  E.g. "55800".
-  std::string LastChange() const;
+  static std::string LastChange();
 
   // Whether this is an "official" release of the current Version():
   // whether knowing Version() is enough to completely determine what
   // LastChange() is.
-  bool IsOfficialBuild() const;
+  static bool IsOfficialBuild();
 
   // OS type. E.g. "Windows", "Linux", "FreeBSD", ...
-  std::string OSType() const;
+  static std::string OSType();
 
   // Returns a human-readable modifier for the version string. For a branded
   // build, this modifier is the channel ("canary", "dev", or "beta", but ""
   // for stable). On Windows, this may be modified with additional information
   // after a hyphen. For multi-user installations, it will return "canary-m",
   // "dev-m", "beta-m", and for a stable channel multi-user installation, "m".
   // In branded builds, when the channel cannot be determined, "unknown" will
   // be returned. In unbranded builds, the modifier is usually an empty string
   // (""), although on Linux, it may vary in certain distributions.
   // GetVersionStringModifier() is intended to be used for display purposes.
   // To simply test the channel, use GetChannel().
   static std::string GetVersionStringModifier();
 
   // Returns the channel for the installation. In branded builds, this will be
   // CHANNEL_STABLE, CHANNEL_BETA, CHANNEL_DEV, or CHANNEL_CANARY. In unbranded
   // builds, or in branded builds when the channel cannot be determined, this
   // will be CHANNEL_UNKNOWN.
-  static Channel GetChannel();
+  static version_info::Channel GetChannel();
 
   // Returns a string equivalent of the channel, independent of whether it is a
   // branded build or not and without any additional modifiers.
   static std::string GetChannelString();
 
 #if defined(OS_CHROMEOS)
   // Sets channel before use.
   static void SetChannel(const std::string& channel);
 #endif
 
   // Returns a version string to be displayed in "About Chromium" dialog.
-  std::string CreateVersionString() const;
+  static std::string CreateVersionString();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(VersionInfo);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_COMMON_CHROME_VERSION_INFO_H_