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

Unified Diff: chrome/browser/webdata/web_intents_table.h

Issue 12225076: Delete most web intents code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 10 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 | « chrome/browser/webdata/web_database.cc ('k') | chrome/browser/webdata/web_intents_table.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/webdata/web_intents_table.h
diff --git a/chrome/browser/webdata/web_intents_table.h b/chrome/browser/webdata/web_intents_table.h
index aa6f97c5a559c119d7c4a8613af2a6090502aa8b..844de554c7a1d0470620b4d8ab44ff269e612d27 100644
--- a/chrome/browser/webdata/web_intents_table.h
+++ b/chrome/browser/webdata/web_intents_table.h
@@ -11,7 +11,6 @@
#include "base/compiler_specific.h"
#include "base/string16.h"
#include "chrome/browser/webdata/web_database_table.h"
-#include "webkit/glue/web_intent_service_data.h"
namespace sql {
class Connection;
@@ -20,6 +19,9 @@ class MetaTable;
struct DefaultWebIntentService;
+// TODO(thakis): Delete this class once there's a migration that drops the
+// table backing it.
+
// This class manages the WebIntents table within the SQLite database passed
// to the constructor. It expects the following schema:
//
@@ -58,57 +60,6 @@ class WebIntentsTable : public WebDatabaseTable {
// Adds "scheme" column to the web_intents and web_intents_defaults tables.
bool MigrateToVersion46AddSchemeColumn();
-#if (ENABLE_WEB_INTENTS)
- // Adds a web intent service to the WebIntents table.
- // If |service| already exists, replaces it.
- bool SetWebIntentService(const webkit_glue::WebIntentServiceData& service);
-
- // Retrieve all |services| from WebIntents table that match |action|.
- bool GetWebIntentServicesForAction(
- const string16& action,
- std::vector<webkit_glue::WebIntentServiceData>* services);
-
- // Retrieve all |services| from WebIntents table that match |scheme|.
- bool GetWebIntentServicesForScheme(
- const string16& scheme,
- std::vector<webkit_glue::WebIntentServiceData>* services);
-
- // Retrieves all |services| from WebIntents table that match |service_url|.
- bool GetWebIntentServicesForURL(
- const string16& service_url,
- std::vector<webkit_glue::WebIntentServiceData>* services);
-
- // Retrieve all |services| from WebIntents table.
- bool GetAllWebIntentServices(
- std::vector<webkit_glue::WebIntentServiceData>* services);
-
- // Removes |service| from WebIntents table - must match all parameters
- // exactly.
- bool RemoveWebIntentService(const webkit_glue::WebIntentServiceData& service);
-
- // Get the default service to be used for the given intent invocation.
- // If any overlapping defaults are found, they're placed in
- // |default_services|, otherwise, it is untouched.
- // Returns true if the method runs successfully, false on database error.
- bool GetDefaultServices(
- const string16& action,
- std::vector<DefaultWebIntentService>* default_services);
-
- // Get a list of all installed default services.
- bool GetAllDefaultServices(
- std::vector<DefaultWebIntentService>* default_services);
-
- // Set a default service to be used on given intent invocations.
- bool SetDefaultService(const DefaultWebIntentService& default_service);
-
- // Removes a default |service| from table - must match the action, type,
- // and url_pattern parameters exactly.
- bool RemoveDefaultService(const DefaultWebIntentService& default_service);
-
- // Removes all default services associated with |service_url|.
- bool RemoveServiceDefaults(const GURL& service_url);
-#endif
-
private:
DISALLOW_COPY_AND_ASSIGN(WebIntentsTable);
};
« no previous file with comments | « chrome/browser/webdata/web_database.cc ('k') | chrome/browser/webdata/web_intents_table.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698