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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: headless/lib/browser/headless_clipboard.cc

Issue 2855503003: Replace uses of legacy SkBitmap::copyTo() API (Closed)
Patch Set: Remove braces Created 3 years, 8 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 | « content/shell/test_runner/pixel_dump.cc ('k') | ios/chrome/browser/google/google_logo_service.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: headless/lib/browser/headless_clipboard.cc
diff --git a/headless/lib/browser/headless_clipboard.cc b/headless/lib/browser/headless_clipboard.cc
index be4048b0b9e6c6f848234b605048b9c6eaaa1b77..dcadd4c36c137420d303955123b916d9de26672a 100644
--- a/headless/lib/browser/headless_clipboard.cc
+++ b/headless/lib/browser/headless_clipboard.cc
@@ -166,7 +166,10 @@ void HeadlessClipboard::WriteWebSmartPaste() {
void HeadlessClipboard::WriteBitmap(const SkBitmap& bitmap) {
// Create a dummy entry.
GetDefaultStore().data[GetBitmapFormatType()];
- bitmap.copyTo(&GetDefaultStore().image);
+ SkBitmap& dst = GetDefaultStore().image;
+ if (dst.tryAllocPixels(bitmap.info())) {
+ bitmap.readPixels(dst.info(), dst.getPixels(), dst.rowBytes(), 0, 0);
+ }
}
void HeadlessClipboard::WriteData(const FormatType& format,
« no previous file with comments | « content/shell/test_runner/pixel_dump.cc ('k') | ios/chrome/browser/google/google_logo_service.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698