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

Unified Diff: ui/ozone/platform/drm/gpu/gbm_surface_factory.cc

Issue 2393913002: Rename GbmSurfaceFactory::drm_thread_ to GbmSurfaceFactory::drm_thread_proxy_ (Closed)
Patch Set: Created 4 years, 2 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 | « ui/ozone/platform/drm/gpu/gbm_surface_factory.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/ozone/platform/drm/gpu/gbm_surface_factory.cc
diff --git a/ui/ozone/platform/drm/gpu/gbm_surface_factory.cc b/ui/ozone/platform/drm/gpu/gbm_surface_factory.cc
index 742733b1959e20d6d6fa17ca68d7bd4adfc2829e..e45a2542e2a9aecef01f7f7027eac76bfbfb96cc 100644
--- a/ui/ozone/platform/drm/gpu/gbm_surface_factory.cc
+++ b/ui/ozone/platform/drm/gpu/gbm_surface_factory.cc
@@ -33,20 +33,24 @@ namespace {
class GLOzoneEGLGbm : public GLOzoneEGL {
public:
- GLOzoneEGLGbm(GbmSurfaceFactory* surface_factory, DrmThreadProxy* drm_thread)
- : surface_factory_(surface_factory), drm_thread_(drm_thread) {}
+ GLOzoneEGLGbm(GbmSurfaceFactory* surface_factory,
+ DrmThreadProxy* drm_thread_proxy)
+ : surface_factory_(surface_factory),
+ drm_thread_proxy_(drm_thread_proxy) {}
~GLOzoneEGLGbm() override {}
scoped_refptr<gl::GLSurface> CreateViewGLSurface(
gfx::AcceleratedWidget window) override {
return gl::InitializeGLSurface(new GbmSurface(
- surface_factory_, drm_thread_->CreateDrmWindowProxy(window), window));
+ surface_factory_, drm_thread_proxy_->CreateDrmWindowProxy(window),
+ window));
}
scoped_refptr<gl::GLSurface> CreateSurfacelessViewGLSurface(
gfx::AcceleratedWidget window) override {
return gl::InitializeGLSurface(new GbmSurfaceless(
- surface_factory_, drm_thread_->CreateDrmWindowProxy(window), window));
+ surface_factory_, drm_thread_proxy_->CreateDrmWindowProxy(window),
+ window));
}
scoped_refptr<gl::GLSurface> CreateOffscreenGLSurface(
@@ -63,16 +67,16 @@ class GLOzoneEGLGbm : public GLOzoneEGL {
private:
GbmSurfaceFactory* surface_factory_;
- DrmThreadProxy* drm_thread_;
+ DrmThreadProxy* drm_thread_proxy_;
DISALLOW_COPY_AND_ASSIGN(GLOzoneEGLGbm);
};
} // namespace
-GbmSurfaceFactory::GbmSurfaceFactory(DrmThreadProxy* drm_thread)
- : egl_implementation_(new GLOzoneEGLGbm(this, drm_thread)),
- drm_thread_(drm_thread) {}
+GbmSurfaceFactory::GbmSurfaceFactory(DrmThreadProxy* drm_thread_proxy)
+ : egl_implementation_(new GLOzoneEGLGbm(this, drm_thread_proxy)),
+ drm_thread_proxy_(drm_thread_proxy) {}
GbmSurfaceFactory::~GbmSurfaceFactory() {
DCHECK(thread_checker_.CalledOnValidThread());
@@ -126,7 +130,7 @@ std::unique_ptr<SurfaceOzoneCanvas> GbmSurfaceFactory::CreateCanvasForWidget(
std::vector<gfx::BufferFormat> GbmSurfaceFactory::GetScanoutFormats(
gfx::AcceleratedWidget widget) {
std::vector<gfx::BufferFormat> scanout_formats;
- drm_thread_->GetScanoutFormats(widget, &scanout_formats);
+ drm_thread_proxy_->GetScanoutFormats(widget, &scanout_formats);
return scanout_formats;
}
@@ -142,7 +146,7 @@ scoped_refptr<ui::NativePixmap> GbmSurfaceFactory::CreateNativePixmap(
#endif
scoped_refptr<GbmBuffer> buffer =
- drm_thread_->CreateBuffer(widget, size, format, usage);
+ drm_thread_proxy_->CreateBuffer(widget, size, format, usage);
if (!buffer.get())
return nullptr;
@@ -170,7 +174,7 @@ scoped_refptr<ui::NativePixmap> GbmSurfaceFactory::CreateNativePixmapFromHandle(
planes.push_back(plane);
}
- scoped_refptr<GbmBuffer> buffer = drm_thread_->CreateBufferFromFds(
+ scoped_refptr<GbmBuffer> buffer = drm_thread_proxy_->CreateBufferFromFds(
widget, size, format, std::move(scoped_fds), planes);
if (!buffer)
return nullptr;
« no previous file with comments | « ui/ozone/platform/drm/gpu/gbm_surface_factory.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698