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

Issue 6384003: Gpu process restart fix. (Closed)

Created:
9 years, 11 months ago by Alexey Marinichev
Modified:
9 years, 7 months ago
Reviewers:
apatrick_chromium
CC:
chromium-reviews, darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Gpu process restart fix. If the gpu process dies, CommandBufferProxy might be the first one to find out about it. When the context is being created, GpuChannelHost still thinks there is no error. It will send CreateViewCommandBuffer message to establish a communication channel, which will fail, and GPU process will not restart. To prevent that, we set GpuChannelHost state to lost as soon as CommandBufferProxy finds out about it. BUG=58785 TEST=open a composited tab, in another tab go to about:gpucrash Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72023

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -1 line) Patch
M chrome/renderer/ggl/ggl.cc View 1 chunk +6 lines, -1 line 0 comments Download
M chrome/renderer/gpu_channel_host.h View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/renderer/gpu_channel_host.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Alexey Marinichev
9 years, 11 months ago (2011-01-20 21:17:28 UTC) #1
apatrick_chromium
9 years, 11 months ago (2011-01-20 21:38:36 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698