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