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

Issue 155700: Reverting 20516. (Closed)

Created:
11 years, 5 months ago by brettw
Modified:
9 years, 7 months ago
Reviewers:
brettw
CC:
chromium-reviews_googlegroups.com, darin (slow to review), brettw, jam
Visibility:
Public.

Description

Reverting 20516 by re-landing this previous patch. Keep the cairo clipping region in sync with the Skia one. The PlatformCanvas now tracks this, so we don't need to have the similar code in gfx::Canvas. I moved most references of cairo_surface_t to cairo_t since the cairo_t has a transform and clip but the surface does not. Review URL: http://codereview.chromium.org/149409 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20992

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+146 lines, -68 lines) Patch
M app/gfx/canvas_linux.cc View 2 chunks +1 line, -20 lines 0 comments Download
M base/gfx/native_widget_types.h View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/renderer/webplugin_delegate_proxy.cc View 1 chunk +3 lines, -10 lines 0 comments Download
M skia/ext/bitmap_platform_device_linux.h View 2 chunks +5 lines, -3 lines 0 comments Download
M skia/ext/bitmap_platform_device_linux.cc View 4 chunks +109 lines, -10 lines 0 comments Download
M skia/ext/bitmap_platform_device_win.h View 1 chunk +3 lines, -0 lines 0 comments Download
M skia/ext/canvas_paint_linux.h View 4 chunks +6 lines, -5 lines 0 comments Download
M skia/ext/platform_canvas_linux.cc View 1 chunk +1 line, -1 line 0 comments Download
M skia/ext/platform_device_linux.h View 1 chunk +3 lines, -3 lines 0 comments Download
M webkit/glue/plugins/webplugin_delegate_impl_gtk.cc View 4 chunks +10 lines, -10 lines 0 comments Download
M webkit/tools/test_shell/webwidget_host_gtk.cc View 1 chunk +3 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
brettw
11 years, 5 months ago (2009-07-17 18:10:01 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698