Update comments in InstallableManager.

chrome/browser/installable/installable_manager.h

Index: chrome/browser/installable/installable_manager.h
diff --git a/chrome/browser/installable/installable_manager.h b/chrome/browser/installable/installable_manager.h
index 92ac41eb5d66afa7e45beab6257fd72bae7dc22a..8cec9c20f60df2a3d1e2a5d90d64f3de62444096 100644
--- a/chrome/browser/installable/installable_manager.h
+++ b/chrome/browser/installable/installable_manager.h
@@ -157,11 +157,13 @@ class InstallableManager
   void OnDidGetManifest(const GURL& manifest_url,
                         const content::Manifest& manifest);
 
+  // Methods checking for InstallableParams-independent basic requirements.

[dominickn, 2016/10/27 00:26:36]

This comment isn't accurate. CheckServiceWorker() isn't a basic requirement,
it's only run if InstallableParams.check_installable is true.

I don't think either of the new comments in this header file are really
necessary. Are the method names not descriptive enough?

[F, 2016/10/27 17:49:24]

Thanks for pointing out the check_installable bit! I was trying to further
distinguish these two groups of functions because CheckAndFetchBestIcon()
actually uses InstallableParams inside function body like a function parameter
while CheckInstallable() does not.

   void CheckInstallable();
   bool IsManifestValidForWebApp(const content::Manifest& manifest);
   void CheckServiceWorker();
   void OnDidCheckHasServiceWorker(bool has_service_worker);
 
+  // Methods checking for requirements specified by InstallableParams.
   void CheckAndFetchBestIcon();
   void OnAppIconFetched(const GURL icon_url, const SkBitmap& bitmap);