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

Side by Side Diff: components/bookmarks_enhanced/persistent_image_store.cc

Issue 259863007: Local salient image storage for enhanced bookmark experiment. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed unneeded header include Created 6 years, 7 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/bookmarks_enhanced/persistent_image_store.h"
6
7 #include "components/bookmarks_enhanced/image_store_util.h"
8 #include "sql/statement.h"
9 #include "sql/transaction.h"
10 #include "ui/gfx/geometry/size.h"
11 #include "url/gurl.h"
12
13 namespace {
14
15 bool InitTables(sql::Connection& db) {
16 const char kTableSql[] =
17 "CREATE TABLE IF NOT EXISTS images_by_url ("
18 " page_url LONGVARCHAR NOT NULL,"
19 " image_url LONGVARCHAR NOT NULL,"
20 " image_data BLOB,"
21 " width INTEGER,"
22 " height INTEGER"
23 ")";
24 if (!db.Execute(kTableSql))
25 return false;
26 return true;
27 }
28
29 bool InitIndices(sql::Connection& db) {
30 const char kIndexSql[] =
31 "CREATE INDEX IF NOT EXISTS images_by_url_idx ON images_by_url(page_url)";
32 if (!db.Execute(kIndexSql))
33 return false;
34 return true;
35 }
36
37 sql::InitStatus OpenDatabaseImpl(sql::Connection& db,
38 const base::FilePath& db_path) {
39 DCHECK(!db.is_open());
40
41 db.set_histogram_tag("BookmarkImages");
42 // TODO(noyau): Set page and cache sizes?
43 // TODO(noyau): Set error callback?
44
45 // Run the database in exclusive mode. Nobody else should be accessing the
46 // database while we're running, and this will give somewhat improved perf.
47 db.set_exclusive_locking();
48
49 if (!db.Open(db_path))
50 return sql::INIT_FAILURE;
51
52 // Scope initialization in a transaction so we can't be partially initialized.
53 sql::Transaction transaction(&db);
54 if (!transaction.Begin())
55 return sql::INIT_FAILURE;
56
57 // Create the tables.
58 if (!InitTables(db) ||
59 !InitIndices(db)) {
60 return sql::INIT_FAILURE;
61 }
62
63 // Initialization is complete.
64 if (!transaction.Commit())
65 return sql::INIT_FAILURE;
66
67 return sql::INIT_OK;
68 }
69
70 } // namespace
71
72 PersistentImageStore::PersistentImageStore(const base::FilePath& path)
73 : ImageStore(), path_(path.Append("BookmarkImageAndUrlStore.db")) {}
74
75 bool PersistentImageStore::HasKey(const GURL& page_url) {
76 DCHECK(thread_checker_.CalledOnValidThread());
77
78 OpenDatabase();
79 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
80 "SELECT COUNT(*) FROM images_by_url WHERE page_url = ?"));
81 statement.BindString(0, page_url.possibly_invalid_spec());
82
83 int count = statement.Step() ? statement.ColumnInt(0) : 0;
84
85 return !!count;
86 }
87
88 void PersistentImageStore::Insert(const GURL& page_url,
89 const GURL& image_url,
90 const gfx::Image& image) {
91 DCHECK(thread_checker_.CalledOnValidThread());
92
93 Erase(page_url); // Remove previous image for this url, if any.
94 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
95 "INSERT INTO images_by_url "
96 " (page_url, image_url, image_data, width, height)"
97 " VALUES (?, ?, ?, ?, ?)"));
98
99 statement.BindString(0, page_url.possibly_invalid_spec());
100 statement.BindString(1, image_url.possibly_invalid_spec());
101
102 scoped_refptr<base::RefCountedMemory> image_bytes =
103 bookmarks_enhanced::BytesForImage(image);
104
105 // Insert an empty image in case encoding fails.
106 if (!image_bytes.get())
107 image_bytes = bookmarks_enhanced::BytesForImage(gfx::Image());
108
109 CHECK(image_bytes.get());
110
111 statement.BindBlob(2, image_bytes->front(), image_bytes->size());
112
113 statement.BindInt(3, image.Size().width());
114 statement.BindInt(4, image.Size().height());
115 statement.Run();
116 }
117
118 void PersistentImageStore::Erase(const GURL& page_url) {
119 DCHECK(thread_checker_.CalledOnValidThread());
120
121 OpenDatabase();
122 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
123 "DELETE FROM images_by_url WHERE page_url = ?"));
124 statement.BindString(0, page_url.possibly_invalid_spec());
125 statement.Run();
126 }
127
128 std::pair<gfx::Image, GURL> PersistentImageStore::Get(const GURL& page_url) {
129 DCHECK(thread_checker_.CalledOnValidThread());
130
131 OpenDatabase();
132 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
133 "SELECT image_data, image_url FROM images_by_url WHERE page_url = ?"));
134
135 statement.BindString(0, page_url.possibly_invalid_spec());
136
137 while (statement.Step()) {
138 if (statement.ColumnByteLength(0) > 0) {
139 scoped_refptr<base::RefCountedBytes> data(new base::RefCountedBytes());
140 statement.ColumnBlobAsVector(0, &data->data());
141
142 return std::make_pair(bookmarks_enhanced::ImageForBytes(data),
143 GURL(statement.ColumnString(1)));
144 }
145 }
146
147 return std::make_pair(gfx::Image(), GURL());
148 }
149
150 gfx::Size PersistentImageStore::GetSize(const GURL& page_url) {
151 DCHECK(thread_checker_.CalledOnValidThread());
152
153 OpenDatabase();
154 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
155 "SELECT width, height FROM images_by_url WHERE page_url = ?"));
156
157 statement.BindString(0, page_url.possibly_invalid_spec());
158
159 while (statement.Step()) {
160 if (statement.ColumnByteLength(0) > 0) {
161 int width = statement.ColumnInt(0);
162 int height = statement.ColumnInt(1);
163 return gfx::Size(width, height);
164 }
165 }
166 return gfx::Size();
167 }
168
169 void PersistentImageStore::GetAllPageUrls(std::set<GURL>* urls) {
170 DCHECK(thread_checker_.CalledOnValidThread());
171 DCHECK(urls->empty());
172
173 OpenDatabase();
174 sql::Statement statement(db_.GetCachedStatement(SQL_FROM_HERE,
175 "SELECT page_url FROM images_by_url"));
176 while (statement.Step())
177 urls->insert(GURL(statement.ColumnString(0)));
178 }
179
180 PersistentImageStore::~PersistentImageStore() {
181 DCHECK(thread_checker_.CalledOnValidThread());
182 }
183
184 void PersistentImageStore::OpenDatabase() {
185 DCHECK(thread_checker_.CalledOnValidThread());
186
187 if (db_.is_open())
188 return;
189
190 const size_t kAttempts = 2;
191
192 sql::InitStatus status = sql::INIT_FAILURE;
193 for (size_t i = 0; i < kAttempts; ++i) {
194 status = OpenDatabaseImpl(db_, path_);
195 if (status == sql::INIT_OK)
196 return;
197
198 // Can't open, raze().
199 if (db_.is_open())
200 db_.Raze();
201 db_.Close();
202 }
203 CHECK(false) << "Can't open image DB";
204 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698