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

Unified Diff: content/browser/compositor/io_surface_texture_mac.mm

Issue 753933002: MacViews: Move content::AcceleratedWidget to new component, ui/accelerated_widget_mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@20141124-MacViews-MoveSoftwareLayerMac-fromcl
Patch Set: gn check Created 6 years, 1 month 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
Index: content/browser/compositor/io_surface_texture_mac.mm
diff --git a/content/browser/compositor/io_surface_texture_mac.mm b/content/browser/compositor/io_surface_texture_mac.mm
deleted file mode 100644
index dea6b2876828f0d477ab96ebc989fa0985a5341b..0000000000000000000000000000000000000000
--- a/content/browser/compositor/io_surface_texture_mac.mm
+++ /dev/null
@@ -1,297 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/browser/compositor/io_surface_texture_mac.h"
-
-#include <OpenGL/CGLIOSurface.h>
-#include <OpenGL/CGLRenderers.h>
-#include <OpenGL/OpenGL.h>
-#include <OpenGL/gl.h>
-
-#include "base/bind.h"
-#include "base/bind_helpers.h"
-#include "base/callback_helpers.h"
-#include "base/debug/trace_event.h"
-#include "base/logging.h"
-#include "base/mac/bind_objc_block.h"
-#include "base/mac/mac_util.h"
-#include "base/message_loop/message_loop.h"
-#include "base/threading/platform_thread.h"
-#include "content/browser/compositor/io_surface_context_mac.h"
-#include "content/browser/gpu/gpu_data_manager_impl.h"
-#include "content/browser/renderer_host/render_widget_host_impl.h"
-#include "content/browser/renderer_host/render_widget_host_view_mac.h"
-#include "content/common/content_constants_internal.h"
-#include "gpu/config/gpu_driver_bug_workaround_type.h"
-#include "media/base/video_util.h"
-#include "third_party/skia/include/core/SkBitmap.h"
-#include "ui/gfx/rect.h"
-#include "ui/gfx/scoped_ns_graphics_context_save_gstate_mac.h"
-#include "ui/gfx/size_conversions.h"
-#include "ui/gl/gl_context.h"
-
-namespace content {
-
-// static
-scoped_refptr<IOSurfaceTexture> IOSurfaceTexture::Create() {
- scoped_refptr<IOSurfaceContext> offscreen_context =
- IOSurfaceContext::Get(
- IOSurfaceContext::kOffscreenContext);
- if (!offscreen_context.get()) {
- LOG(ERROR) << "Failed to create context for offscreen operations";
- return NULL;
- }
-
- return new IOSurfaceTexture(offscreen_context);
-}
-
-IOSurfaceTexture::IOSurfaceTexture(
- const scoped_refptr<IOSurfaceContext>& offscreen_context)
- : offscreen_context_(offscreen_context),
- texture_(0),
- gl_error_(GL_NO_ERROR),
- eviction_queue_iterator_(eviction_queue_.Get().end()),
- eviction_has_been_drawn_since_updated_(false) {
- CHECK(offscreen_context_.get());
-}
-
-IOSurfaceTexture::~IOSurfaceTexture() {
- ReleaseIOSurfaceAndTexture();
- offscreen_context_ = NULL;
- DCHECK(eviction_queue_iterator_ == eviction_queue_.Get().end());
-}
-
-bool IOSurfaceTexture::DrawIOSurface() {
- TRACE_EVENT0("browser", "IOSurfaceTexture::DrawIOSurface");
-
- // If we have release the IOSurface, clear the screen to light grey and
- // early-out.
- if (!io_surface_) {
- glClearColor(0.9, 0.9, 0.9, 1);
- glClear(GL_COLOR_BUFFER_BIT);
- return false;
- }
-
- // The viewport is the size of the CALayer, which should always match the
- // IOSurface pixel size.
- GLint viewport[4];
- glGetIntegerv(GL_VIEWPORT, viewport);
- gfx::Rect viewport_rect(viewport[0], viewport[1], viewport[2], viewport[3]);
- DCHECK_EQ(pixel_size_.ToString(), viewport_rect.size().ToString());
-
- // Set the projection matrix to match 1 unit to 1 pixel.
- glMatrixMode(GL_PROJECTION);
- glLoadIdentity();
- glOrtho(0, viewport_rect.width(), 0, viewport_rect.height(), -1, 1);
- glMatrixMode(GL_MODELVIEW);
- glLoadIdentity();
-
- // Draw a quad the size of the IOSurface. This should cover the full viewport.
- glColor4f(1, 1, 1, 1);
- glEnable(GL_TEXTURE_RECTANGLE_ARB);
- glBindTexture(GL_TEXTURE_RECTANGLE_ARB, texture_);
- glBegin(GL_QUADS);
- glTexCoord2f(0, 0);
- glVertex2f(0, 0);
- glTexCoord2f(pixel_size_.width(), 0);
- glVertex2f(pixel_size_.width(), 0);
- glTexCoord2f(pixel_size_.width(), pixel_size_.height());
- glVertex2f(pixel_size_.width(), pixel_size_.height());
- glTexCoord2f(0, pixel_size_.height());
- glVertex2f(0, pixel_size_.height());
- glEnd();
- glBindTexture(GL_TEXTURE_RECTANGLE_ARB, 0);
- glDisable(GL_TEXTURE_RECTANGLE_ARB);
-
- // Workaround for issue 158469. Issue a dummy draw call with texture_ not
- // bound to a texture, in order to shake all references to the IOSurface out
- // of the driver.
- glBegin(GL_TRIANGLES);
- glEnd();
-
- bool workaround_needed =
- GpuDataManagerImpl::GetInstance()->IsDriverBugWorkaroundActive(
- gpu::FORCE_GL_FINISH_AFTER_COMPOSITING);
- if (workaround_needed) {
- TRACE_EVENT0("gpu", "glFinish");
- glFinish();
- }
-
- // Check if any of the drawing calls result in an error.
- GetAndSaveGLError();
- bool result = true;
- if (gl_error_ != GL_NO_ERROR) {
- LOG(ERROR) << "GL error in DrawIOSurface: " << gl_error_;
- result = false;
- // If there was an error, clear the screen to a light grey to avoid
- // rendering artifacts.
- glClearColor(0.8, 0.8, 0.8, 1.0);
- glClear(GL_COLOR_BUFFER_BIT);
- }
-
- eviction_has_been_drawn_since_updated_ = true;
- return result;
-}
-
-bool IOSurfaceTexture::SetIOSurface(
- IOSurfaceID io_surface_id,
- const gfx::Size& pixel_size) {
- TRACE_EVENT0("browser", "IOSurfaceTexture::MapIOSurfaceToTexture");
-
- // Destroy the old IOSurface and texture if it is no longer needed.
- bool needs_new_iosurface =
- !io_surface_ || io_surface_id != IOSurfaceGetID(io_surface_);
- if (needs_new_iosurface)
- ReleaseIOSurfaceAndTexture();
-
- // Note that because IOSurface sizes are rounded, the same IOSurface may have
- // two different sizes associated with it, so update the sizes before the
- // early-out.
- pixel_size_ = pixel_size;
-
- // Early-out if the IOSurface has not changed.
- if (!needs_new_iosurface)
- return true;
-
- // If we early-out at any point from now on, it's because of an error, and we
- // should destroy the texture and release the IOSurface.
- base::ScopedClosureRunner error_runner(base::BindBlock(^{
- ReleaseIOSurfaceAndTexture();
- }));
-
- // Open the IOSurface handle.
- io_surface_.reset(IOSurfaceLookup(io_surface_id));
- if (!io_surface_)
- return false;
-
- // Actual IOSurface size is rounded up to reduce reallocations during window
- // resize. Get the actual size to properly map the texture.
- gfx::Size rounded_size(IOSurfaceGetWidth(io_surface_),
- IOSurfaceGetHeight(io_surface_));
-
- // Create the GL texture and set it to be backed by the IOSurface.
- CGLError cgl_error = kCGLNoError;
- {
- gfx::ScopedCGLSetCurrentContext scoped_set_current_context(
- offscreen_context_->cgl_context());
- glGenTextures(1, &texture_);
- glBindTexture(GL_TEXTURE_RECTANGLE_ARB, texture_);
- glTexParameterf(
- GL_TEXTURE_RECTANGLE_ARB, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
- glTexParameterf(
- GL_TEXTURE_RECTANGLE_ARB, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
- cgl_error = CGLTexImageIOSurface2D(
- offscreen_context_->cgl_context(),
- GL_TEXTURE_RECTANGLE_ARB,
- GL_RGBA,
- rounded_size.width(),
- rounded_size.height(),
- GL_BGRA,
- GL_UNSIGNED_INT_8_8_8_8_REV,
- io_surface_.get(),
- 0 /* plane */);
- glBindTexture(GL_TEXTURE_RECTANGLE_ARB, 0);
- GetAndSaveGLError();
- }
-
- // Return failure if an error was encountered by CGL or GL.
- if (cgl_error != kCGLNoError) {
- LOG(ERROR) << "CGLTexImageIOSurface2D failed with CGL error: " << cgl_error;
- return false;
- }
- if (gl_error_ != GL_NO_ERROR) {
- LOG(ERROR) << "Hit GL error in SetIOSurface: " << gl_error_;
- return false;
- }
-
- ignore_result(error_runner.Release());
- return true;
-}
-
-void IOSurfaceTexture::ReleaseIOSurfaceAndTexture() {
- gfx::ScopedCGLSetCurrentContext scoped_set_current_context(
- offscreen_context_->cgl_context());
-
- if (texture_) {
- glDeleteTextures(1, &texture_);
- texture_ = 0;
- }
- pixel_size_ = gfx::Size();
- io_surface_.reset();
-
- EvictionMarkEvicted();
-}
-
-bool IOSurfaceTexture::HasBeenPoisoned() const {
- return offscreen_context_->HasBeenPoisoned();
-}
-
-GLenum IOSurfaceTexture::GetAndSaveGLError() {
- GLenum gl_error = glGetError();
- if (gl_error_ == GL_NO_ERROR)
- gl_error_ = gl_error;
- return gl_error;
-}
-
-void IOSurfaceTexture::EvictionMarkUpdated() {
- EvictionMarkEvicted();
- eviction_queue_.Get().push_back(this);
- eviction_queue_iterator_ = --eviction_queue_.Get().end();
- eviction_has_been_drawn_since_updated_ = false;
- EvictionScheduleDoEvict();
-}
-
-void IOSurfaceTexture::EvictionMarkEvicted() {
- if (eviction_queue_iterator_ == eviction_queue_.Get().end())
- return;
- eviction_queue_.Get().erase(eviction_queue_iterator_);
- eviction_queue_iterator_ = eviction_queue_.Get().end();
- eviction_has_been_drawn_since_updated_ = false;
-}
-
-// static
-void IOSurfaceTexture::EvictionScheduleDoEvict() {
- if (eviction_scheduled_)
- return;
- if (eviction_queue_.Get().size() <= kMaximumUnevictedSurfaces)
- return;
-
- eviction_scheduled_ = true;
- base::MessageLoop::current()->PostTask(
- FROM_HERE,
- base::Bind(&IOSurfaceTexture::EvictionDoEvict));
-}
-
-// static
-void IOSurfaceTexture::EvictionDoEvict() {
- eviction_scheduled_ = false;
- // Walk the list of allocated surfaces from least recently used to most
- // recently used.
- for (EvictionQueue::iterator it = eviction_queue_.Get().begin();
- it != eviction_queue_.Get().end();) {
- IOSurfaceTexture* surface = *it;
- ++it;
-
- // If the number of IOSurfaces allocated is less than the threshold,
- // stop walking the list of surfaces.
- if (eviction_queue_.Get().size() <= kMaximumUnevictedSurfaces)
- break;
-
- // Don't evict anything that has not yet been drawn.
- if (!surface->eviction_has_been_drawn_since_updated_)
- continue;
-
- // Evict the surface.
- surface->ReleaseIOSurfaceAndTexture();
- }
-}
-
-// static
-base::LazyInstance<IOSurfaceTexture::EvictionQueue>
- IOSurfaceTexture::eviction_queue_;
-
-// static
-bool IOSurfaceTexture::eviction_scheduled_ = false;
-
-} // namespace content
« no previous file with comments | « content/browser/compositor/io_surface_texture_mac.h ('k') | content/browser/compositor/software_layer_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698