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 4474c3d7ed7bb4315be07e9b621ccb5a50a064fc..aafdad76bb72078f26616e85c42cd7d192ac63f7 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
@@ -211,7 +202,11 @@ class ComponentUpdateService {
   virtual Status RegisterComponent(const CrxComponent& component) = 0;
 
   // Returns a list of registered components.
-  virtual void GetComponents(std::vector<CrxComponentInfo>* components) = 0;
+  virtual void GetComponentIDs(std::vector<std::string>* component_ids) = 0;

[Sorin Jianu, 2014/05/03 01:09:06]

Of course these two members would have to be declared in the private section of
the the class since only the friends of the class could call them.

[Sorin Jianu, 2014/05/03 01:09:06]

I would do this:

virtual std::vector<std::string> GetComponentIDs() const = 0;

virtual const CrxUpdateItem* GetComponentDetails(
const std::string& component_id) const = 0;

[Shrikant Kelkar, 2014/05/05 23:05:18]

Done.

[Shrikant Kelkar, 2014/05/05 23:05:18]

Done.

+
+  // Returns details about registered component.
+  virtual void GetComponentDetails(const std::string& component_id,
+                                   CrxUpdateItem** item) = 0;
 
   // Returns a network resource throttle. It means that a component will be
   // downloaded and installed before the resource is unthrottled. This is the
@@ -240,7 +235,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_