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

Side by Side Diff: gpu/ipc/in_process_command_buffer.cc

Issue 2814583002: Service/ClientDiscardableManager (Closed)
Patch Set: feeedback Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "gpu/ipc/in_process_command_buffer.h" 5 #include "gpu/ipc/in_process_command_buffer.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <queue> 10 #include <queue>
(...skipping 20 matching lines...) Expand all
31 #include "gpu/command_buffer/service/command_executor.h" 31 #include "gpu/command_buffer/service/command_executor.h"
32 #include "gpu/command_buffer/service/context_group.h" 32 #include "gpu/command_buffer/service/context_group.h"
33 #include "gpu/command_buffer/service/gl_context_virtual.h" 33 #include "gpu/command_buffer/service/gl_context_virtual.h"
34 #include "gpu/command_buffer/service/gpu_preferences.h" 34 #include "gpu/command_buffer/service/gpu_preferences.h"
35 #include "gpu/command_buffer/service/image_factory.h" 35 #include "gpu/command_buffer/service/image_factory.h"
36 #include "gpu/command_buffer/service/image_manager.h" 36 #include "gpu/command_buffer/service/image_manager.h"
37 #include "gpu/command_buffer/service/mailbox_manager.h" 37 #include "gpu/command_buffer/service/mailbox_manager.h"
38 #include "gpu/command_buffer/service/memory_program_cache.h" 38 #include "gpu/command_buffer/service/memory_program_cache.h"
39 #include "gpu/command_buffer/service/memory_tracking.h" 39 #include "gpu/command_buffer/service/memory_tracking.h"
40 #include "gpu/command_buffer/service/query_manager.h" 40 #include "gpu/command_buffer/service/query_manager.h"
41 #include "gpu/command_buffer/service/service_discardable_manager.h"
41 #include "gpu/command_buffer/service/service_utils.h" 42 #include "gpu/command_buffer/service/service_utils.h"
42 #include "gpu/command_buffer/service/sync_point_manager.h" 43 #include "gpu/command_buffer/service/sync_point_manager.h"
43 #include "gpu/command_buffer/service/transfer_buffer_manager.h" 44 #include "gpu/command_buffer/service/transfer_buffer_manager.h"
44 #include "gpu/config/gpu_feature_info.h" 45 #include "gpu/config/gpu_feature_info.h"
45 #include "gpu/ipc/gpu_in_process_thread_service.h" 46 #include "gpu/ipc/gpu_in_process_thread_service.h"
46 #include "gpu/ipc/service/image_transport_surface.h" 47 #include "gpu/ipc/service/image_transport_surface.h"
47 #include "ui/gfx/geometry/size.h" 48 #include "ui/gfx/geometry/size.h"
48 #include "ui/gl/gl_context.h" 49 #include "ui/gl/gl_context.h"
49 #include "ui/gl/gl_image.h" 50 #include "ui/gl/gl_image.h"
50 #include "ui/gl/gl_image_shared_memory.h" 51 #include "ui/gl/gl_image_shared_memory.h"
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 command_buffer->SetParseErrorCallback(base::Bind( 302 command_buffer->SetParseErrorCallback(base::Bind(
302 &InProcessCommandBuffer::OnContextLostOnGpuThread, gpu_thread_weak_ptr_)); 303 &InProcessCommandBuffer::OnContextLostOnGpuThread, gpu_thread_weak_ptr_));
303 304
304 gl_share_group_ = params.context_group ? params.context_group->gl_share_group_ 305 gl_share_group_ = params.context_group ? params.context_group->gl_share_group_
305 : service_->share_group(); 306 : service_->share_group();
306 307
307 bool bind_generates_resource = false; 308 bool bind_generates_resource = false;
308 scoped_refptr<gles2::FeatureInfo> feature_info = 309 scoped_refptr<gles2::FeatureInfo> feature_info =
309 new gles2::FeatureInfo(service_->gpu_driver_bug_workarounds()); 310 new gles2::FeatureInfo(service_->gpu_driver_bug_workarounds());
310 311
312 discardable_manager_.reset(new ServiceDiscardableManager());
311 context_group_ = 313 context_group_ =
312 params.context_group 314 params.context_group
313 ? params.context_group->decoder_->GetContextGroup() 315 ? params.context_group->decoder_->GetContextGroup()
314 : new gles2::ContextGroup( 316 : new gles2::ContextGroup(
315 service_->gpu_preferences(), service_->mailbox_manager(), NULL, 317 service_->gpu_preferences(), service_->mailbox_manager(), NULL,
316 service_->shader_translator_cache(), 318 service_->shader_translator_cache(),
317 service_->framebuffer_completeness_cache(), feature_info, 319 service_->framebuffer_completeness_cache(), feature_info,
318 bind_generates_resource, nullptr, nullptr, GpuFeatureInfo()); 320 bind_generates_resource, nullptr, nullptr, GpuFeatureInfo(),
321 discardable_manager_.get());
piman 2017/05/10 22:13:59 So, it could be that the context_group outlives th
ericrk 2017/05/12 16:22:12 Ah, yes, forgot about this (I think this is why I
319 322
320 decoder_.reset(gles2::GLES2Decoder::Create(context_group_.get())); 323 decoder_.reset(gles2::GLES2Decoder::Create(context_group_.get()));
321 324
322 executor_.reset(new CommandExecutor(command_buffer.get(), decoder_.get(), 325 executor_.reset(new CommandExecutor(command_buffer.get(), decoder_.get(),
323 decoder_.get())); 326 decoder_.get()));
324 command_buffer->SetGetBufferChangeCallback(base::Bind( 327 command_buffer->SetGetBufferChangeCallback(base::Bind(
325 &CommandExecutor::SetGetBuffer, base::Unretained(executor_.get()))); 328 &CommandExecutor::SetGetBuffer, base::Unretained(executor_.get())));
326 command_buffer_ = std::move(command_buffer); 329 command_buffer_ = std::move(command_buffer);
327 330
328 decoder_->set_engine(executor_.get()); 331 decoder_->set_engine(executor_.get());
(...skipping 821 matching lines...) Expand 10 before | Expand all | Expand 10 after
1150 return wrapped_callback; 1153 return wrapped_callback;
1151 } 1154 }
1152 1155
1153 InProcessCommandBuffer::GpuTask::GpuTask(const base::Closure& callback, 1156 InProcessCommandBuffer::GpuTask::GpuTask(const base::Closure& callback,
1154 uint32_t order_number) 1157 uint32_t order_number)
1155 : callback(callback), order_number(order_number) {} 1158 : callback(callback), order_number(order_number) {}
1156 1159
1157 InProcessCommandBuffer::GpuTask::~GpuTask() {} 1160 InProcessCommandBuffer::GpuTask::~GpuTask() {}
1158 1161
1159 } // namespace gpu 1162 } // namespace gpu
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698