Modified components ui to address concern of all the time disabled check update button.

chrome/browser/component_updater/component_updater_service.h

Index: chrome/browser/component_updater/component_updater_service.h
diff --git a/chrome/browser/component_updater/component_updater_service.h b/chrome/browser/component_updater/component_updater_service.h
index 99164c3b16b4ff568626f586b126b4657e28ec56..6b7bd80eaff6c711d0547eaa0fcaf788e34736b2 100644
--- a/chrome/browser/component_updater/component_updater_service.h
+++ b/chrome/browser/component_updater/component_updater_service.h
@@ -78,16 +78,7 @@ struct CrxComponent {
   ~CrxComponent();
 };
 
-// Convenience structure to use with component listing / enumeration.
-struct CrxComponentInfo {
-  // |id| is currently derived from |CrxComponent.pk_hash|, see rest of the
-  // class implementation for details.
-  std::string id;
-  std::string version;
-  std::string name;
-  CrxComponentInfo();
-  ~CrxComponentInfo();
-};
+struct CrxUpdateItem;
 
 // The component update service is in charge of installing or upgrading
 // select parts of chrome. Each part is called a component and managed by
@@ -205,9 +196,6 @@ class ComponentUpdateService {
   // before calling Start().
   virtual Status RegisterComponent(const CrxComponent& component) = 0;
 
-  // Returns a list of registered components.
-  virtual void GetComponents(std::vector<CrxComponentInfo>* components) = 0;
-
   // Returns a network resource throttle. It means that a component will be
   // downloaded and installed before the resource is unthrottled. This is the
   // only function callable from the IO thread.
@@ -221,6 +209,13 @@ class ComponentUpdateService {
   friend class OnDemandTester;
 
  private:
+  // Returns a list of registered components.
+  virtual std::vector<std::string> GetComponentIDs() const = 0;
+
+  // Returns details about registered component.

[Sorin Jianu, 2014/05/16 16:36:09]

We need to add a comment here saying that the object returned by the function is
owned by this class. I am not very fond of this particular function but I don't
have a better idea.

[Shrikant Kelkar, 2014/05/16 18:58:56]

Done.

+  virtual CrxUpdateItem* GetComponentDetails(
+      const std::string& component_id) const = 0;
+
   // Ask the component updater to do an update check for a previously
   // registered component, immediately. If an update or check is already
   // in progress, returns |kInProgress|.
@@ -236,7 +231,6 @@ typedef ComponentUpdateService::Observer ServiceObserver;
 // the heap which the component updater will own.
 ComponentUpdateService* ComponentUpdateServiceFactory(
     ComponentUpdateService::Configurator* config);
-
 }  // namespace component_updater
 
 #endif  // CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_UPDATER_SERVICE_H_