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

Unified Diff: chrome/browser/webdata/web_apps_table.cc

Issue 382703002: Remove unused SQLLite tables from WebData (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase 2 Created 6 years, 5 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_apps_table.h ('k') | chrome/browser/webdata/web_apps_table_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/webdata/web_apps_table.cc
diff --git a/chrome/browser/webdata/web_apps_table.cc b/chrome/browser/webdata/web_apps_table.cc
deleted file mode 100644
index c30562e02e99ec6c6cf763e730634f9536b3f01a..0000000000000000000000000000000000000000
--- a/chrome/browser/webdata/web_apps_table.cc
+++ /dev/null
@@ -1,147 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/webdata/web_apps_table.h"
-
-#include "base/logging.h"
-#include "chrome/browser/history/history_database.h"
-#include "components/webdata/common/web_database.h"
-#include "sql/statement.h"
-#include "third_party/skia/include/core/SkBitmap.h"
-#include "ui/gfx/codec/png_codec.h"
-#include "url/gurl.h"
-
-namespace {
-
-WebDatabaseTable::TypeKey GetKey() {
- // We just need a unique constant. Use the address of a static that
- // COMDAT folding won't touch in an optimizing linker.
- static int table_key = 0;
- return reinterpret_cast<void*>(&table_key);
-}
-
-} // namespace
-
-WebAppsTable* WebAppsTable::FromWebDatabase(WebDatabase* db) {
- return static_cast<WebAppsTable*>(db->GetTable(GetKey()));
-}
-
-WebDatabaseTable::TypeKey WebAppsTable::GetTypeKey() const {
- return GetKey();
-}
-
-bool WebAppsTable::CreateTablesIfNecessary() {
- return (InitWebAppIconsTable() && InitWebAppsTable());
-}
-
-bool WebAppsTable::IsSyncable() {
- return true;
-}
-
-bool WebAppsTable::MigrateToVersion(int version,
- bool* update_compatible_version) {
- return true;
-}
-
-bool WebAppsTable::InitWebAppIconsTable() {
- if (!db_->DoesTableExist("web_app_icons")) {
- if (!db_->Execute("CREATE TABLE web_app_icons ("
- "url LONGVARCHAR,"
- "width int,"
- "height int,"
- "image BLOB, UNIQUE (url, width, height))")) {
- NOTREACHED();
- return false;
- }
- }
- return true;
-}
-
-bool WebAppsTable::InitWebAppsTable() {
- if (!db_->DoesTableExist("web_apps")) {
- if (!db_->Execute("CREATE TABLE web_apps ("
- "url LONGVARCHAR UNIQUE,"
- "has_all_images INTEGER NOT NULL)")) {
- NOTREACHED();
- return false;
- }
- if (!db_->Execute("CREATE INDEX web_apps_url_index ON web_apps (url)")) {
- NOTREACHED();
- return false;
- }
- }
- return true;
-}
-
-bool WebAppsTable::SetWebAppImage(const GURL& url, const SkBitmap& image) {
- // Don't bother with a cached statement since this will be a relatively
- // infrequent operation.
- sql::Statement s(db_->GetUniqueStatement(
- "INSERT OR REPLACE INTO web_app_icons "
- "(url, width, height, image) VALUES (?, ?, ?, ?)"));
- std::vector<unsigned char> image_data;
- gfx::PNGCodec::EncodeBGRASkBitmap(image, false, &image_data);
- s.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
- s.BindInt(1, image.width());
- s.BindInt(2, image.height());
- s.BindBlob(3, &image_data.front(), static_cast<int>(image_data.size()));
-
- return s.Run();
-}
-
-bool WebAppsTable::GetWebAppImages(const GURL& url,
- std::vector<SkBitmap>* images) {
- sql::Statement s(db_->GetUniqueStatement(
- "SELECT image FROM web_app_icons WHERE url=?"));
- s.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
-
- while (s.Step()) {
- SkBitmap image;
- int col_bytes = s.ColumnByteLength(0);
- if (col_bytes > 0) {
- if (gfx::PNGCodec::Decode(
- reinterpret_cast<const unsigned char*>(s.ColumnBlob(0)),
- col_bytes, &image)) {
- images->push_back(image);
- } else {
- // Should only have valid image data in the db.
- NOTREACHED();
- }
- }
- }
- return s.Succeeded();
-}
-
-bool WebAppsTable::SetWebAppHasAllImages(const GURL& url,
- bool has_all_images) {
- sql::Statement s(db_->GetUniqueStatement(
- "INSERT OR REPLACE INTO web_apps (url, has_all_images) VALUES (?, ?)"));
- s.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
- s.BindInt(1, has_all_images ? 1 : 0);
-
- return s.Run();
-}
-
-bool WebAppsTable::GetWebAppHasAllImages(const GURL& url) {
- sql::Statement s(db_->GetUniqueStatement(
- "SELECT has_all_images FROM web_apps WHERE url=?"));
- s.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
-
- return (s.Step() && s.ColumnInt(0) == 1);
-}
-
-bool WebAppsTable::RemoveWebApp(const GURL& url) {
- sql::Statement delete_s(db_->GetUniqueStatement(
- "DELETE FROM web_app_icons WHERE url = ?"));
- delete_s.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
-
- if (!delete_s.Run())
- return false;
-
- sql::Statement delete_s2(db_->GetUniqueStatement(
- "DELETE FROM web_apps WHERE url = ?"));
- delete_s2.BindString(0, history::HistoryDatabase::GURLToDatabaseURL(url));
-
- return delete_s2.Run();
-}
« no previous file with comments | « chrome/browser/webdata/web_apps_table.h ('k') | chrome/browser/webdata/web_apps_table_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698