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

Side by Side Diff: chrome/browser/webdata/web_database.cc

Issue 483003: linux: remove a NOTIMPL since we have a bug on it (Closed)
Patch Set: Created 11 years 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
« no previous file with comments | « chrome/browser/password_manager/encryptor_linux.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/webdata/web_database.h" 5 #include "chrome/browser/webdata/web_database.h"
6 6
7 #include <limits> 7 #include <limits>
8 8
9 #include "app/gfx/codec/png_codec.h" 9 #include "app/gfx/codec/png_codec.h"
10 #include "app/l10n_util.h" 10 #include "app/l10n_util.h"
11 #include "app/sql/statement.h" 11 #include "app/sql/statement.h"
12 #include "app/sql/transaction.h" 12 #include "app/sql/transaction.h"
13 #include "base/utf_string_conversions.h" 13 #include "base/utf_string_conversions.h"
14 #include "chrome/browser/diagnostics/sqlite_diagnostics.h" 14 #include "chrome/browser/diagnostics/sqlite_diagnostics.h"
15 #include "chrome/browser/history/history_database.h" 15 #include "chrome/browser/history/history_database.h"
16 #include "webkit/glue/password_form.h" 16 #include "webkit/glue/password_form.h"
17 17
18 // Encryptor is the *wrong* way of doing things; we need to turn it into a 18 // Encryptor is the *wrong* way of doing things; we need to turn it into a
19 // bottleneck to use the platform methods (e.g. Keychain on the Mac, Gnome 19 // bottleneck to use the platform methods (e.g. Keychain on the Mac, Gnome
20 // Keyring / KWallet on Linux). That's going to take a massive change in its 20 // Keyring / KWallet on Linux). That's going to take a massive change in its
21 // API... see: 21 // API... see:
22 // http://code.google.com/p/chromium/issues/detail?id=8205 (Linux) 22 // http://code.google.com/p/chromium/issues/detail?id=25404 (Linux)
23 // but the (possibly-now-unused) Mac encryptor stub code needs to die too.
23 #include "chrome/browser/password_manager/encryptor.h" 24 #include "chrome/browser/password_manager/encryptor.h"
24 25
25 using webkit_glue::FormField; 26 using webkit_glue::FormField;
26 using webkit_glue::PasswordForm; 27 using webkit_glue::PasswordForm;
27 28
28 //////////////////////////////////////////////////////////////////////////////// 29 ////////////////////////////////////////////////////////////////////////////////
29 // 30 //
30 // Schema 31 // Schema
31 // 32 //
32 // keywords Most of the columns mirror that of a field in 33 // keywords Most of the columns mirror that of a field in
(...skipping 1112 matching lines...) Expand 10 before | Expand all | Expand 10 after
1145 1146
1146 // Add successive versions here. Each should set the version number and 1147 // Add successive versions here. Each should set the version number and
1147 // compatible version number as appropriate, then fall through to the next 1148 // compatible version number as appropriate, then fall through to the next
1148 // case. 1149 // case.
1149 1150
1150 case kCurrentVersionNumber: 1151 case kCurrentVersionNumber:
1151 // No migration needed. 1152 // No migration needed.
1152 return; 1153 return;
1153 } 1154 }
1154 } 1155 }
OLDNEW
« no previous file with comments | « chrome/browser/password_manager/encryptor_linux.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698