packages: chromium-browser/no-sqlite-debug.patch (NEW) - Revert sqlite debu...
glen
glen at pld-linux.org
Sun Sep 20 12:41:59 CEST 2009
Author: glen Date: Sun Sep 20 10:41:59 2009 GMT
Module: packages Tag: HEAD
---- Log message:
- Revert sqlite debugging to stop crashes
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/sqlite_utils.cc?r1=24321&r2=25633
---- Files affected:
packages/chromium-browser:
no-sqlite-debug.patch (NONE -> 1.1) (NEW)
---- Diffs:
================================================================
Index: packages/chromium-browser/no-sqlite-debug.patch
diff -u /dev/null packages/chromium-browser/no-sqlite-debug.patch:1.1
--- /dev/null Sun Sep 20 12:41:59 2009
+++ packages/chromium-browser/no-sqlite-debug.patch Sun Sep 20 12:41:54 2009
@@ -0,0 +1,30 @@
+diff -up chromium-20090916svn26392/src/chrome/common/sqlite_utils.cc.no-sqlite-debug chromium-20090916svn26392/src/chrome/common/sqlite_utils.cc
+--- chromium-20090916svn26392/src/chrome/common/sqlite_utils.cc.no-sqlite-debug 2009-09-16 20:15:08.000000000 -0400
++++ chromium-20090916svn26392/src/chrome/common/sqlite_utils.cc 2009-09-16 20:16:20.000000000 -0400
+@@ -42,14 +42,8 @@ class ReleaseSQLErrorHandler : public Va
+ public:
+ virtual int HandleError(int error, sqlite3* db) {
+ error_ = error;
+- // TOD(cpu): Remove this code once it has a few days of air time.
+- if (error == SQLITE_INTERNAL ||
+- error == SQLITE_NOMEM ||
+- error == SQLITE_CORRUPT ||
+- error == SQLITE_IOERR ||
+- error == SQLITE_CONSTRAINT ||
+- error == SQLITE_NOTADB)
+- CHECK(false) << "sqlite fatal error " << error;
++ // TODO(cpu): need to write to some place so we can trigger
++ // the diagnostic-repair mode.
+ return error;
+ }
+ };
+@@ -283,8 +277,7 @@ int SQLStatement::prepare(sqlite3* db, c
+ DCHECK(!stmt_);
+ int rv = sqlite3_prepare_v2(db, sql, sql_len, &stmt_, NULL);
+ if (rv != SQLITE_OK) {
+- SQLErrorHandler* error_handler = GetErrorHandlerFactory()->Make();
+- return error_handler->HandleError(rv, db_handle());
++ DLOG(ERROR) << "SQLStatement.prepare_v2 failed: " << sqlite3_errmsg(db);
+ }
+ return rv;
+ }
================================================================
More information about the pld-cvs-commit
mailing list