Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4647)

Unified Diff: chrome/browser/chromeos/drive/drive_app_registry.h

Issue 125163004: Clean up DriveAppRegistry (part 1 of 2). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove unused helper function. Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_app_registry.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/drive/drive_app_registry.h
diff --git a/chrome/browser/chromeos/drive/drive_app_registry.h b/chrome/browser/chromeos/drive/drive_app_registry.h
index 30d0945ebcedaeb9c2b57067a10fc4470098e801..52efd41a638d55459d667a00063b405fbc40faef 100644
--- a/chrome/browser/chromeos/drive/drive_app_registry.h
+++ b/chrome/browser/chromeos/drive/drive_app_registry.h
@@ -33,10 +33,7 @@ struct DriveAppInfo {
DriveAppInfo(const std::string& app_id,
const google_apis::InstalledApp::IconList& app_icons,
const google_apis::InstalledApp::IconList& document_icons,
- const std::string& web_store_id,
const std::string& app_name,
- const std::string& object_type,
- bool is_primary_selector,
const GURL& create_url);
~DriveAppInfo();
@@ -48,14 +45,8 @@ struct DriveAppInfo {
// 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;
// App name.
std::string app_name;
- // Object (file) type description handled by this app.
- std::string object_type;
- // Is app the primary selector for file (default open action).
- bool is_primary_selector;
// URL for opening a new file in the app.
GURL create_url;
};
@@ -92,13 +83,10 @@ class DriveAppRegistry {
// Helper function for loading Drive application file |selectors| into
// corresponding |map|.
static void AddAppSelectorList(
- const std::string& web_store_id,
const std::string& app_name,
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 GURL& create_url,
const ScopedVector<std::string>& selectors,
DriveAppFileSelectorMap* map);
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_app_registry.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698