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

Unified Diff: content/browser/gpu/gpu_client.cc

Issue 2805623002: gpu: Notify callbacks the reason for channel creation failure (Closed)
Patch Set: . Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/gpu/gpu_client.h ('k') | content/browser/gpu/gpu_process_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/gpu/gpu_client.cc
diff --git a/content/browser/gpu/gpu_client.cc b/content/browser/gpu/gpu_client.cc
index 3faea935db116b06c4a03c4c9c791d9618dc8db5..9fe3e051cea95e9987bd998ee0c850fccf87880d 100644
--- a/content/browser/gpu/gpu_client.cc
+++ b/content/browser/gpu/gpu_client.cc
@@ -41,7 +41,8 @@ void GpuClient::OnError() {
void GpuClient::OnEstablishGpuChannel(
const EstablishGpuChannelCallback& callback,
const IPC::ChannelHandle& channel,
- const gpu::GPUInfo& gpu_info) {
+ const gpu::GPUInfo& gpu_info,
+ GpuProcessHost::EstablishChannelStatus status) {
mojo::ScopedMessagePipeHandle channel_handle;
channel_handle.reset(channel.mojo_handle);
callback.Run(render_process_id_, std::move(channel_handle), gpu_info);
@@ -58,7 +59,9 @@ void GpuClient::EstablishGpuChannel(
GpuProcessHost* host =
GpuProcessHost::Get(GpuProcessHost::GPU_PROCESS_KIND_SANDBOXED);
if (!host) {
- OnEstablishGpuChannel(callback, IPC::ChannelHandle(), gpu::GPUInfo());
+ OnEstablishGpuChannel(
+ callback, IPC::ChannelHandle(), gpu::GPUInfo(),
+ GpuProcessHost::EstablishChannelStatus::GPU_ACCESS_DENIED);
return;
}
« no previous file with comments | « content/browser/gpu/gpu_client.h ('k') | content/browser/gpu/gpu_process_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698