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

Issue 7046013: Use the DatabaseTracker only on the FILE thread. (Closed)

Created:
9 years, 7 months ago by michaeln
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Use the DatabaseTracker only on the FILE thread. TEST=none BUG=81916 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86030

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -15 lines) Patch
M chrome/browser/browsing_data_database_helper.h View 4 chunks +7 lines, -7 lines 0 comments Download
M chrome/browser/browsing_data_database_helper.cc View 3 chunks +8 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
michaeln
9 years, 7 months ago (2011-05-19 20:09:12 UTC) #1
michaeln
please take a look
9 years, 7 months ago (2011-05-19 20:18:33 UTC) #2
jochen (gone - plz use gerrit)
On 2011/05/19 20:18:33, michaeln wrote: > please take a look See http://crbug.com/74433 - I think ...
9 years, 7 months ago (2011-05-19 20:44:47 UTC) #3
michaeln
On 2011/05/19 20:44:47, jochen wrote: > On 2011/05/19 20:18:33, michaeln wrote: > > please take ...
9 years, 7 months ago (2011-05-19 20:48:49 UTC) #4
jochen (gone - plz use gerrit)
On 2011/05/19 20:48:49, michaeln wrote: > On 2011/05/19 20:44:47, jochen wrote: > > On 2011/05/19 ...
9 years, 7 months ago (2011-05-19 21:11:45 UTC) #5
michaeln
9 years, 7 months ago (2011-05-19 21:29:26 UTC) #6
> I'm preparing a CL to fix the initialization

Please take a look at the additions to webkit_thread.cc

Powered by Google App Engine
This is Rietveld 408576698