a11: using ATK_ROLE_DOCUMENT_FRAME it's more correct

[evince.git] / backend / dvi / cairo-device.c

diff --git a/backend/dvi/cairo-device.c b/backend/dvi/cairo-device.c
index 51aea3790c5fe2d6501db737e759740d87242012..9f4badb2546496ebf7e575f4b12ebb787bc19dc2 100644 (file)
--- a/backend/dvi/cairo-device.c
+++ b/backend/dvi/cairo-device.c
@@ -202,12 +202,9 @@ dvi_cairo_alloc_colors (void  *device_data,
                        int    density)
 {
        double  frac;
-       GdkColor color, color_fg, color_bg;
+       GdkColor color, color_fg;
        int     i, n;
-
-       color_bg.red = (bg >> 16) & 0xff;
-       color_bg.green = (bg >> 8) & 0xff;
-       color_bg.blue = (bg >> 0) & 0xff;
+       unsigned int alpha;
 
        color_fg.red = (fg >> 16) & 0xff;
        color_fg.green = (fg >> 8) & 0xff;
@@ -219,11 +216,12 @@ dvi_cairo_alloc_colors (void  *device_data,
                        pow ((double)i / n, 1 / gamma) :
                        1 - pow ((double)(n - i) / n, -gamma);
                
-               color.red = frac * ((double)color_fg.red - color_bg.red) + color_bg.red;
-               color.green = frac * ((double)color_fg.green - color_bg.green) + color_bg.green;
-               color.blue = frac * ((double)color_fg.blue - color_bg.blue) + color_bg.blue;
-               
-               pixels[i] = (color.red << 16) + (color.green << 8) + color.blue + 0xff000000;
+               color.red = frac * color_fg.red;
+               color.green = frac * color_fg.green;
+               color.blue = frac * color_fg.blue;
+               alpha = frac * 0xFF;
+
+               pixels[i] = (alpha << 24) + (color.red << 16) + (color.green << 8) + color.blue;
        }
 
        return npixels;
@@ -235,7 +233,7 @@ dvi_cairo_create_image (void *device_data,
                        Uint  height,
                        Uint  bpp)
 {
-       return cairo_image_surface_create (CAIRO_FORMAT_RGB24, width, height);
+       return cairo_image_surface_create (CAIRO_FORMAT_ARGB32, width, height);
 }
 
 static void
@@ -256,6 +254,8 @@ dvi_cairo_put_pixel (void *image, int x, int y, Ulong color)
        rowstride = cairo_image_surface_get_stride (surface);
        p = (guint32*) (cairo_image_surface_get_data (surface) + y * rowstride + x * 4);
 
+        /* per cairo docs, must flush before modifying outside of cairo */
+        cairo_surface_flush(surface);
        *p = color;
 }
 
@@ -326,9 +326,6 @@ mdvi_cairo_device_render (DviContext* dvi)
        gint             page_width;
        gint             page_height;
        cairo_surface_t *surface;
-       guchar          *pixels;
-       gint             rowstride;
-       static const cairo_user_data_key_t key;
 
        cairo_device = (DviCairoDevice *) dvi->device.device_data;
 
@@ -338,19 +335,14 @@ mdvi_cairo_device_render (DviContext* dvi)
        page_width = dvi->dvi_page_w * dvi->params.conv + 2 * cairo_device->xmargin;
        page_height = dvi->dvi_page_h * dvi->params.vconv + 2 * cairo_device->ymargin;
 
-       rowstride = page_width * 4;
-       pixels = (guchar *) g_malloc (page_height * rowstride);
-       memset (pixels, 0xff, page_height * rowstride);
-
-       surface = cairo_image_surface_create_for_data (pixels,
-                                                      CAIRO_FORMAT_RGB24,
-                                                      page_width, page_height,
-                                                      rowstride);
-       cairo_surface_set_user_data (surface, &key,
-                                    pixels, (cairo_destroy_func_t)g_free);
+       surface = cairo_image_surface_create (CAIRO_FORMAT_ARGB32,
+                                              page_width, page_height);
 
        cairo_device->cr = cairo_create (surface);
-       cairo_surface_destroy (surface);
+        cairo_surface_destroy (surface);
+
+        cairo_set_source_rgb (cairo_device->cr, 1., 1., 1.);
+        cairo_paint (cairo_device->cr);
 
        mdvi_dopage (dvi, dvi->currpage);
 }