drive: Rename DriveWebApp* to DriveApp

chrome/browser/chromeos/drive/drive_app_registry.h

 // Copyright (c) 2012 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_BROWSER_CHROMEOS_DRIVE_DRIVE_WEBAPPS_REGISTRY_H_
-#define CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_WEBAPPS_REGISTRY_H_
+#ifndef CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_APP_REGISTRY_H_
+#define CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_APP_REGISTRY_H_
 
 #include <map>
 #include <set>
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/google_apis/gdata_errorcode.h"
 #include "chrome/browser/google_apis/gdata_wapi_parser.h"
 #include "googleurl/src/gurl.h"
 
 namespace base {
 class FilePath;
 }  // namespace base
 
 namespace google_apis {
 class AppList;
 }  // namespace AppList
 
 namespace drive {
 
 class JobScheduler;
 
-// Data structure that defines WebApp
-struct DriveWebAppInfo {
-  DriveWebAppInfo(const std::string& app_id,
+// Data structure that defines Drive app. See
+// https://chrome.google.com/webstore/category/collection/drive_apps for
+// Drive apps available on the webstore.
+struct DriveAppInfo {
+  DriveAppInfo(const std::string& app_id,
                   const google_apis::InstalledApp::IconList& app_icons,

[hashimoto, 2013/05/28 03:59:47]

nit: Indent

[satorux1, 2013/05/28 04:05:16]

Done.

                   const google_apis::InstalledApp::IconList& document_icons,
                   const std::string& web_store_id,
                   const string16& app_name,
                   const string16& object_type,
                   bool is_primary_selector);
-  ~DriveWebAppInfo();
+  ~DriveAppInfo();
 
-  // Drive app id
+  // Drive app id.
   std::string app_id;
   // Drive application icon URLs for this app, paired with their size (length of
   // a side in pixels).
   google_apis::InstalledApp::IconList app_icons;
   // Drive document icon URLs for this app, paired with their size (length of
   // a side in pixels).
   google_apis::InstalledApp::IconList document_icons;
   // Web store id/extension id;
   std::string web_store_id;
-  // WebApp name.
+  // App name.
   string16 app_name;
   // Object (file) type description handled by this app.
   string16 object_type;
   // Is app the primary selector for file (default open action).
   bool is_primary_selector;
 };
 
-// Keeps the track of installed drive web application and provider in-memory.
-class DriveWebAppsRegistry {
+// Keeps the track of installed drive applications in-memory.
+class DriveAppRegistry {
  public:
-  explicit DriveWebAppsRegistry(JobScheduler* scheduler);
-  ~DriveWebAppsRegistry();
+  explicit DriveAppRegistry(JobScheduler* scheduler);
+  ~DriveAppRegistry();
 
   // Returns a list of web app information for the |file| with |mime_type|.
-  void GetWebAppsForFile(const base::FilePath& file,
-                         const std::string& mime_type,
-                         ScopedVector<DriveWebAppInfo>* apps);
+  void GetAppsForFile(const base::FilePath& file_path,
+                      const std::string& mime_type,
+                      ScopedVector<DriveAppInfo>* apps);
 
   // Updates this registry by fetching the data from the server.
   void Update();
 
  private:
-  // Defines WebApp application details that are associated with a given
+  // Defines application details that are associated with a given
   // file extension or content mimetype.
-  struct WebAppFileSelector {
-    WebAppFileSelector(
+  struct DriveAppFileSelector {
+    DriveAppFileSelector(
         const GURL& product_link,
         const google_apis::InstalledApp::IconList& app_icons,
         const google_apis::InstalledApp::IconList& document_icons,
         const string16& object_type,
         const std::string& app_id,
         bool is_primary_selector);
-    ~WebAppFileSelector();
-    // WebApp product link.
+    ~DriveAppFileSelector();
+    // Product link to the webstore.
     GURL product_link;
     // Drive application icon URLs for this app, paired with their size (length
     // of a side in pixels).
     google_apis::InstalledApp::IconList app_icons;
     // Drive document icon URLs for this app, paired with their size (length of
     // a side in pixels).
     google_apis::InstalledApp::IconList document_icons;
     // Object (file) type description.
     string16 object_type;
     // Drive app id
     std::string app_id;
     // True if the selector is the default one. The default selector should
     // trigger on file double-click events. Non-default selectors only show up
     // in "Open with..." pop-up menu.
     bool is_primary_selector;
   };
 
   // Defines mapping between file content type selectors (extensions, MIME
-  // types) and corresponding WebApp.
-  typedef std::multimap<std::string, WebAppFileSelector*> WebAppFileSelectorMap;
+  // types) and corresponding app.
+  typedef std::multimap<std::string,
+                        DriveAppFileSelector*> DriveAppFileSelectorMap;
 
   // Helper map used for deduplication of selector matching results.
-  typedef std::map<const WebAppFileSelector*,
-                   DriveWebAppInfo*> SelectorWebAppList;
+  typedef std::map<const DriveAppFileSelector*,
+                   DriveAppInfo*> SelectorAppList;
 
   // Part of Update(). Runs upon the completion of fetching the web apps
   // data from the server.
   void UpdateAfterGetAppList(google_apis::GDataErrorCode gdata_error,
                              scoped_ptr<google_apis::AppList> app_list);
 
   // Helper function for loading web application file |selectors| into
   // corresponding |map|.
   static void AddAppSelectorList(
       const GURL& product_link,
       const google_apis::InstalledApp::IconList& app_icons,
       const google_apis::InstalledApp::IconList& document_icons,
       const std::string& object_type,
       const std::string& app_id,
       bool is_primary_selector,
       const ScopedVector<std::string>& selectors,
-      WebAppFileSelectorMap* map);
+      DriveAppFileSelectorMap* map);
 
   // Finds matching |apps| from |map| based on provided file |selector|.
-  void FindWebAppsForSelector(const std::string& selector,
-                              const WebAppFileSelectorMap& map,
-                              SelectorWebAppList* apps);
+  void FindAppsForSelector(const std::string& selector,
+                           const DriveAppFileSelectorMap& map,
+                           SelectorAppList* apps);
 
   JobScheduler* scheduler_;
 
   // Map of web store product URL to application name.
   std::map<GURL, std::string> url_to_name_map_;
 
   // Map of filename extension to application info.
-  WebAppFileSelectorMap webapp_extension_map_;
+  DriveAppFileSelectorMap app_extension_map_;
 
   // Map of MIME type to application info.
-  WebAppFileSelectorMap webapp_mimetypes_map_;
+  DriveAppFileSelectorMap app_mimetypes_map_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate the weak pointers before any other members are destroyed.
-  base::WeakPtrFactory<DriveWebAppsRegistry> weak_ptr_factory_;
-  DISALLOW_COPY_AND_ASSIGN(DriveWebAppsRegistry);
+  base::WeakPtrFactory<DriveAppRegistry> weak_ptr_factory_;
+  DISALLOW_COPY_AND_ASSIGN(DriveAppRegistry);
 };
 
 }  // namespace drive
 
-#endif  // CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_WEBAPPS_REGISTRY_H_
+#endif  // CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_APP_REGISTRY_H_