aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar Mike Klein <mtklein@chromium.org>2017-03-27 14:28:07 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2017-03-27 19:16:59 +0000
commitb338484d23d6fa4d6726361440962e75ec5855e0 (patch)
tree77baa4b18ff2bcfb7306eed8dff56eb5c7c7329d
parentb436655ad5c40a04b65c1642d0a0e781ce296c96 (diff)
quiet some annoying logging in unit tests
Noticed all these while adding some unit test support to ok. Change-Id: Ie33b96da95840628657211cb28b2134fd314a48a Reviewed-on: https://skia-review.googlesource.com/10202 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
-rw-r--r--src/pipe/SkPipeCanvas.cpp22
-rw-r--r--src/ports/SkOSLibrary_posix.cpp6
-rw-r--r--tests/CanvasTest.cpp21
-rw-r--r--tests/InterpolatorTest.cpp1
-rw-r--r--tools/gpu/gl/command_buffer/GLTestContext_command_buffer.cpp4
5 files changed, 13 insertions, 41 deletions
diff --git a/src/pipe/SkPipeCanvas.cpp b/src/pipe/SkPipeCanvas.cpp
index b05a643cb8..1473b843f6 100644
--- a/src/pipe/SkPipeCanvas.cpp
+++ b/src/pipe/SkPipeCanvas.cpp
@@ -240,7 +240,7 @@ SkCanvas::SaveLayerStrategy SkPipeCanvas::getSaveLayerStrategy(const SaveLayerRe
if (rec.fBackdrop) {
extra |= kHasBackdrop_SaveLayerMask;
}
-
+
writer.write32(pack_verb(SkPipeVerb::kSaveLayer, extra));
if (rec.fBounds) {
writer.writeRect(*rec.fBounds);
@@ -480,7 +480,7 @@ void SkPipeCanvas::onDrawBitmapLattice(const SkBitmap& bitmap, const Lattice& la
}
///////////////////////////////////////////////////////////////////////////////////////////////////
-
+
void SkPipeCanvas::onDrawImage(const SkImage* image, SkScalar left, SkScalar top,
const SkPaint* paint) {
unsigned extra = 0;
@@ -604,9 +604,9 @@ void SkPipeCanvas::onDrawText(const void* text, size_t byteLength, SkScalar x, S
void SkPipeCanvas::onDrawPosText(const void* text, size_t byteLength, const SkPoint pos[],
const SkPaint& paint) {
SkASSERT(byteLength);
-
+
bool compact = fits_in(byteLength, 24);
-
+
SkPipeWriter writer(this);
writer.write32(pack_verb(SkPipeVerb::kDrawPosText, compact ? (unsigned)byteLength : 0));
if (!compact) {
@@ -620,9 +620,9 @@ void SkPipeCanvas::onDrawPosText(const void* text, size_t byteLength, const SkPo
void SkPipeCanvas::onDrawPosTextH(const void* text, size_t byteLength, const SkScalar xpos[],
SkScalar constY, const SkPaint& paint) {
SkASSERT(byteLength);
-
+
bool compact = fits_in(byteLength, 24);
-
+
SkPipeWriter writer(this);
writer.write32(pack_verb(SkPipeVerb::kDrawPosTextH, compact ? (unsigned)byteLength : 0));
if (!compact) {
@@ -661,13 +661,13 @@ void SkPipeCanvas::onDrawTextOnPath(const void* text, size_t byteLength, const S
void SkPipeCanvas::onDrawTextRSXform(const void* text, size_t byteLength, const SkRSXform xform[],
const SkRect* cull, const SkPaint& paint) {
SkASSERT(byteLength);
-
+
bool compact = fits_in(byteLength, 23);
unsigned extra = compact ? (byteLength << 1) : 0;
if (cull) {
extra |= 1;
}
-
+
SkPipeWriter writer(this);
writer.write32(pack_verb(SkPipeVerb::kDrawTextRSXform, extra));
if (!compact) {
@@ -878,8 +878,10 @@ int SkPipeDeduper::findOrDefinePicture(SkPicture* picture) {
ASSERT_FITS_IN(index, kObjectDefinitionBits);
fStream->write32(pack_verb(SkPipeVerb::kEndPicture, index));
- SkDebugf(" definePicture(%d) %d\n",
- index - 1, SkToU32(fStream->bytesWritten() - prevWritten));
+ if (show_deduper_traffic) {
+ SkDebugf(" definePicture(%d) %d\n",
+ index - 1, SkToU32(fStream->bytesWritten() - prevWritten));
+ }
return index;
}
diff --git a/src/ports/SkOSLibrary_posix.cpp b/src/ports/SkOSLibrary_posix.cpp
index 6372a8122a..901ee22f3c 100644
--- a/src/ports/SkOSLibrary_posix.cpp
+++ b/src/ports/SkOSLibrary_posix.cpp
@@ -12,11 +12,7 @@
#include <dlfcn.h>
void* DynamicLoadLibrary(const char* libraryName) {
- void* result = dlopen(libraryName, RTLD_LAZY);
- if (!result) {
- SkDebugf("Error loading %s {\n %s\n}\n", libraryName, dlerror());
- }
- return result;
+ return dlopen(libraryName, RTLD_LAZY);
}
void* GetProcedureAddress(void* library, const char* functionName) {
diff --git a/tests/CanvasTest.cpp b/tests/CanvasTest.cpp
index f17ff42e48..8327b555cc 100644
--- a/tests/CanvasTest.cpp
+++ b/tests/CanvasTest.cpp
@@ -802,27 +802,6 @@ DEF_TEST(PaintFilterCanvas_ConsistentState, reporter) {
///////////////////////////////////////////////////////////////////////////////////////////////////
-#include "SkDeferredCanvas.h"
-#include "SkDumpCanvas.h"
-
-DEF_TEST(DeferredCanvas, r) {
- SkDebugfDumper dumper;
- SkDumpCanvas dumpC(&dumper);
-
- SkDeferredCanvas canvas(&dumpC, SkDeferredCanvas::kEager);
-
- SkPaint paint;
-// paint.setShader(SkShader::MakeColorShader(SK_ColorRED));
-
- canvas.save();
- canvas.clipRect(SkRect::MakeWH(55, 55));
- canvas.translate(10, 20);
- canvas.drawRect(SkRect::MakeWH(50, 50), paint);
- canvas.restore();
-}
-
-///////////////////////////////////////////////////////////////////////////////////////////////////
-
#include "SkCanvasStack.h"
#include "SkNWayCanvas.h"
diff --git a/tests/InterpolatorTest.cpp b/tests/InterpolatorTest.cpp
index f520ebb700..8ba1bdac56 100644
--- a/tests/InterpolatorTest.cpp
+++ b/tests/InterpolatorTest.cpp
@@ -87,6 +87,5 @@ DEF_TEST(Interpolator, reporter) {
SkUnitCubicInterp(val, t[0], t[1], t[2], t[3])));
}
++i;
- SkDebugf("\n");
}
}
diff --git a/tools/gpu/gl/command_buffer/GLTestContext_command_buffer.cpp b/tools/gpu/gl/command_buffer/GLTestContext_command_buffer.cpp
index 56bc5da3d6..82888d8984 100644
--- a/tools/gpu/gl/command_buffer/GLTestContext_command_buffer.cpp
+++ b/tools/gpu/gl/command_buffer/GLTestContext_command_buffer.cpp
@@ -108,8 +108,6 @@ static void load_command_buffer_functions() {
gfCreateContext && gfDestroyContext && gfMakeCurrent &&
gfSwapBuffers && gfGetProcAddress;
- } else {
- SkDebugf("Could not load %s.\n", libName);
}
}
}
@@ -159,8 +157,6 @@ CommandBufferGLTestContext::CommandBufferGLTestContext(CommandBufferGLTestContex
load_command_buffer_once();
if (!gfFunctionsLoadedSuccessfully) {
- static SkOnce once;
- once([] { SkDebugf("Command Buffer: Could not load EGL functions.\n"); });
return;
}