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

Unified Diff: chrome/browser/api/webdata/web_data_service_base.h

Issue 12957002: wd4-ui for try only (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: wd4-ui_with_windows_fix Created 7 years, 9 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
Index: chrome/browser/api/webdata/web_data_service_base.h
diff --git a/chrome/browser/api/webdata/web_data_service_base.h b/chrome/browser/api/webdata/web_data_service_base.h
index d0b740caaa9403099af97cabae8e8b90d0360fc9..dc30fc123559bddf8d1a042a7cda0500b0ecfcac 100644
--- a/chrome/browser/api/webdata/web_data_service_base.h
+++ b/chrome/browser/api/webdata/web_data_service_base.h
@@ -5,23 +5,90 @@
#ifndef CHROME_BROWSER_API_WEBDATA_WEB_DATA_SERVICE_BASE_H_
#define CHROME_BROWSER_API_WEBDATA_WEB_DATA_SERVICE_BASE_H_
+#include "base/callback_forward.h"
+#include "base/files/file_path.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_source.h"
+#include "sql/init_status.h"
+
+class WebDatabase;
+class WebDatabaseService;
+
+namespace base {
+class Thread;
+}
// Base for WebDataService class hierarchy.
-class WebDataServiceBase {
+class WebDataServiceBase
+ : public base::RefCountedThreadSafe<WebDataServiceBase,
+ content::BrowserThread::DeleteOnUIThread> {
public:
// All requests return an opaque handle of the following type.
typedef int Handle;
- virtual ~WebDataServiceBase() {}
+ // Users of this class may provide a callback to handle errors
+ // (e.g. by showing a UI). The callback is called only on error, and
+ // takes a single parameter, the sql::InitStatus value from trying
+ // to open the database.
+ // TODO(joi): Should we combine this with WebDatabaseService::InitCallback?
+ typedef base::Callback<void(sql::InitStatus)> ProfileErrorCallback;
+
+ // |callback| will only be invoked on error, and only if
+ // |callback.is_null()| evaluates to false.
+ explicit WebDataServiceBase(const ProfileErrorCallback& callback);
// Cancel any pending request. You need to call this method if your
// WebDataServiceConsumer is about to be deleted.
- virtual void CancelRequest(Handle h) = 0;
+ virtual void CancelRequest(Handle h);
// Returns the notification source for this service. This may use a
// pointer other than this object's |this| pointer.
- virtual content::NotificationSource GetNotificationSource() = 0;
+ virtual content::NotificationSource GetNotificationSource();
+
+ // Shutdown the web data service. The service can no longer be used after this
+ // call.
+ virtual void ShutdownOnUIThread();
+
+ // Initializes the web data service.
+ virtual void Init(const base::FilePath& path);
+
+ // Unloads the database without actually shutting down the service. This can
+ // be used to temporarily reduce the browser process' memory footprint.
+ void UnloadDatabase();
+
+ // Unloads the database permanently and shuts down service.
+ void ShutdownDatabase();
+
+ // Returns true if the database load has completetd successfully, and
+ // ShutdownOnUIThread has not yet been called.
+ virtual bool IsDatabaseLoaded();
+
+ // Returns a pointer to the DB (used by SyncableServices). May return NULL if
+ // the database is not loaded or otherwise unavailable. Must be called on
+ // DBThread.
+ virtual WebDatabase* GetDatabase();
+
+ protected:
+ virtual ~WebDataServiceBase();
+
+ // Our database service.
+ scoped_ptr<WebDatabaseService> wdbs_;
+
+ // True if we've received a notification that the WebDatabase has loaded.
+ bool db_loaded_;
+
+ private:
+ friend struct content::BrowserThread::DeleteOnThread<
+ content::BrowserThread::UI>;
+ friend class base::DeleteHelper<WebDataServiceBase>;
+
+ ProfileErrorCallback profile_error_callback_;
+
+ void DBInitFailed(sql::InitStatus sql_status);
+ void NotifyDatabaseLoadedOnUIThread();
+ void DatabaseInitOnDB(sql::InitStatus status);
};
#endif // CHROME_BROWSER_API_WEBDATA_WEB_DATA_SERVICE_BASE_H_
« no previous file with comments | « chrome/browser/api/webdata/autofill_web_data_service.h ('k') | chrome/browser/password_manager/password_store_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698