Return of image-surface backed canvas (windows graphics performance)

This partially reverts 2edbda2526.

Using cairo-groups increases performance on MacOS, and retains
retina-resolution.
However it adds a performance regression for MS Windows graphics
rendering. cairo-groups use a "similar" surface, not an image surface.
Empirically this adds significant overhead compared to rendering
using the CPU and using bitblt.
This commit is contained in:
Robin Gareus 2020-01-04 00:29:48 +01:00
parent f892e5bcfa
commit 0b266a54f0
No known key found for this signature in database
GPG key ID: A090BCE02CF57F04
2 changed files with 42 additions and 7 deletions

View file

@ -58,11 +58,12 @@ Canvas::Canvas ()
: _root (this) : _root (this)
, _bg_color (Gtkmm2ext::rgba_to_color (0, 1.0, 0.0, 1.0)) , _bg_color (Gtkmm2ext::rgba_to_color (0, 1.0, 0.0, 1.0))
, _last_render_start_timestamp(0) , _last_render_start_timestamp(0)
, _use_intermediate_surface (false)
{ {
#if (defined USE_CAIRO_IMAGE_SURFACE || defined __APPLE__) #ifdef __APPLE__
_use_intermediate_surface = true; _use_intermediate_surface = true;
#else #else
_use_intermediate_surface = NULL != getenv("ARDOUR_IMAGE_SURFACE"); _use_intermediate_surface = NULL != g_getenv("ARDOUR_INTERMEDIATE_SURFACE");
#endif #endif
set_epoch (); set_epoch ();
} }
@ -70,6 +71,9 @@ Canvas::Canvas ()
void void
Canvas::use_intermediate_surface (bool yn) Canvas::use_intermediate_surface (bool yn)
{ {
if (_use_intermediate_surface == yn) {
return;
}
_use_intermediate_surface = yn; _use_intermediate_surface = yn;
} }
@ -442,12 +446,19 @@ GtkCanvas::GtkCanvas ()
, _new_current_item (0) , _new_current_item (0)
, _grabbed_item (0) , _grabbed_item (0)
, _focused_item (0) , _focused_item (0)
, _single_exposure (1) , _single_exposure (true)
, _use_image_surface (false)
, current_tooltip_item (0) , current_tooltip_item (0)
, tooltip_window (0) , tooltip_window (0)
, _in_dtor (false) , _in_dtor (false)
, _nsglview (0) , _nsglview (0)
{ {
#ifdef USE_CAIRO_IMAGE_SURFACE /* usually Windows builds */
_use_image_surface = true
#else
_use_image_surface = NULL != g_getenv("ARDOUR_IMAGE_SURFACE");
#endif
/* these are the events we want to know about */ /* these are the events we want to know about */
add_events (Gdk::BUTTON_PRESS_MASK | Gdk::BUTTON_RELEASE_MASK | Gdk::POINTER_MOTION_MASK | add_events (Gdk::BUTTON_PRESS_MASK | Gdk::BUTTON_RELEASE_MASK | Gdk::POINTER_MOTION_MASK |
Gdk::SCROLL_MASK | Gdk::ENTER_NOTIFY_MASK | Gdk::LEAVE_NOTIFY_MASK | Gdk::SCROLL_MASK | Gdk::ENTER_NOTIFY_MASK | Gdk::LEAVE_NOTIFY_MASK |
@ -847,6 +858,11 @@ GtkCanvas::on_size_allocate (Gtk::Allocation& a)
{ {
EventBox::on_size_allocate (a); EventBox::on_size_allocate (a);
if (_use_image_surface) {
_canvas_image.clear ();
_canvas_image = Cairo::ImageSurface::create (Cairo::FORMAT_ARGB32, a.get_width(), a.get_height());
}
#ifdef __APPLE__ #ifdef __APPLE__
if (_nsglview) { if (_nsglview) {
gint xx, yy; gint xx, yy;
@ -880,7 +896,15 @@ GtkCanvas::on_expose_event (GdkEventExpose* ev)
const int64_t start = g_get_monotonic_time (); const int64_t start = g_get_monotonic_time ();
#endif #endif
Cairo::RefPtr<Cairo::Context> draw_context = get_window()->create_cairo_context (); Cairo::RefPtr<Cairo::Context> draw_context;
if (_use_image_surface) {
if (!_canvas_image) {
_canvas_image = Cairo::ImageSurface::create (Cairo::FORMAT_ARGB32, get_width(), get_height());
}
draw_context = Cairo::Context::create (_canvas_image);
} else {
draw_context = get_window()->create_cairo_context ();
}
draw_context->rectangle (ev->area.x, ev->area.y, ev->area.width, ev->area.height); draw_context->rectangle (ev->area.x, ev->area.y, ev->area.width, ev->area.height);
draw_context->clip(); draw_context->clip();
@ -901,7 +925,7 @@ GtkCanvas::on_expose_event (GdkEventExpose* ev)
* *
* Fixing this for good likely involves changes to GdkQuartzWindow, GdkQuartzView * Fixing this for good likely involves changes to GdkQuartzWindow, GdkQuartzView
*/ */
if (_use_intermediate_surface) { if (_use_intermediate_surface && !_use_image_surface) {
draw_context->push_group (); draw_context->push_group ();
} }
@ -911,7 +935,7 @@ GtkCanvas::on_expose_event (GdkEventExpose* ev)
draw_context->fill (); draw_context->fill ();
/* render canvas */ /* render canvas */
if ( _single_exposure ) { if (_single_exposure) {
Canvas::render (Rect (ev->area.x, ev->area.y, ev->area.x + ev->area.width, ev->area.y + ev->area.height), draw_context); Canvas::render (Rect (ev->area.x, ev->area.y, ev->area.x + ev->area.width, ev->area.y + ev->area.height), draw_context);
@ -927,11 +951,20 @@ GtkCanvas::on_expose_event (GdkEventExpose* ev)
g_free (rects); g_free (rects);
} }
if (_use_intermediate_surface) { if (_use_image_surface) {
_canvas_image->flush ();
Cairo::RefPtr<Cairo::Context> window_context = get_window()->create_cairo_context ();
window_context->rectangle (ev->area.x, ev->area.y, ev->area.width, ev->area.height);
window_context->clip ();
window_context->set_source (_canvas_image, 0, 0);
window_context->set_operator (Cairo::OPERATOR_SOURCE);
window_context->paint ();
} else if (_use_intermediate_surface) {
draw_context->pop_group_to_source (); draw_context->pop_group_to_source ();
draw_context->paint (); draw_context->paint ();
} }
#ifdef CANVAS_PROFILE #ifdef CANVAS_PROFILE
const int64_t end = g_get_monotonic_time (); const int64_t end = g_get_monotonic_time ();
const int64_t elapsed = end - start; const int64_t elapsed = end - start;

View file

@ -279,6 +279,7 @@ private:
Item * _focused_item; Item * _focused_item;
bool _single_exposure; bool _single_exposure;
bool _use_image_surface;
sigc::connection tooltip_timeout_connection; sigc::connection tooltip_timeout_connection;
Item* current_tooltip_item; Item* current_tooltip_item;
@ -291,6 +292,7 @@ private:
bool _in_dtor; bool _in_dtor;
void* _nsglview; void* _nsglview;
Cairo::RefPtr<Cairo::Surface> _canvas_image;
}; };
/** A GTK::Alignment with a GtkCanvas inside it plus some Gtk::Adjustments for /** A GTK::Alignment with a GtkCanvas inside it plus some Gtk::Adjustments for