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

Unified Diff: webkit/api/src/ChromiumBridge.cpp

Issue 308001: Chromium fix for "Copy Image".... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 months 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
« no previous file with comments | « app/clipboard/clipboard_mac.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/api/src/ChromiumBridge.cpp
===================================================================
--- webkit/api/src/ChromiumBridge.cpp (revision 29796)
+++ webkit/api/src/ChromiumBridge.cpp (working copy)
@@ -57,10 +57,13 @@
#include "WebFontInfo.h"
#endif
+#if WEBKIT_USING_SKIA
+#include "NativeImageSkia.h"
+#endif
+
#include "BitmapImage.h"
#include "GraphicsContext.h"
#include "KURL.h"
-#include "NativeImageSkia.h"
#include "NotImplemented.h"
#include "PlatformContextSkia.h"
#include "PluginData.h"
@@ -118,17 +121,16 @@
webKitClient()->clipboard()->writeURL(url, title);
}
-void ChromiumBridge::clipboardWriteImage(const NativeImageSkia* image,
+void ChromiumBridge::clipboardWriteImage(NativeImagePtr image,
const KURL& sourceURL,
const String& title)
{
#if WEBKIT_USING_SKIA
- webKitClient()->clipboard()->writeImage(
- WebImage(*image), sourceURL, title);
+ WebImage webImage(*image);
#else
- // FIXME clipboardWriteImage probably shouldn't take a NativeImageSkia
- notImplemented();
+ WebImage webImage(image);
#endif
+ webKitClient()->clipboard()->writeImage(webImage, sourceURL, title);
}
// Cookies --------------------------------------------------------------------
« no previous file with comments | « app/clipboard/clipboard_mac.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698