Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(201)

Unified Diff: gears/base/common/permissions_db.cc

Issue 800: Database2Versions table, get/set version operations implemented (Closed) SVN Base: http://google-gears.googlecode.com/svn/contrib/dimitri.glazkov/database2/
Patch Set: Created 4 months, 4 weeks ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side by-side-diff with in-line comments
Download patch
Index: gears/base/common/permissions_db.cc
===================================================================
--- gears/base/common/permissions_db.cc (revision 1639)
+++ gears/base/common/permissions_db.cc (working copy)
@@ -71,7 +71,8 @@
: version_table_(&db_, kVersionTableName),
access_table_(&db_, kAccessTableName),
shortcut_table_(&db_),
- database_name_table_(&db_) {
+ database_name_table_(&db_),
+ database2_versions_table_(&db_) {
}
@@ -310,6 +311,21 @@
basename);
}
+bool PermissionsDB::GetDatabase2Version(const SecurityOrigin &origin,
Aaron 2008/05/19 15:33:50 I think we can eliminate this extra indirection by
Dimitri 2008/05/20 03:07:08 On 2008/05/19 15:33:50, Aaron wrote: > I think we
+ const std::string16 &name,
+ std::string16 *version,
+ bool *found) {
+ return database2_versions_table_.GetVersion(origin.url().c_str(), name,
+ version, found);
+}
+
+bool PermissionsDB::SetDatabase2Version(const SecurityOrigin &origin,
+ const std::string16 &name,
+ const std::string16 &version) {
+ return database2_versions_table_.SetVersion(origin.url().c_str(), name,
+ version);
+}
+
bool PermissionsDB::CreateDatabase() {
ASSERT_SINGLE_THREAD();
@@ -325,7 +341,8 @@
if (!version_table_.MaybeCreateTable() ||
!access_table_.MaybeCreateTable() ||
!shortcut_table_.MaybeCreateTableLatestVersion() ||
- !database_name_table_.MaybeCreateTableLatestVersion()) {
+ !database_name_table_.MaybeCreateTableLatestVersion() ||
+ !database2_versions_table_.MaybeCreateTableLatestVersion()) {
return false;
}

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld r338