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

Unified Diff: content/gpu/gpu_child_thread.cc

Issue 2322893002: gpu: Remove unused GpuChildThread::gpu_preferences_. (Closed)
Patch Set: . Created 4 years, 3 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/gpu/gpu_child_thread.h ('k') | content/gpu/in_process_gpu_thread.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/gpu/gpu_child_thread.cc
diff --git a/content/gpu/gpu_child_thread.cc b/content/gpu/gpu_child_thread.cc
index 7c80e57e40a380af95cc22e96ede0b56de1dd380..efd1a4459cbdc37beffe9d14390305f5c0ccaf55 100644
--- a/content/gpu/gpu_child_thread.cc
+++ b/content/gpu/gpu_child_thread.cc
@@ -162,7 +162,6 @@ GpuChildThread::GpuChildThread(
}
GpuChildThread::GpuChildThread(
- const gpu::GpuPreferences& gpu_preferences,
const InProcessChildThreadParams& params,
gpu::GpuMemoryBufferFactory* gpu_memory_buffer_factory)
: ChildThreadImpl(ChildThreadImpl::Options::Builder()
@@ -172,7 +171,6 @@ GpuChildThread::GpuChildThread(
gpu_memory_buffer_factory))
.ConnectToBrowser(true)
.Build()),
- gpu_preferences_(gpu_preferences),
dead_on_arrival_(false),
in_browser_process_(true),
gpu_memory_buffer_factory_(gpu_memory_buffer_factory) {
@@ -317,13 +315,10 @@ void GpuChildThread::StoreShaderToDisk(int32_t client_id,
}
void GpuChildThread::OnInitialize(const gpu::GpuPreferences& gpu_preferences) {
- gpu_preferences_ = gpu_preferences;
-
gpu_info_.video_decode_accelerator_capabilities =
- media::GpuVideoDecodeAccelerator::GetCapabilities(gpu_preferences_);
+ media::GpuVideoDecodeAccelerator::GetCapabilities(gpu_preferences);
gpu_info_.video_encode_accelerator_supported_profiles =
- media::GpuVideoEncodeAccelerator::GetSupportedProfiles(
- gpu_preferences_);
+ media::GpuVideoEncodeAccelerator::GetSupportedProfiles(gpu_preferences);
gpu_info_.jpeg_decode_accelerator_supported =
media::GpuJpegDecodeAccelerator::IsSupported();
@@ -361,12 +356,12 @@ void GpuChildThread::OnInitialize(const gpu::GpuPreferences& gpu_preferences) {
// Defer creation of the render thread. This is to prevent it from handling
// IPC messages before the sandbox has been enabled and all other necessary
// initialization has succeeded.
- gpu_channel_manager_.reset(
- new gpu::GpuChannelManager(gpu_preferences_, this, watchdog_thread_.get(),
- base::ThreadTaskRunnerHandle::Get().get(),
- ChildProcess::current()->io_task_runner(),
- ChildProcess::current()->GetShutDownEvent(),
- sync_point_manager, gpu_memory_buffer_factory_));
+ gpu_channel_manager_.reset(new gpu::GpuChannelManager(
+ gpu_preferences, this, watchdog_thread_.get(),
+ base::ThreadTaskRunnerHandle::Get().get(),
+ ChildProcess::current()->io_task_runner(),
+ ChildProcess::current()->GetShutDownEvent(), sync_point_manager,
+ gpu_memory_buffer_factory_));
media_service_.reset(new media::MediaService(gpu_channel_manager_.get()));
@@ -377,8 +372,8 @@ void GpuChildThread::OnInitialize(const gpu::GpuPreferences& gpu_preferences) {
&GpuChildThread::BindServiceFactoryRequest, base::Unretained(this)));
if (GetContentClient()->gpu()) { // NULL in tests.
- GetContentClient()->gpu()->ExposeInterfacesToBrowser(
- GetInterfaceRegistry(), gpu_preferences_);
+ GetContentClient()->gpu()->ExposeInterfacesToBrowser(GetInterfaceRegistry(),
+ gpu_preferences);
GetContentClient()->gpu()->ConsumeInterfacesFromBrowser(
GetRemoteInterfaces());
}
« no previous file with comments | « content/gpu/gpu_child_thread.h ('k') | content/gpu/in_process_gpu_thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698