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

Unified Diff: chrome/renderer/chrome_render_frame_observer.cc

Issue 2895953003: Use SkJpegEncoder in gfx jpeg_codec (Closed)
Patch Set: Remove brackets Created 3 years, 6 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
Index: chrome/renderer/chrome_render_frame_observer.cc
diff --git a/chrome/renderer/chrome_render_frame_observer.cc b/chrome/renderer/chrome_render_frame_observer.cc
index a49d2e2f8da0a217b5cd9506a3dc3c8bc1d47275..08ca11e128025445d355c026ae12b3265e4af132 100644
--- a/chrome/renderer/chrome_render_frame_observer.cc
+++ b/chrome/renderer/chrome_render_frame_observer.cc
@@ -213,15 +213,10 @@ void ChromeRenderFrameObserver::RequestThumbnailForContextNode(
}
std::vector<uint8_t> thumbnail_data;
- if (bitmap.getPixels()) {
- const int kDefaultQuality = 90;
- std::vector<unsigned char> data;
- if (gfx::JPEGCodec::Encode(
- reinterpret_cast<unsigned char*>(bitmap.getAddr32(0, 0)),
- gfx::JPEGCodec::FORMAT_SkBitmap, bitmap.width(), bitmap.height(),
- static_cast<int>(bitmap.rowBytes()), kDefaultQuality, &data)) {
- thumbnail_data.swap(data);
- }
+ constexpr int kDefaultQuality = 90;
+ std::vector<unsigned char> data;
+ if (gfx::JPEGCodec::Encode(bitmap, kDefaultQuality, &data)) {
+ thumbnail_data.swap(data);
}
callback.Run(thumbnail_data, original_size);

Powered by Google App Engine
This is Rietveld 408576698