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

Issue 350012: Purge memory for DBs in the renderer process. (Closed)

Created:
11 years, 1 month ago by Peter Kasting
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, brettw+cc_chromium.org, darin (slow to review), jam
Visibility:
Public.

Description

Purge memory for DBs in the renderer process. BUG=23400 TEST=none

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -5 lines) Patch
M chrome/renderer/render_thread.cc View 5 chunks +15 lines, -5 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
Peter Kasting
11 years, 1 month ago (2009-10-31 00:52:49 UTC) #1
darin (slow to review)
LGTM http://codereview.chromium.org/350012/diff/1/2 File chrome/renderer/render_thread.cc (right): http://codereview.chromium.org/350012/diff/1/2#newcode587 Line 587: while (sqlite3_release_memory(std::numeric_limits<int>::max()) > 0) do we compile ...
11 years, 1 month ago (2009-10-31 22:39:40 UTC) #2
Peter Kasting
11 years, 1 month ago (2009-11-01 04:16:36 UTC) #3
On 2009/10/31 22:39:40, darin wrote:
> do we compile with -DSQLITE_ENABLE_MEMORY_MANAGEMENT=1

Yes.

Powered by Google App Engine
This is Rietveld 408576698