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

Unified Diff: src/effects/SkBitmapSource.cpp

Issue 108623002: Reverting 12528 & 12533 due to N4 GM diffs (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 7 years 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 | « include/effects/SkBitmapSource.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/effects/SkBitmapSource.cpp
===================================================================
--- src/effects/SkBitmapSource.cpp (revision 12533)
+++ src/effects/SkBitmapSource.cpp (working copy)
@@ -6,71 +6,24 @@
*/
#include "SkBitmapSource.h"
-#include "SkDevice.h"
-#include "SkCanvas.h"
-#include "SkFlattenableBuffers.h"
-#include "SkValidationUtils.h"
SkBitmapSource::SkBitmapSource(const SkBitmap& bitmap)
: INHERITED(0, 0),
- fBitmap(bitmap),
- fSrcRect(SkRect::MakeWH(SkIntToScalar(bitmap.width()),
- SkIntToScalar(bitmap.height()))),
- fDstRect(fSrcRect) {
+ fBitmap(bitmap) {
}
-SkBitmapSource::SkBitmapSource(const SkBitmap& bitmap, const SkRect& srcRect, const SkRect& dstRect)
- : INHERITED(0, 0),
- fBitmap(bitmap),
- fSrcRect(srcRect),
- fDstRect(dstRect) {
-}
-
SkBitmapSource::SkBitmapSource(SkFlattenableReadBuffer& buffer)
: INHERITED(0, buffer) {
fBitmap.unflatten(buffer);
- buffer.readRect(&fSrcRect);
- buffer.readRect(&fDstRect);
- buffer.validate(SkIsValidRect(fSrcRect) && SkIsValidRect(fDstRect));
}
void SkBitmapSource::flatten(SkFlattenableWriteBuffer& buffer) const {
this->INHERITED::flatten(buffer);
fBitmap.flatten(buffer);
- buffer.writeRect(fSrcRect);
- buffer.writeRect(fDstRect);
}
-bool SkBitmapSource::onFilterImage(Proxy* proxy, const SkBitmap&, const SkMatrix& matrix,
+bool SkBitmapSource::onFilterImage(Proxy*, const SkBitmap&, const SkMatrix&,
SkBitmap* result, SkIPoint* offset) {
- SkRect bounds, dstRect;
- fBitmap.getBounds(&bounds);
- matrix.mapRect(&dstRect, fDstRect);
- if (fSrcRect == bounds && dstRect == bounds) {
- *result = fBitmap;
- return true;
- }
- SkIRect dstIRect;
- dstRect.roundOut(&dstIRect);
-
- SkAutoTUnref<SkBaseDevice> device(proxy->createDevice(dstIRect.width(), dstIRect.height()));
- if (NULL == device.get()) {
- return false;
- }
-
- SkCanvas canvas(device.get());
- SkPaint paint;
-
- // Subtract off the integer component of the translation (will be applied in loc, below).
- dstRect.offset(-SkIntToScalar(dstIRect.fLeft), -SkIntToScalar(dstIRect.fTop));
- paint.setXfermodeMode(SkXfermode::kSrc_Mode);
- // FIXME: this probably shouldn't be necessary, but drawBitmapRectToRect asserts
- // None filtering when it's translate-only
- paint.setFilterLevel(fSrcRect.width() == dstRect.width() && fSrcRect.height() == dstRect.height() ? SkPaint::kNone_FilterLevel : SkPaint::kMedium_FilterLevel);
- canvas.drawBitmapRectToRect(fBitmap, &fSrcRect, dstRect, &paint);
-
- *result = device.get()->accessBitmap(false);
- offset->fX += dstIRect.fLeft;
- offset->fY += dstIRect.fTop;
+ *result = fBitmap;
return true;
}
« no previous file with comments | « include/effects/SkBitmapSource.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698