Get web application list using Drive V2 API.

chrome/browser/chromeos/gdata/drive_webapps_registry.cc

 // 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.
 
 #include "chrome/browser/chromeos/gdata/drive_webapps_registry.h"
 
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
+#include "chrome/browser/chromeos/gdata/drive_api_parser.h"
 #include "content/public/browser/browser_thread.h"
 
 using content::BrowserThread;
 
 namespace gdata {
 
 namespace {
 
 // WebApp store URL prefix.
 const char kStoreProductUrl[] = "https://chrome.google.com/webstore/";
 
 // Extracts Web store id from its web store URL.
 std::string GetWebStoreIdFromUrl(const GURL& url) {
   if (!StartsWithASCII(url.spec(), kStoreProductUrl, false)) {
     LOG(WARNING) << "Unrecognized product URL " << url.spec();
     return std::string();
   }
 
   FilePath path(url.path());
   std::vector<FilePath::StringType> components;
   path.GetComponents(&components);
   DCHECK_LE(2U, components.size());  // Coming from kStoreProductUrl
 
   // Return the last part of the path
   return components[components.size() - 1];
 }
 
+// TODO(kochi): This is duplicate from gdata_wapi_parser.cc.
+bool SortBySize(const InstalledApp::IconList::value_type& a,
+                const InstalledApp::IconList::value_type& b) {
+  return a.first < b.first;
+}
+
 }  // namespace
 
 // DriveWebAppInfo struct implementation.
 
 DriveWebAppInfo::DriveWebAppInfo(const std::string& app_id,
                                  const InstalledApp::IconList& app_icons,
                                  const InstalledApp::IconList& document_icons,
                                  const std::string& web_store_id,
                                  const string16& app_name,
                                  const string16& object_type,
@@ skipping 73 matching lines @@
   // web_store_id.
   for (WebAppFileSelectorMap::iterator iter = webapp_extension_map_.begin();
       iter != webapp_extension_map_.end(); ++iter) {
     std::string id = GetWebStoreIdFromUrl(iter->second->product_link);
     if (id == web_store_id)
       extensions.insert(iter->first);
   }
   return extensions;
 }
 
-void DriveWebAppsRegistry::UpdateFromFeed(AccountMetadataFeed* metadata) {
+void DriveWebAppsRegistry::UpdateFromFeed(
+    const AccountMetadataFeed* metadata) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   url_to_name_map_.clear();
   STLDeleteValues(&webapp_extension_map_);
   STLDeleteValues(&webapp_mimetypes_map_);
   for (ScopedVector<InstalledApp>::const_iterator it =
            metadata->installed_apps().begin();
        it != metadata->installed_apps().end(); ++it) {
     const InstalledApp* app = *it;
     GURL product_url = app->GetProductUrl();
@@ skipping 35 matching lines @@
                        app_icons,
                        document_icons,
                        app->object_type(),
                        app->app_id(),
                        false,   // primary
                        app->secondary_extensions(),
                        &webapp_extension_map_);
   }
 }
 
+void DriveWebAppsRegistry::UpdateFromApplicationList(const AppList* applist) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
+  url_to_name_map_.clear();
+  STLDeleteValues(&webapp_extension_map_);
+  STLDeleteValues(&webapp_mimetypes_map_);
+  for (ScopedVector<AppResource>::const_iterator it = applist->items().begin();
+       it != applist->items().end(); ++it) {

[satorux1, 2012/08/10 17:26:02]

matter of taste, but the following is more concise. 

  for (size_t i = 0; i <  applist->items().size(); ++i)

[kochi, 2012/08/13 09:12:24]

Done.

The same is done for UpdateFeed().

+    const AppResource* app = *it;

[satorux1, 2012/08/10 17:26:02]

maybe  const AppResource& app = *applist->items()[i];

[kochi, 2012/08/13 09:12:24]

Done.

The same is done for UpdateFeed().

+    if (app->product_url().is_empty())
+      continue;
+
+    InstalledApp::IconList app_icons;
+    InstalledApp::IconList document_icons;
+    for (ScopedVector<DriveAppIcon>::const_iterator it = app->icons().begin();

[satorux1, 2012/08/10 17:26:02]

|it| is already used in the outer loop. please use a different name. I'd go with
size_t i and size_t j. 

[kochi, 2012/08/13 09:12:24]

This |it| removed in favor of size_t loop.

+         it != app->icons().end(); ++it) {
+      DriveAppIcon* icon = *it;

[satorux1, 2012/08/10 17:26:02]

const reference?

[kochi, 2012/08/13 09:12:24]

Done.

+      if (icon->icon_url().is_empty())
+        continue;
+      if (icon->category() == DriveAppIcon::APPLICATION)
+        app_icons.push_back(std::make_pair(icon->icon_side_length(),
+                                           icon->icon_url()));
+      if (icon->category() == DriveAppIcon::DOCUMENT)
+        document_icons.push_back(std::make_pair(icon->icon_side_length(),
+                                                icon->icon_url()));
+    }
+    std::sort(app_icons.begin(), app_icons.end(), SortBySize);
+    std::sort(document_icons.begin(), document_icons.end(), SortBySize);
+
+    url_to_name_map_.insert(
+        std::make_pair(app->product_url(), UTF8ToUTF16(app->name())));
+    AddAppSelectorList(app->product_url(),
+                       app_icons,
+                       document_icons,
+                       UTF8ToUTF16(app->object_type()),
+                       app->application_id(),
+                       true,   // primary
+                       app->primary_mimetypes(),
+                       &webapp_mimetypes_map_);
+    AddAppSelectorList(app->product_url(),
+                       app_icons,
+                       document_icons,
+                       UTF8ToUTF16(app->object_type()),
+                       app->application_id(),
+                       false,   // primary
+                       app->secondary_mimetypes(),
+                       &webapp_mimetypes_map_);
+    AddAppSelectorList(app->product_url(),
+                       app_icons,
+                       document_icons,
+                       UTF8ToUTF16(app->object_type()),
+                       app->application_id(),
+                       true,   // primary
+                       app->primary_file_extensions(),
+                       &webapp_extension_map_);
+    AddAppSelectorList(app->product_url(),
+                       app_icons,
+                       document_icons,
+                       UTF8ToUTF16(app->object_type()),
+                       app->application_id(),
+                       false,   // primary
+                       app->secondary_file_extensions(),
+                       &webapp_extension_map_);
+  }
+}
+
 // static.
 void DriveWebAppsRegistry::AddAppSelectorList(
     const GURL& product_link,
     const InstalledApp::IconList& app_icons,
     const InstalledApp::IconList& document_icons,
     const string16& object_type,
     const std::string& app_id,
     bool is_primary_selector,
     const ScopedVector<std::string>& selectors,
     WebAppFileSelectorMap* map) {
@@ skipping 37 matching lines @@
                             web_app->app_icons,
                             web_app->document_icons,
                             web_store_id,
                             product_iter->second,   // app name.
                             web_app->object_type,
                             web_app->is_primary_selector)));
   }
 }
 
 }  // namespace gdata