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

Issue 8098017: Make content_shell handle renderers going away. I've made TabContents call TabContentsView::OnTab... (Closed)

Created:
9 years, 2 months ago by jam
Modified:
9 years, 2 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, Avi (use Gerrit), dpranke+watch-content_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Make content_shell handle renderers going away. I've made TabContents call TabContentsView::OnTabCrashed, instead of before where the embedder called that function. BUG=90445 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=103655

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -13 lines) Patch
M chrome/browser/tab_contents/tab_contents_view_gtk.cc View 2 chunks +7 lines, -0 lines 0 comments Download
M chrome/browser/tab_contents/tab_contents_view_mac.mm View 2 chunks +7 lines, -0 lines 0 comments Download
M chrome/browser/ui/tab_contents/tab_contents_wrapper.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/tab_contents/tab_contents_wrapper.cc View 2 chunks +0 lines, -12 lines 0 comments Download
M chrome/browser/ui/views/tab_contents/tab_contents_view_views.cc View 2 chunks +7 lines, -0 lines 0 comments Download
M content/browser/tab_contents/tab_contents.cc View 1 chunk +1 line, -0 lines 0 comments Download
M content/browser/tab_contents/tab_contents_view_win.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jam
9 years, 2 months ago (2011-10-01 00:22:11 UTC) #1
Dirk Pranke
9 years, 2 months ago (2011-10-01 01:56:37 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698