diff options
| author | alokp@chromium.org <alokp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-08 17:36:26 +0000 |
|---|---|---|
| committer | alokp@chromium.org <alokp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-08 17:36:26 +0000 |
| commit | afc4ce91f94bc0cb976884e1c1c785a9e75f127a (patch) | |
| tree | 0c64c063fe026abd1f2a81a97fe04b9dfd514702 /webkit/plugins/sad_plugin.cc | |
| parent | 73689e53255c6ac22a90b0cd01fe40180ec2bbba (diff) | |
| download | chromium_src-afc4ce91f94bc0cb976884e1c1c785a9e75f127a.zip chromium_src-afc4ce91f94bc0cb976884e1c1c785a9e75f127a.tar.gz chromium_src-afc4ce91f94bc0cb976884e1c1c785a9e75f127a.tar.bz2 | |
Eliminate skia::PlatformCanvas, a subclass of SkCanvas. Skia provides multiple types of SkCanvas classes that we would like to use. Unfortunately these classes are implemented as subclasses of SkCanvas. Subclassing SkCanvas in both Skia and Chromium makes it impossible to dynamically use any SkCanvas. There is also no reason for chromium to subclass SkCanvas. Most of the extra functionalities can be implemented by hanging meta-data from SkCanvas.
We cannot eliminate skia::PlatformCanvas in one step due to WebKit's dependency on skia::PlatformCanvas. WebKit::WebCanvas is typedef as skia::PlatformDevice. It should be SkCanvas. So we need to do it in multiple steps:
1. Prepare Chromium tree for the change in WebKit::WebCanvas tyepdef. This basically means adding a couple of static_cast<skia::PlatformCanvas>(WebCanvas).
2. Change WebKit::WebCanvas typedef from skia::PlatformCanvas to SkCanvas
3. Eliminate skia::PlatformCanvas in chromium
This CL accomplishes the first step on windows.
WebKit BUG=https://bugs.webkit.org/show_bug.cgi?id=57563
Review URL: http://codereview.chromium.org/6783023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80955 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/plugins/sad_plugin.cc')
| -rw-r--r-- | webkit/plugins/sad_plugin.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/webkit/plugins/sad_plugin.cc b/webkit/plugins/sad_plugin.cc index 7512b4b..c129d34 100644 --- a/webkit/plugins/sad_plugin.cc +++ b/webkit/plugins/sad_plugin.cc @@ -38,9 +38,9 @@ void PaintSadPlugin(WebKit::WebCanvas* webcanvas, // then copy that to the screen than to use the native APIs. The small speed // penalty is not important when drawing crashed plugins. #if WEBKIT_USING_SKIA - gfx::NativeDrawingContext context = webcanvas->beginPlatformPaint(); + gfx::NativeDrawingContext context = skia::BeginPlatformPaint(webcanvas); BlitCanvasToContext(context, plugin_rect, &canvas, gfx::Point(0, 0)); - webcanvas->endPlatformPaint(); + skia::EndPlatformPaint(webcanvas); #elif WEBKIT_USING_CG BlitCanvasToContext(webcanvas, plugin_rect, &canvas, gfx::Point(0, 0)); #endif |