Validate SkSpecialSurface raster info
BUG=chromium:716311
Change-Id: I01ea2e77ba8920f735395dd46ef2cea78a858308
Reviewed-on: https://skia-review.googlesource.com/15230
Reviewed-by: Mike Reed <reed@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
diff --git a/src/image/SkSurface_Raster.cpp b/src/image/SkSurface_Raster.cpp
index c4796c4..1ee3d7e 100644
--- a/src/image/SkSurface_Raster.cpp
+++ b/src/image/SkSurface_Raster.cpp
@@ -11,12 +11,8 @@
#include "SkDevice.h"
#include "SkMallocPixelRef.h"
-static const size_t kIgnoreRowBytesValue = (size_t)~0;
-
class SkSurface_Raster : public SkSurface_Base {
public:
- static bool Valid(const SkImageInfo&, size_t rb = kIgnoreRowBytesValue);
-
SkSurface_Raster(const SkImageInfo&, void*, size_t rb,
void (*releaseProc)(void* pixels, void* context), void* context,
const SkSurfaceProps*);
@@ -39,7 +35,7 @@
///////////////////////////////////////////////////////////////////////////////
-bool SkSurface_Raster::Valid(const SkImageInfo& info, size_t rowBytes) {
+bool SkSurfaceValidateRasterInfo(const SkImageInfo& info, size_t rowBytes) {
if (info.isEmpty()) {
return false;
}
@@ -186,7 +182,7 @@
if (nullptr == releaseProc) {
context = nullptr;
}
- if (!SkSurface_Raster::Valid(info, rb)) {
+ if (!SkSurfaceValidateRasterInfo(info, rb)) {
return nullptr;
}
if (nullptr == pixels) {
@@ -203,7 +199,7 @@
sk_sp<SkSurface> SkSurface::MakeRaster(const SkImageInfo& info, size_t rowBytes,
const SkSurfaceProps* props) {
- if (!SkSurface_Raster::Valid(info)) {
+ if (!SkSurfaceValidateRasterInfo(info)) {
return nullptr;
}