DEF_BENCH( return SkNEW_ARGS(AAClipBench, (true, true)); )
DEF_BENCH( return SkNEW_ARGS(NestedAAClipBench, (false)); )
DEF_BENCH( return SkNEW_ARGS(NestedAAClipBench, (true)); )
+
'<(skia_src_path)/core/SkScan_Antihair.cpp',
'<(skia_src_path)/core/SkScan_Hairline.cpp',
'<(skia_src_path)/core/SkScan_Path.cpp',
- '<(skia_src_path)/core/SkValidatingReadBuffer.cpp',
'<(skia_src_path)/core/SkShader.cpp',
'<(skia_src_path)/core/SkSpriteBlitter_ARGB32.cpp',
'<(skia_src_path)/core/SkSpriteBlitter_RGB16.cpp',
kRGB_565_Config, //!< 16-bits per pixel, (see SkColorPriv.h for packing)
kARGB_4444_Config, //!< 16-bits per pixel, (see SkColorPriv.h for packing)
kARGB_8888_Config, //!< 32-bits per pixel, (see SkColorPriv.h for packing)
- kLastConfig = kARGB_8888_Config,
};
// do not add this to the Config enum, otherwise the compiler will let us
kCrossProcess_Flag = 1 << 0,
kScalarIsFloat_Flag = 1 << 1,
kPtrIs64Bit_Flag = 1 << 2,
- /** The kValidation_Flag is used to force stream validations (by making
- * sure that no operation reads past the end of the stream, for example)
- * and error handling if any reading operation yields an invalid value.
- */
- kValidation_Flag = 1 << 3,
};
void setFlags(uint32_t flags) { fFlags = flags; }
uint32_t getFlags() const { return fFlags; }
- bool isCrossProcess() const { return SkToBool(fFlags & (kCrossProcess_Flag | kValidation_Flag)); }
+ bool isCrossProcess() const { return SkToBool(fFlags & kCrossProcess_Flag); }
bool isScalarFloat() const { return SkToBool(fFlags & kScalarIsFloat_Flag); }
bool isPtr64Bit() const { return SkToBool(fFlags & kPtrIs64Bit_Flag); }
- bool isValidating() const { return SkToBool(fFlags & kValidation_Flag); }
// primitives
virtual bool readBool() = 0;
return static_cast<T*>(this->readFlattenable());
}
- void validate(bool isValid) {
- fError |= !isValid;
- }
-
-protected:
- bool fError;
-
private:
uint32_t fFlags;
};
enum Flags {
kCrossProcess_Flag = 0x01,
- /** The kValidation_Flag is used here to make sure the write operation
- * is symmetric with the read operation using the equivalent flag
- * SkFlattenableReadBuffer::kValidation_Flag.
- */
- kValidation_Flag = 0x02,
};
uint32_t getFlags() const { return fFlags; }
void setFlags(uint32_t flags) { fFlags = flags; }
bool isCrossProcess() const {
- return SkToBool(fFlags & (kCrossProcess_Flag | kValidation_Flag));
- }
-
- bool isValidating() const {
- return SkToBool(fFlags & kValidation_Flag);
+ return SkToBool(fFlags & kCrossProcess_Flag);
}
bool persistTypeface() const { return (fFlags & kCrossProcess_Flag) != 0; }
class SkData;
class SkFlattenable;
-/**
- * These utility functions are used by the chromium codebase to safely
- * serialize and deserialize SkFlattenable objects. These aren't made for
- * optimal speed, but rather designed with security in mind in order to
- * prevent Skia from being an entry point for potential attacks.
- */
-SK_API SkData* SkValidatingSerializeFlattenable(SkFlattenable*);
-SK_API SkFlattenable* SkValidatingDeserializeFlattenable(const void* data, size_t size);
-
-// Temporary fix for canary build
-#define SkSerializeFlattenable SkValidatingSerializeFlattenable
-#define SkDeserializeFlattenable SkValidatingDeserializeFlattenable
+SK_API SkData* SkSerializeFlattenable(SkFlattenable*);
+SK_API SkFlattenable* SkDeserializeFlattenable(const void* data, size_t size);
#endif
*/
bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }
- bool isInverted() const { return fLeft > fRight || fTop > fBottom; }
-
bool isLargest() const { return SK_MinS32 == fLeft &&
SK_MinS32 == fTop &&
SK_MaxS32 == fRight &&
*/
bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }
- bool isInverted() const { return fLeft > fRight || fTop > fBottom; }
-
/**
* Returns true iff all values in the rect are finite. If any are
* infinite or NaN (or SK_FixedNaN when SkScalar is fixed) then this
#include "SkThread.h"
#include "SkUnPreMultiply.h"
#include "SkUtils.h"
-#include "SkValidationUtils.h"
#include "SkPackBits.h"
#include <new>
int width = buffer.readInt();
int height = buffer.readInt();
int rowBytes = buffer.readInt();
- Config config = (Config)buffer.readInt();
- buffer.validate((width >= 0) && (height >= 0) && (rowBytes >= 0) &&
- SkIsValidConfig(config));
+ int config = buffer.readInt();
- this->setConfig(config, width, height, rowBytes);
+ this->setConfig((Config)config, width, height, rowBytes);
this->setIsOpaque(buffer.readBool());
int reftype = buffer.readInt();
case SERIALIZE_PIXELTYPE_NONE:
break;
default:
- buffer.validate(false);
SkDEBUGFAIL("unrecognized pixeltype in serialized data");
sk_throw();
}
return false;
}
}
-
+
SkASSERT(fBitmap);
bool trivialMatrix = (fInvMatrix.getType() & ~SkMatrix::kTranslate_Mask) == 0;
#include "SkPaint.h"
#include "SkTypeface.h"
-SkFlattenableReadBuffer::SkFlattenableReadBuffer() :
- fError(false) {
+SkFlattenableReadBuffer::SkFlattenableReadBuffer() {
// Set default values. These should be explicitly set by our client
// via setFlags() if the buffer came from serialization.
fFlags = 0;
#include "SkData.h"
#include "SkFlattenable.h"
-#include "SkValidatingReadBuffer.h"
+#include "SkOrderedReadBuffer.h"
#include "SkOrderedWriteBuffer.h"
-SkData* SkValidatingSerializeFlattenable(SkFlattenable* flattenable) {
+SkData* SkSerializeFlattenable(SkFlattenable* flattenable) {
SkOrderedWriteBuffer writer(1024);
- writer.setFlags(SkOrderedWriteBuffer::kValidation_Flag);
+ writer.setFlags(SkOrderedWriteBuffer::kCrossProcess_Flag);
writer.writeFlattenable(flattenable);
uint32_t size = writer.bytesWritten();
void* data = sk_malloc_throw(size);
return SkData::NewFromMalloc(data, size);
}
-SkFlattenable* SkValidatingDeserializeFlattenable(const void* data, size_t size) {
- SkValidatingReadBuffer reader(data, size);
- return reader.readFlattenable();
+SkFlattenable* SkDeserializeFlattenable(const void* data, size_t size) {
+ SkOrderedReadBuffer buffer(data, size);
+ return buffer.readFlattenable();
}
#include "SkBitmap.h"
#include "SkFlattenableBuffers.h"
#include "SkRect.h"
-#include "SkValidationUtils.h"
#if SK_SUPPORT_GPU
#include "GrContext.h"
#include "GrTexture.h"
}
}
buffer.readIRect(&fCropRect);
- buffer.validate(SkIsValidRect(fCropRect));
}
void SkImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
factory = flattenable->getFactory();
}
if (NULL == factory) {
- if (this->isValidating()) {
- this->writeString(NULL);
- } else if (fFactorySet != NULL || fNamedFactorySet != NULL) {
+ if (fFactorySet != NULL || fNamedFactorySet != NULL) {
this->write32(0);
} else {
this->writeFunctionPtr(NULL);
* name. SkGPipe uses this technique so it can write the name to its
* stream before writing the flattenable.
*/
- if (this->isValidating()) {
- const char* name = SkFlattenable::FactoryToName(factory);
- this->writeString(name);
- if (NULL == name) {
- SkASSERT(0); // Missing factory name
- return;
- }
- } else if (fFactorySet) {
+ if (fFactorySet) {
this->write32(fFactorySet->add(factory));
} else if (fNamedFactorySet) {
int32_t index = fNamedFactorySet->find(factory);
+++ /dev/null
-/*
- * Copyright 2013 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-#include "SkBitmap.h"
-#include "SkErrorInternals.h"
-#include "SkValidatingReadBuffer.h"
-#include "SkStream.h"
-#include "SkTypeface.h"
-
-SkValidatingReadBuffer::SkValidatingReadBuffer() : INHERITED() {
- fMemoryPtr = NULL;
-
- fBitmapStorage = NULL;
- fTFArray = NULL;
- fTFCount = 0;
-
- fFactoryTDArray = NULL;
- fFactoryArray = NULL;
- fFactoryCount = 0;
- fBitmapDecoder = NULL;
-#ifdef DEBUG_NON_DETERMINISTIC_ASSERT
- fDecodedBitmapIndex = -1;
-#endif // DEBUG_NON_DETERMINISTIC_ASSERT
-
- setFlags(SkFlattenableReadBuffer::kValidation_Flag);
-}
-
-SkValidatingReadBuffer::SkValidatingReadBuffer(const void* data, size_t size) : INHERITED() {
- this->setMemory(data, size);
- fMemoryPtr = NULL;
-
- fBitmapStorage = NULL;
- fTFArray = NULL;
- fTFCount = 0;
-
- fFactoryTDArray = NULL;
- fFactoryArray = NULL;
- fFactoryCount = 0;
- fBitmapDecoder = NULL;
-#ifdef DEBUG_NON_DETERMINISTIC_ASSERT
- fDecodedBitmapIndex = -1;
-#endif // DEBUG_NON_DETERMINISTIC_ASSERT
-
- setFlags(SkFlattenableReadBuffer::kValidation_Flag);
-}
-
-SkValidatingReadBuffer::SkValidatingReadBuffer(SkStream* stream) {
- const size_t length = stream->getLength();
- fMemoryPtr = sk_malloc_throw(length);
- stream->read(fMemoryPtr, length);
- this->setMemory(fMemoryPtr, length);
-
- fBitmapStorage = NULL;
- fTFArray = NULL;
- fTFCount = 0;
-
- fFactoryTDArray = NULL;
- fFactoryArray = NULL;
- fFactoryCount = 0;
- fBitmapDecoder = NULL;
-#ifdef DEBUG_NON_DETERMINISTIC_ASSERT
- fDecodedBitmapIndex = -1;
-#endif // DEBUG_NON_DETERMINISTIC_ASSERT
-
- setFlags(SkFlattenableReadBuffer::kValidation_Flag);
-}
-
-SkValidatingReadBuffer::~SkValidatingReadBuffer() {
- sk_free(fMemoryPtr);
- SkSafeUnref(fBitmapStorage);
-}
-
-void SkValidatingReadBuffer::setMemory(const void* data, size_t size) {
- fError |= (!ptr_align_4(data) || (SkAlign4(size) != size));
- if (!fError) {
- fReader.setMemory(data, size);
- }
-}
-
-const void* SkValidatingReadBuffer::skip(size_t size) {
- size_t inc = SkAlign4(size);
- const void* addr = fReader.peek();
- fError |= !ptr_align_4(addr) || !fReader.isAvailable(inc);
- if (!fError) {
- fReader.skip(size);
- }
- return addr;
-}
-
-bool SkValidatingReadBuffer::readBool() {
- return this->readInt() != 0;
-}
-
-SkColor SkValidatingReadBuffer::readColor() {
- return this->readInt();
-}
-
-SkFixed SkValidatingReadBuffer::readFixed() {
- return this->readInt();
-}
-
-int32_t SkValidatingReadBuffer::readInt() {
- size_t inc = sizeof(int32_t);
- fError |= !ptr_align_4(fReader.peek()) || !fReader.isAvailable(inc);
- return fError ? 0 : fReader.readInt();
-}
-
-SkScalar SkValidatingReadBuffer::readScalar() {
- size_t inc = sizeof(SkScalar);
- fError |= !ptr_align_4(fReader.peek()) || !fReader.isAvailable(inc);
- return fError ? 0 : fReader.readScalar();
-}
-
-uint32_t SkValidatingReadBuffer::readUInt() {
- return this->readInt();
-}
-
-int32_t SkValidatingReadBuffer::read32() {
- return this->readInt();
-}
-
-void SkValidatingReadBuffer::readString(SkString* string) {
- size_t len = this->readInt();
- const void* ptr = fReader.peek();
-
- // skip over the string + '\0' and then pad to a multiple of 4
- size_t alignedSize = SkAlign4(len + 1);
- this->skip(alignedSize);
- if (!fError) {
- string->set((const char*)ptr, len);
- }
-}
-
-void* SkValidatingReadBuffer::readEncodedString(size_t* length, SkPaint::TextEncoding encoding) {
- int32_t encodingType = fReader.readInt();
- if (encodingType == encoding) {
- fError = true;
- }
- *length = this->readInt();
- const void* ptr = this->skip(SkAlign4(*length));
- void* data = NULL;
- if (!fError) {
- data = sk_malloc_throw(*length);
- memcpy(data, ptr, *length);
- }
- return data;
-}
-
-void SkValidatingReadBuffer::readPoint(SkPoint* point) {
- point->fX = fReader.readScalar();
- point->fY = fReader.readScalar();
-}
-
-void SkValidatingReadBuffer::readMatrix(SkMatrix* matrix) {
- size_t size = matrix->readFromMemory(fReader.peek());
- fError |= (SkAlign4(size) != size);
- if (!fError) {
- (void)this->skip(size);
- }
-}
-
-void SkValidatingReadBuffer::readIRect(SkIRect* rect) {
- memcpy(rect, this->skip(sizeof(SkIRect)), sizeof(SkIRect));
-}
-
-void SkValidatingReadBuffer::readRect(SkRect* rect) {
- memcpy(rect, this->skip(sizeof(SkRect)), sizeof(SkRect));
-}
-
-void SkValidatingReadBuffer::readRegion(SkRegion* region) {
- size_t size = region->readFromMemory(fReader.peek());
- fError |= (SkAlign4(size) != size);
- if (!fError) {
- (void)this->skip(size);
- }
-}
-
-void SkValidatingReadBuffer::readPath(SkPath* path) {
- size_t size = path->readFromMemory(fReader.peek());
- fError |= (SkAlign4(size) != size);
- if (!fError) {
- (void)this->skip(size);
- }
-}
-
-uint32_t SkValidatingReadBuffer::readByteArray(void* value) {
- const uint32_t length = this->readUInt();
- memcpy(value, this->skip(SkAlign4(length)), length);
- return fError ? 0 : length;
-}
-
-uint32_t SkValidatingReadBuffer::readColorArray(SkColor* colors) {
- const uint32_t count = this->readUInt();
- const uint32_t byteLength = count * sizeof(SkColor);
- memcpy(colors, this->skip(SkAlign4(byteLength)), byteLength);
- return fError ? 0 : count;
-}
-
-uint32_t SkValidatingReadBuffer::readIntArray(int32_t* values) {
- const uint32_t count = this->readUInt();
- const uint32_t byteLength = count * sizeof(int32_t);
- memcpy(values, this->skip(SkAlign4(byteLength)), byteLength);
- return fError ? 0 : count;
-}
-
-uint32_t SkValidatingReadBuffer::readPointArray(SkPoint* points) {
- const uint32_t count = this->readUInt();
- const uint32_t byteLength = count * sizeof(SkPoint);
- memcpy(points, this->skip(SkAlign4(byteLength)), byteLength);
- return fError ? 0 : count;
-}
-
-uint32_t SkValidatingReadBuffer::readScalarArray(SkScalar* values) {
- const uint32_t count = this->readUInt();
- const uint32_t byteLength = count * sizeof(SkScalar);
- memcpy(values, this->skip(SkAlign4(byteLength)), byteLength);
- return fError ? 0 : count;
-}
-
-uint32_t SkValidatingReadBuffer::getArrayCount() {
- return *(uint32_t*)fReader.peek();
-}
-
-void SkValidatingReadBuffer::readBitmap(SkBitmap* bitmap) {
- const int width = this->readInt();
- const int height = this->readInt();
- const size_t length = this->readUInt();
- // A size of zero means the SkBitmap was simply flattened.
- if (length != 0) {
- fError = true;
- }
- if (fError) {
- return;
- }
- bitmap->unflatten(*this);
- if ((bitmap->width() != width) || (bitmap->height() != height)) {
- fError = true;
- }
-}
-
-SkTypeface* SkValidatingReadBuffer::readTypeface() {
-
- uint32_t index = this->readUInt();
- if (0 == index || index > (unsigned)fTFCount || fError) {
- if (index) {
- SkDebugf("====== typeface index %d\n", index);
- }
- return NULL;
- } else {
- SkASSERT(fTFArray);
- return fTFArray[index - 1];
- }
-}
-
-SkFlattenable* SkValidatingReadBuffer::readFlattenable() {
- SkString string;
- this->readString(&string);
- if (fError) {
- return NULL;
- }
- SkFlattenable::Factory factory = SkFlattenable::NameToFactory(string.c_str());
- if (NULL == factory) {
- return NULL; // writer failed to give us the flattenable
- }
-
- // if we get here, factory may still be null, but if that is the case, the
- // failure was ours, not the writer.
- SkFlattenable* obj = NULL;
- uint32_t sizeRecorded = this->readUInt();
- if (factory) {
- uint32_t offset = fReader.offset();
- obj = (*factory)(*this);
- // check that we read the amount we expected
- uint32_t sizeRead = fReader.offset() - offset;
- if (sizeRecorded != sizeRead) {
- // we could try to fix up the offset...
- fError = true;
- delete obj;
- obj = NULL;
- }
- } else {
- // we must skip the remaining data
- this->skip(sizeRecorded);
- }
- return obj;
-}
+++ /dev/null
-/*
- * Copyright 2013 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-#ifndef SkValidatingReadBuffer_DEFINED
-#define SkValidatingReadBuffer_DEFINED
-
-#include "SkRefCnt.h"
-#include "SkBitmapHeap.h"
-#include "SkFlattenableBuffers.h"
-#include "SkPath.h"
-#include "SkPicture.h"
-#include "SkReader32.h"
-
-class SkBitmap;
-
-#if defined(SK_DEBUG) && defined(SK_BUILD_FOR_MAC)
- #define DEBUG_NON_DETERMINISTIC_ASSERT
-#endif
-
-class SkValidatingReadBuffer : public SkFlattenableReadBuffer {
-public:
- SkValidatingReadBuffer();
- SkValidatingReadBuffer(const void* data, size_t size);
- SkValidatingReadBuffer(SkStream* stream);
- virtual ~SkValidatingReadBuffer();
-
- SkReader32* getReader32() { return &fReader; }
-
- uint32_t size() { return fReader.size(); }
- uint32_t offset() { return fReader.offset(); }
- bool eof() { return fReader.eof(); }
- const void* skip(size_t size);
-
- // primitives
- virtual bool readBool() SK_OVERRIDE;
- virtual SkColor readColor() SK_OVERRIDE;
- virtual SkFixed readFixed() SK_OVERRIDE;
- virtual int32_t readInt() SK_OVERRIDE;
- virtual SkScalar readScalar() SK_OVERRIDE;
- virtual uint32_t readUInt() SK_OVERRIDE;
- virtual int32_t read32() SK_OVERRIDE;
-
- // strings -- the caller is responsible for freeing the string contents
- virtual void readString(SkString* string) SK_OVERRIDE;
- virtual void* readEncodedString(size_t* length, SkPaint::TextEncoding encoding) SK_OVERRIDE;
-
- // common data structures
- virtual SkFlattenable* readFlattenable() SK_OVERRIDE;
- virtual void readPoint(SkPoint* point) SK_OVERRIDE;
- virtual void readMatrix(SkMatrix* matrix) SK_OVERRIDE;
- virtual void readIRect(SkIRect* rect) SK_OVERRIDE;
- virtual void readRect(SkRect* rect) SK_OVERRIDE;
- virtual void readRegion(SkRegion* region) SK_OVERRIDE;
- virtual void readPath(SkPath* path) SK_OVERRIDE;
-
- // binary data and arrays
- virtual uint32_t readByteArray(void* value) SK_OVERRIDE;
- virtual uint32_t readColorArray(SkColor* colors) SK_OVERRIDE;
- virtual uint32_t readIntArray(int32_t* values) SK_OVERRIDE;
- virtual uint32_t readPointArray(SkPoint* points) SK_OVERRIDE;
- virtual uint32_t readScalarArray(SkScalar* values) SK_OVERRIDE;
-
- // helpers to get info about arrays and binary data
- virtual uint32_t getArrayCount() SK_OVERRIDE;
-
- virtual void readBitmap(SkBitmap* bitmap) SK_OVERRIDE;
- virtual SkTypeface* readTypeface() SK_OVERRIDE;
-
- void setBitmapStorage(SkBitmapHeapReader* bitmapStorage) {
- SkRefCnt_SafeAssign(fBitmapStorage, bitmapStorage);
- }
-
- void setTypefaceArray(SkTypeface* array[], int count) {
- fTFArray = array;
- fTFCount = count;
- }
-
- /**
- * Call this with a pre-loaded array of Factories, in the same order as
- * were created/written by the writer. SkPicture uses this.
- */
- void setFactoryPlayback(SkFlattenable::Factory array[], int count) {
- fFactoryTDArray = NULL;
- fFactoryArray = array;
- fFactoryCount = count;
- }
-
- /**
- * Call this with an initially empty array, so the reader can cache each
- * factory it sees by name. Used by the pipe code in conjunction with
- * SkOrderedWriteBuffer::setNamedFactoryRecorder.
- */
- void setFactoryArray(SkTDArray<SkFlattenable::Factory>* array) {
- fFactoryTDArray = array;
- fFactoryArray = NULL;
- fFactoryCount = 0;
- }
-
- /**
- * Provide a function to decode an SkBitmap from encoded data. Only used if the writer
- * encoded the SkBitmap. If the proper decoder cannot be used, a red bitmap with the
- * appropriate size will be used.
- */
- void setBitmapDecoder(SkPicture::InstallPixelRefProc bitmapDecoder) {
- fBitmapDecoder = bitmapDecoder;
- }
-
-private:
- void setMemory(const void* data, size_t size);
-
- static bool ptr_align_4(const void* ptr) {
- return (((const char*)ptr - (const char*)NULL) & 3) == 0;
- }
-
- SkReader32 fReader;
- void* fMemoryPtr;
-
- SkBitmapHeapReader* fBitmapStorage;
- SkTypeface** fTFArray;
- int fTFCount;
-
- SkTDArray<SkFlattenable::Factory>* fFactoryTDArray;
- SkFlattenable::Factory* fFactoryArray;
- int fFactoryCount;
-
- SkPicture::InstallPixelRefProc fBitmapDecoder;
-
-#ifdef DEBUG_NON_DETERMINISTIC_ASSERT
- // Debugging counter to keep track of how many bitmaps we
- // have decoded.
- int fDecodedBitmapIndex;
-#endif // DEBUG_NON_DETERMINISTIC_ASSERT
-
- typedef SkFlattenableReadBuffer INHERITED;
-};
-
-#endif // SkValidatingReadBuffer_DEFINED
+++ /dev/null
-/*
- * Copyright 2013 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-#ifndef SkValidationUtils_DEFINED
-#define SkValidationUtils_DEFINED
-
-#include "SkBitmap.h"
-#include "SkXfermode.h"
-
-/** Returns true if coeff's value is in the SkXfermode::Coeff enum.
- */
-static inline bool SkIsValidCoeff(SkXfermode::Coeff coeff) {
- return coeff >= 0 && coeff < SkXfermode::kCoeffCount;
-}
-
-/** Returns true if mode's value is in the SkXfermode::Mode enum.
- */
-static inline bool SkIsValidMode(SkXfermode::Mode mode) {
- return (mode >= 0) && (mode <= SkXfermode::kLastMode);
-}
-
-/** Returns true if config's value is in the SkBitmap::Config enum.
- */
-static inline bool SkIsValidConfig(SkBitmap::Config config) {
- return (config >= 0) && (config <= SkBitmap::kLastConfig);
-}
-
-/** Returns true if the rect's dimensions are between 0 and SK_MaxS32
- */
-static inline bool SkIsValidRect(const SkIRect& rect) {
- return rect.width() >= 0 && rect.height() >= 0;
-}
-
-/** Returns true if the rect's dimensions are between 0 and SK_ScalarMax
- */
-static inline bool SkIsValidRect(const SkRect& rect) {
- return !rect.isInverted() &&
- SkScalarIsFinite(rect.width()) &&
- SkScalarIsFinite(rect.height());
-}
-
-#endif
#include "SkFlattenableBuffers.h"
#include "SkMathPriv.h"
#include "SkString.h"
-#include "SkValidationUtils.h"
SK_DEFINE_INST_COUNT(SkXfermode)
fDstCoeff = rec.fDC;
// now update our function-ptr in the super class
this->INHERITED::setProc(rec.fProc);
-
- buffer.validate(SkIsValidMode(fMode) &&
- SkIsValidCoeff(fSrcCoeff) &&
- SkIsValidCoeff(fDstCoeff));
}
virtual void flatten(SkFlattenableWriteBuffer& buffer) const SK_OVERRIDE {
SkASSERT(readSize == 16);
fScale.fWidth = buffer.readScalar();
fScale.fHeight = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fScale.fWidth) &&
- SkScalarIsFinite(fScale.fHeight) &&
- (fScale.fWidth >= 0) &&
- (fScale.fHeight >= 0));
}
void SkBicubicImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
: INHERITED(buffer) {
fSigma.fWidth = buffer.readScalar();
fSigma.fHeight = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fSigma.fWidth) &&
- SkScalarIsFinite(fSigma.fHeight) &&
- (fSigma.fWidth >= 0) &&
- (fSigma.fHeight >= 0));
}
SkBlurImageFilter::SkBlurImageFilter(SkScalar sigmaX,
#include "SkFlattenableBuffers.h"
#include "SkUtils.h"
#include "SkString.h"
-#include "SkValidationUtils.h"
#define ILLEGAL_XFERMODE_MODE ((SkXfermode::Mode)-1)
fColor = buffer.readColor();
fMode = (SkXfermode::Mode)buffer.readUInt();
this->updateCache();
- buffer.validate(SkIsValidMode(fMode));
}
private:
SkASSERT(buffer.getArrayCount() == 20);
buffer.readScalarArray(fMatrix.fMat);
this->initState(fMatrix.fMat);
- for (int i = 0; i < 20; ++i) {
- buffer.validate(SkScalarIsFinite(fMatrix.fMat[0]));
- }
}
bool SkColorMatrixFilter::asColorMatrix(SkScalar matrix[20]) const {
}
}
-bool ChannelSelectorTypeIsValid(SkDisplacementMapEffect::ChannelSelectorType channelSelector) {
- switch (channelSelector) {
- case SkDisplacementMapEffect::kUnknown_ChannelSelectorType:
- case SkDisplacementMapEffect::kR_ChannelSelectorType:
- case SkDisplacementMapEffect::kG_ChannelSelectorType:
- case SkDisplacementMapEffect::kB_ChannelSelectorType:
- case SkDisplacementMapEffect::kA_ChannelSelectorType:
- return true;
- default:
- break;
- }
- return false;
-}
-
} // end namespace
///////////////////////////////////////////////////////////////////////////////
fXChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.readInt();
fYChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.readInt();
fScale = buffer.readScalar();
- buffer.validate(ChannelSelectorTypeIsValid(fXChannelSelector) &&
- ChannelSelectorTypeIsValid(fYChannelSelector) &&
- SkScalarIsFinite(fScale));
}
void SkDisplacementMapEffect::flatten(SkFlattenableWriteBuffer& buffer) const {
fDy = buffer.readScalar();
fSigma = buffer.readScalar();
fColor = buffer.readColor();
- buffer.validate(SkScalarIsFinite(fDx) &&
- SkScalarIsFinite(fDy) &&
- SkScalarIsFinite(fSigma));
}
void SkDropShadowImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const
point.fX = buffer.readScalar();
point.fY = buffer.readScalar();
point.fZ = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(point.fX) &&
- SkScalarIsFinite(point.fY) &&
- SkScalarIsFinite(point.fZ));
return point;
};
fCosInnerConeAngle = buffer.readScalar();
fConeScale = buffer.readScalar();
fS = readPoint3(buffer);
- buffer.validate(SkScalarIsFinite(fSpecularExponent) &&
- SkScalarIsFinite(fCosOuterConeAngle) &&
- SkScalarIsFinite(fCosInnerConeAngle) &&
- SkScalarIsFinite(fConeScale));
}
SkSpotLight(const SkPoint3& location, const SkPoint3& target, SkScalar specularExponent, SkScalar cosOuterConeAngle, SkScalar cosInnerConeAngle, SkScalar coneScale, const SkPoint3& s, const SkPoint3& color)
: INHERITED(color),
{
fLight = buffer.readFlattenableT<SkLight>();
fSurfaceScale = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fSurfaceScale));
}
void SkLightingImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
: INHERITED(buffer)
{
fKD = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fKD));
}
void SkDiffuseLightingImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
{
fKS = buffer.readScalar();
fShininess = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fKS) &&
- SkScalarIsFinite(fShininess));
}
void SkSpecularLightingImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
#include "SkMagnifierImageFilter.h"
#include "SkColorPriv.h"
#include "SkFlattenableBuffers.h"
-#include "SkValidationUtils.h"
////////////////////////////////////////////////////////////////////////////////
#if SK_SUPPORT_GPU
float height = buffer.readScalar();
fSrcRect = SkRect::MakeXYWH(x, y, width, height);
fInset = buffer.readScalar();
-
- buffer.validate(SkIsValidRect(fSrcRect) && SkScalarIsFinite(fInset));
}
// FIXME: implement single-input semantics
#endif
-namespace {
-
-bool TileModeIsValid(SkMatrixConvolutionImageFilter::TileMode tileMode) {
- switch (tileMode) {
- case SkMatrixConvolutionImageFilter::kClamp_TileMode:
- case SkMatrixConvolutionImageFilter::kRepeat_TileMode:
- case SkMatrixConvolutionImageFilter::kClampToBlack_TileMode:
- return true;
- default:
- break;
- }
- return false;
-}
-
-}
-
SkMatrixConvolutionImageFilter::SkMatrixConvolutionImageFilter(const SkISize& kernelSize, const SkScalar* kernel, SkScalar gain, SkScalar bias, const SkIPoint& target, TileMode tileMode, bool convolveAlpha, SkImageFilter* input)
: INHERITED(input),
fKernelSize(kernelSize),
fTarget.fY = buffer.readInt();
fTileMode = (TileMode) buffer.readInt();
fConvolveAlpha = buffer.readBool();
- buffer.validate(SkScalarIsFinite(fGain) &&
- SkScalarIsFinite(fBias) &&
- TileModeIsValid(fTileMode));
}
void SkMatrixConvolutionImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
#include "SkCanvas.h"
#include "SkDevice.h"
#include "SkFlattenableBuffers.h"
-#include "SkValidationUtils.h"
///////////////////////////////////////////////////////////////////////////////
bool hasModes = buffer.readBool();
if (hasModes) {
this->initAllocModes();
- int nbInputs = countInputs();
- SkASSERT(buffer.getArrayCount() == nbInputs * sizeof(fModes[0]));
+ SkASSERT(buffer.getArrayCount() == countInputs() * sizeof(fModes[0]));
buffer.readByteArray(fModes);
- for (int i = 0; i < nbInputs; ++i) {
- buffer.validate(SkIsValidMode((SkXfermode::Mode)fModes[i]));
- }
} else {
fModes = 0;
}
: INHERITED(buffer) {
fRadius.fWidth = buffer.readInt();
fRadius.fHeight = buffer.readInt();
- buffer.validate(SkScalarIsFinite(SkIntToScalar(fRadius.fWidth)) &&
- SkScalarIsFinite(SkIntToScalar(fRadius.fHeight)) &&
- (fRadius.fWidth >= 0) &&
- (fRadius.fHeight >= 0));
}
SkMorphologyImageFilter::SkMorphologyImageFilter(int radiusX, int radiusY, SkImageFilter* input)
SkOffsetImageFilter::SkOffsetImageFilter(SkFlattenableReadBuffer& buffer) : INHERITED(buffer) {
buffer.readPoint(&fOffset);
- buffer.validate(SkScalarIsFinite(fOffset.fX) &&
- SkScalarIsFinite(fOffset.fY));
}
#include "SkDevice.h"
#include "SkFlattenableBuffers.h"
#include "SkShader.h"
-#include "SkValidationUtils.h"
SkRectShaderImageFilter* SkRectShaderImageFilter::Create(SkShader* s, const SkRect& rect) {
SkASSERT(s);
: INHERITED(buffer) {
fShader = buffer.readFlattenableT<SkShader>();
buffer.readRect(&fRect);
- buffer.validate(SkIsValidRect(fRect));
}
void SkRectShaderImageFilter::flatten(SkFlattenableWriteBuffer& buffer) const {
SkDownSampleImageFilter::SkDownSampleImageFilter(SkFlattenableReadBuffer& buffer) : INHERITED(buffer) {
fScale = buffer.readScalar();
- buffer.validate(SkScalarIsFinite(fScale));
}
if (fErrorInDecoding) {
return false;
}
-
+
SkBitmapFactory::Target target;
(void)ComputeMinRowBytesAndSize(info, &target.fRowBytes);
-
+
SkBitmap tmp;
if (!init_from_info(&tmp, info, target.fRowBytes)) {
return false;
}
-
+
target.fAddr = tmp.getPixels();
fErrorInDecoding = !fDecodeProc(fData->data(), fData->size(), &info, &target);
if (fErrorInDecoding) {
*bitmap = tmp;
return true;
}
+
+
for (int x = 0; x < kBitmapSize; x += i) {
canvas.save();
canvas.translate(SkIntToScalar(x), SkIntToScalar(y));
- canvas.drawRect(SkRect::MakeXYWH(0, 0,
- SkIntToScalar(i),
- SkIntToScalar(i)), darkPaint);
- canvas.drawRect(SkRect::MakeXYWH(SkIntToScalar(i),
- 0,
- SkIntToScalar(i),
- SkIntToScalar(i)), lightPaint);
- canvas.drawRect(SkRect::MakeXYWH(0,
- SkIntToScalar(i),
- SkIntToScalar(i),
- SkIntToScalar(i)), lightPaint);
- canvas.drawRect(SkRect::MakeXYWH(SkIntToScalar(i),
- SkIntToScalar(i),
- SkIntToScalar(i),
- SkIntToScalar(i)), darkPaint);
+ canvas.drawRect(SkRect::MakeXYWH(0, 0, i, i), darkPaint);
+ canvas.drawRect(SkRect::MakeXYWH(i, 0, i, i), lightPaint);
+ canvas.drawRect(SkRect::MakeXYWH(0, i, i, i), lightPaint);
+ canvas.drawRect(SkRect::MakeXYWH(i, i, i, i), darkPaint);
canvas.restore();
}
}
SkPoint3 target(location.fX, location.fY, location.fZ);
// 3 ) large negative specular exponent value
SkScalar specularExponent = SkFloatToScalar(-1000);
-
+
SkPaint paint;
paint.setImageFilter(SkLightingImageFilter::CreateSpotLitSpecular(
location, target, specularExponent, SkFloatToScalar(180),
0xFFFFFFFF, SK_Scalar1, SK_Scalar1, SK_Scalar1,
new SkBitmapSource(bitmap)))->unref();
SkCanvas canvas(result);
- SkRect r = SkRect::MakeWH(SkIntToScalar(kBitmapSize),
- SkIntToScalar(kBitmapSize));
+ SkRect r = SkRect::MakeWH(kBitmapSize, kBitmapSize);
canvas.drawRect(r, paint);
}
{
// This tests for scale bringing width to 0
- SkSize scale = SkSize::Make(SkFloatToScalar(-0.001f), SK_Scalar1);
+ SkSize scale = SkSize::Make(SkFloatToScalar(-0.001), SK_Scalar1);
SkAutoTUnref<SkBicubicImageFilter> bicubic(
SkBicubicImageFilter::CreateMitchell(
scale, new SkBitmapSource(bitmap)));