0

[MappableSI] Remove unused GpuMemoryBufferManager from frame_sink.

Remove unused GpuMemoryBufferManager from cc/slim/frame_sink and all
clients calling it.

Bug: 406821208
Change-Id: I6ae326cd71cf31cad3a3a3dc44f7be0613aab228
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6442665
Commit-Queue: Bo Liu <boliu@chromium.org>
Auto-Submit: vikas soni <vikassoni@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1444858}
This commit is contained in:
vikas soni
2025-04-09 11:31:11 -07:00
committed by Chromium LUCI CQ
parent 37034056af
commit eca801c040
4 changed files with 1 additions and 8 deletions

@ -21,7 +21,6 @@ std::unique_ptr<FrameSink> FrameSink::Create(
scoped_refptr<viz::RasterContextProvider> context_provider,
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
// Parameters below only used when wrapping cc.
gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
base::PlatformThreadId io_thread_id) {
return base::WrapUnique<FrameSink>(
new FrameSinkImpl(std::move(task_runner),

@ -14,10 +14,6 @@
#include "components/viz/common/gpu/raster_context_provider.h"
#include "services/viz/public/mojom/compositing/compositor_frame_sink.mojom.h"
namespace gpu {
class GpuMemoryBufferManager;
}
namespace cc::slim {
// Abstraction and ownership over connections to the GPU process:
@ -33,7 +29,6 @@ class COMPONENT_EXPORT(CC_SLIM) FrameSink {
scoped_refptr<viz::RasterContextProvider> context_provider,
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
// Parameters below only used when wrapping cc.
gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
base::PlatformThreadId io_thread_id);
virtual ~FrameSink() = default;

@ -295,7 +295,7 @@ void MagnifierSurfaceControl::CreateDisplayAndFrameSink() {
layer_tree_->SetFrameSink(cc::slim::FrameSink::Create(
std::move(sink_remote), std::move(client_receiver), nullptr,
GetUIThreadTaskRunner({BrowserTaskType::kUserInput}), nullptr,
GetUIThreadTaskRunner({BrowserTaskType::kUserInput}),
base::kInvalidThreadId));
layer_tree_->SetVisible(true);
}

@ -823,7 +823,6 @@ void CompositorImpl::InitializeVizLayerTreeFrameSink(
auto frame_sink = cc::slim::FrameSink::Create(
std::move(sink_remote), std::move(client_receiver),
std::move(context_provider), std::move(task_runner),
BrowserGpuChannelHostFactory::instance()->GetGpuMemoryBufferManager(),
BrowserMainLoop::GetInstance()->GetIOThreadId());
host_->SetFrameSink(std::move(frame_sink));
}