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

Issue 79263003: Merge 233505 "Tear down IndexedDB dispatcher before WebKit is to..." (Closed)

Created:
7 years, 1 month ago by jsbell
Modified:
7 years, 1 month ago
Reviewers:
jsbell
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Merge 233505 "Tear down IndexedDB dispatcher before WebKit is to..." > Tear down IndexedDB dispatcher before WebKit is torn down > > Ensure callback objects held by the Indexed DB dispatcher are released > before WebKit and V8 are torn down, since the callback objects hold V8 > handles. > > This is the same patch as crrev.com/50333004 which landed as r232620 > but reverted as r232649 due to timeouts on XP - I just tweaked the > test constants slightly. > > BUG=308988 > TBR=marja@chromium.org,jamesr@chromium.org,dgrogan@chromium.org > > Review URL: https://codereview.chromium.org/58113002 TBR=jsbell@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=236351

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, --1 lines) Patch
M content/browser/indexed_db/indexed_db_browsertest.cc View 1 chunk +12 lines, -0 lines 0 comments Download
M content/renderer/render_thread_impl.cc View 1 chunk +4 lines, -0 lines 0 comments Download
A + content/test/data/indexeddb/shutdown_with_requests.html View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jsbell
7 years, 1 month ago (2013-11-20 23:54:56 UTC) #1
jsbell
7 years, 1 month ago (2013-11-20 23:55:06 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r236351.

Powered by Google App Engine
This is Rietveld 408576698