AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

make gm background colors 565 compatible

Browse Source 
Change a batch of GM tests to convert their background color
so that it is representable in 8888 and 565.

Enable portable text in those same tests to minimize platform
differences.

In a couple of bitmap tests, use portable typefaces instead of
choosing 'Times' which may or may not be available on the platform.

R=borenet@google.com

Review URL: https://codereview.chromium.org/1176243006
This commit is contained in:
caryclark 2015-06-15 05:09:59 -07:00 committed by Commit bot
parent b5b50a07ab
commit be7f768a35
38 changed files with 67 additions and 65 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gm/anisotropic.cpp
View File

@ -13,7 +13,7 @@ namespace skiagm {
class AnisotropicGM : public GM {
public:
AnisotropicGM() : fFilterQuality(kHigh_SkFilterQuality) {
this->setBGColor(0xFFCCCCCC);
this->setBGColor(sk_tool_utils::color_to_565(0xFFCCCCCC));
}
protected:

2
gm/arcofzorro.cpp
View File

@ -16,7 +16,7 @@ namespace skiagm {
class ArcOfZorroGM : public GM {
public:
ArcOfZorroGM() {
this->setBGColor(0xFFCCCCCC);
this->setBGColor(sk_tool_utils::color_to_565(0xFFCCCCCC));
}
protected:

2
gm/bigblurs.cpp
View File

@ -19,7 +19,7 @@ namespace skiagm {
class BigBlursGM : public GM {
public:
BigBlursGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/bigmatrix.cpp
View File

@ -15,7 +15,7 @@ namespace skiagm {
class BigMatrixGM : public GM {
public:
BigMatrixGM() {
this->setBGColor(0xFF66AA99);
this->setBGColor(sk_tool_utils::color_to_565(0xFF66AA99));
}
protected:

6
gm/bitmapcopy.cpp
View File

@ -48,7 +48,7 @@ public:
SkBitmap fDst[NUM_CONFIGS];
BitmapCopyGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -75,9 +75,9 @@ protected:
src.copyTo(&fDst[i], gColorTypes[i]);
}
canvas->clear(0xFFDDDDDD);
canvas->clear(sk_tool_utils::color_to_565(0xFFDDDDDD));
paint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&paint);
sk_tool_utils::set_portable_typeface_always(&paint);
SkScalar width = SkIntToScalar(40);
SkScalar height = SkIntToScalar(40);

4
gm/bitmapfilters.cpp
View File

@ -53,7 +53,7 @@ static SkScalar draw_row(SkCanvas* canvas, const SkBitmap& bm) {
const int scale = 32;
paint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&paint);
sk_tool_utils::set_portable_typeface_always(&paint);
const char* name = sk_tool_utils::colortype_name(bm.colorType());
canvas->drawText(name, strlen(name), x, SkIntToScalar(bm.height())*scale*5/8,
paint);
@ -80,7 +80,7 @@ public:
SkBitmap fBM8, fBM4444, fBM16, fBM32;
FilterGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/blurs.cpp
View File

@ -12,7 +12,7 @@
class BlursGM : public skiagm::GM {
public:
BlursGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

3
gm/bmpfilterqualityrepeat.cpp
View File

@ -14,7 +14,7 @@
// the bmp filter respects the repeat mode at the tile seams.
class BmpFilterQualityRepeat : public skiagm::GM {
public:
BmpFilterQualityRepeat() { this->setBGColor(0xFFCCBBAA); }
BmpFilterQualityRepeat() { this->setBGColor(sk_tool_utils::color_to_565(0xFFCCBBAA)); }
protected:
@ -51,6 +51,7 @@ protected:
for (size_t q = 0; q < SK_ARRAY_COUNT(kQualities); ++q) {
SkPaint paint;
sk_tool_utils::set_portable_typeface_always(&paint);
paint.setFilterQuality(kQualities[q].fQuality);
SkPaint bmpPaint(paint);
SkMatrix lm = SkMatrix::I();

2
gm/clipdrawdraw.cpp
View File

@ -20,7 +20,7 @@ namespace skiagm {
// can leave 1 pixel wide remnants of the first rect.
class ClipDrawDrawGM : public GM {
public:
ClipDrawDrawGM() { this->setBGColor(0xFFCCCCCC); }
ClipDrawDrawGM() { this->setBGColor(sk_tool_utils::color_to_565(0xFFCCCCCC)); }
protected:
SkString onShortName() override { return SkString("clipdrawdraw"); }

2
gm/colormatrix.cpp
View File

@ -49,7 +49,7 @@ class ColorMatrixGM : public GM {
public:
ColorMatrixGM() {
this->setBGColor(0xFF808080);
this->setBGColor(sk_tool_utils::color_to_565(0xFF808080));
}
protected:

4
gm/complexclip3.cpp
View File

@ -17,7 +17,7 @@ class ComplexClip3GM : public GM {
public:
ComplexClip3GM(bool doSimpleClipFirst)
: fDoSimpleClipFirst(doSimpleClipFirst) {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -49,7 +49,7 @@ protected:
SkPaint paint;
paint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&paint);
sk_tool_utils::set_portable_typeface_always(&paint);
paint.setTextSize(SkIntToScalar(20));
static const struct {

3
gm/constcolorprocessor.cpp
View File

@ -25,7 +25,7 @@ namespace skiagm {
class ConstColorProcessor : public GM {
public:
ConstColorProcessor() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -124,6 +124,7 @@ protected:
// Draw labels for the input to the processor and the processor to the right of
// the test rect. The input label appears above the processor label.
SkPaint labelPaint;
sk_tool_utils::set_portable_typeface_always(&labelPaint);
labelPaint.setAntiAlias(true);
labelPaint.setTextSize(10.f);
SkString inputLabel;

4
gm/dcshader.cpp
View File

@ -110,7 +110,7 @@ bool DCShader::asFragmentProcessor(GrContext*, const SkPaint& paint, const SkMat
class DCShaderGM : public GM {
public:
DCShaderGM() {
this->setBGColor(0xFFAABBCC);
this->setBGColor(sk_tool_utils::color_to_565(0xFFAABBCC));
}
~DCShaderGM() override {
@ -210,7 +210,7 @@ protected:
}
virtual void setFont(SkPaint* paint) {
sk_tool_utils::set_portable_typeface(paint);
sk_tool_utils::set_portable_typeface_always(paint);
}
virtual const char* text() const { return "Hello, Skia!"; }

14
gm/downsamplebitmap.cpp
View File

@ -31,7 +31,7 @@ static void make_checker(SkBitmap* bm, int size, int numChecks) {
}
static void setTypeface(SkPaint* paint, const char name[], SkTypeface::Style style) {
sk_tool_utils::set_portable_typeface(paint, name, style);
sk_tool_utils::set_portable_typeface_always(paint, name, style);
}
class DownsampleBitmapGM : public skiagm::GM {
@ -44,7 +44,7 @@ public:
DownsampleBitmapGM(SkFilterQuality filterQuality)
: fFilterQuality(filterQuality)
{
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
fBitmapMade = false;
}
@ -126,13 +126,13 @@ class DownsampleBitmapTextGM: public DownsampleBitmapGM {
paint.setSubpixelText(true);
paint.setTextSize(fTextSize);
setTypeface(&paint, "Times", SkTypeface::kNormal);
setTypeface(&paint, "serif", SkTypeface::kNormal);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 1.2f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kBold);
setTypeface(&paint, "serif", SkTypeface::kBold);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 2.4f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kItalic);
setTypeface(&paint, "serif", SkTypeface::kItalic);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 3.6f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kBoldItalic);
setTypeface(&paint, "serif", SkTypeface::kBoldItalic);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 4.8f*fTextSize, paint);
}
private:
@ -202,7 +202,7 @@ public:
SkBitmap fBM;
ShowMipLevels() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
make_checker(&fBM, 512, 256);
}

2
gm/drawlooper.cpp
View File

@ -19,7 +19,7 @@
class DrawLooperGM : public skiagm::GM {
public:
DrawLooperGM() : fLooper(NULL) {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
virtual ~DrawLooperGM() {

2
gm/emptypath.cpp
View File

@ -92,7 +92,7 @@ protected:
SkColor color = rand.nextU();
color = 0xff000000 | color; // force solid
sk_tool_utils::color_to_565(&color);
color = sk_tool_utils::color_to_565(color);
this->drawEmpty(canvas, color, rect,
gStyles[style].fStyle, gFills[fill].fFill);

2
gm/filltypes.cpp
View File

@ -13,7 +13,7 @@ class FillTypeGM : public GM {
SkPath fPath;
public:
FillTypeGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
void makePath() {

12
gm/filterbitmap.cpp
View File

@ -14,7 +14,7 @@
#include "SkTypeface.h"
static void setTypeface(SkPaint* paint, const char name[], SkTypeface::Style style) {
sk_tool_utils::set_portable_typeface(paint, name, style);
sk_tool_utils::set_portable_typeface_always(paint, name, style);
}
static SkSize computeSize(const SkBitmap& bm, const SkMatrix& mat) {
@ -66,7 +66,7 @@ public:
FilterBitmapGM()
{
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -124,13 +124,13 @@ class FilterBitmapTextGM: public FilterBitmapGM {
paint.setSubpixelText(true);
paint.setTextSize(fTextSize);
setTypeface(&paint, "Times", SkTypeface::kNormal);
setTypeface(&paint, "serif", SkTypeface::kNormal);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 1.2f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kBold);
setTypeface(&paint, "serif", SkTypeface::kBold);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 2.4f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kItalic);
setTypeface(&paint, "serif", SkTypeface::kItalic);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 3.6f*fTextSize, paint);
setTypeface(&paint, "Times", SkTypeface::kBoldItalic);
setTypeface(&paint, "serif", SkTypeface::kBoldItalic);
canvas.drawText("Hamburgefons", 12, fTextSize/2, 4.8f*fTextSize, paint);
}
private:

2
gm/filterfastbounds.cpp
View File

@ -150,7 +150,7 @@ static void create_paints(SkImageFilter* source, SkTArray<SkPaint>* paints) {
class ImageFilterFastBoundGM : public GM {
public:
ImageFilterFastBoundGM() {
this->setBGColor(0xFFCCCCCC);
this->setBGColor(sk_tool_utils::color_to_565(0xFFCCCCCC));
}
protected:

2
gm/filterindiabox.cpp
View File

@ -62,7 +62,7 @@ public:
SkString fName;
FilterIndiaBoxGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
FilterIndiaBoxGM(const char filename[]) : fFilename(filename) {

2
gm/gradientDirtyLaundry.cpp
View File

@ -66,7 +66,7 @@ static const GradMaker gGradMakers[] = {
class GradientsGM : public GM {
public:
GradientsGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/gradient_matrix.cpp
View File

@ -116,7 +116,7 @@ namespace skiagm {
class GradientMatrixGM : public GM {
public:
GradientMatrixGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

4
gm/gradients.cpp
View File

@ -101,7 +101,7 @@ static const GradMaker gGradMakers[] = {
class GradientsGM : public GM {
public:
GradientsGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -156,7 +156,7 @@ DEF_GM( return new GradientsGM; )
class GradientsLocalPerspectiveGM : public GM {
public:
GradientsLocalPerspectiveGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/gradients_2pt_conical.cpp
View File

@ -306,7 +306,7 @@ enum GradCaseType { // these must match the order in gGradCases
class ConicalGradientsGM : public GM {
public:
ConicalGradientsGM(GradCaseType gradCaseType) {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
fName.printf("gradients_2pt_conical_%s", gGradCases[gradCaseType].fName);
fGradCaseType = gradCaseType;
}

2
gm/gradients_no_texture.cpp
View File

@ -81,7 +81,7 @@ static const GradMaker gGradMakers[] = {
class GradientsNoTextureGM : public GM {
public:
GradientsNoTextureGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

4
gm/internal_links.cpp
View File

@ -20,7 +20,7 @@ namespace skiagm {
class InternalLinksGM : public GM {
public:
InternalLinksGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -61,7 +61,7 @@ private:
canvas->drawRect(rect, paint);
paint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&paint);
sk_tool_utils::set_portable_typeface_always(&paint);
paint.setTextSize(SkIntToScalar(25));
paint.setColor(SK_ColorBLACK);
canvas->drawText(text, strlen(text), x, y, paint);

2
gm/modecolorfilters.cpp
View File

@ -56,7 +56,7 @@ static SkShader* make_bg_shader(int checkSize) {
class ModeColorFilterGM : public GM {
public:
ModeColorFilterGM() {
this->setBGColor(0xFF303030);
this->setBGColor(sk_tool_utils::color_to_565(0xFF303030));
}
protected:

2
gm/nested.cpp
View File

@ -15,7 +15,7 @@ namespace skiagm {
class NestedGM : public GM {
public:
NestedGM(bool doAA, bool flipped) : fDoAA(doAA), fFlipped(flipped) {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/pathinterior.cpp
View File

@ -21,7 +21,7 @@ static SkRect inset(const SkRect& r) {
class PathInteriorGM : public skiagm::GM {
public:
PathInteriorGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

2
gm/rrects.cpp
View File

@ -27,7 +27,7 @@ public:
kEffect_Type,
};
RRectGM(Type type) : fType(type) {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
this->setUpRRects();
}

4
gm/shadertext.cpp
View File

@ -87,7 +87,7 @@ static const GradMaker gGradMakers[] = {
class ShaderTextGM : public GM {
public:
ShaderTextGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -144,7 +144,7 @@ protected:
SkPaint paint;
paint.setDither(true);
paint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&paint);
sk_tool_utils::set_portable_typeface_always(&paint);
paint.setTextSize(SkIntToScalar(pointSize));
canvas->save();

8
gm/shadertext2.cpp
View File

@ -43,7 +43,7 @@ struct LabeledMatrix {
class ShaderText2GM : public GM {
public:
ShaderText2GM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -90,13 +90,13 @@ protected:
SkPaint fillPaint;
fillPaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&fillPaint);
sk_tool_utils::set_portable_typeface_always(&fillPaint);
fillPaint.setTextSize(SkIntToScalar(kPointSize));
fillPaint.setFilterQuality(kLow_SkFilterQuality);
SkPaint outlinePaint;
outlinePaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&outlinePaint);
sk_tool_utils::set_portable_typeface_always(&outlinePaint);
outlinePaint.setTextSize(SkIntToScalar(kPointSize));
outlinePaint.setStyle(SkPaint::kStroke_Style);
outlinePaint.setStrokeWidth(0.f);
@ -112,7 +112,7 @@ protected:
SkPaint labelPaint;
labelPaint.setColor(0xff000000);
labelPaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&labelPaint);
sk_tool_utils::set_portable_typeface_always(&labelPaint);
labelPaint.setTextSize(12.f);
canvas->translate(15.f, 15.f);

6
gm/shadertext3.cpp
View File

@ -48,7 +48,7 @@ static const int kPointSize = 300;
class ShaderText3GM : public GM {
public:
ShaderText3GM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:
@ -73,7 +73,7 @@ protected:
SkPaint outlinePaint;
outlinePaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&outlinePaint);
sk_tool_utils::set_portable_typeface_always(&outlinePaint);
outlinePaint.setTextSize(SkIntToScalar(kPointSize));
outlinePaint.setStyle(SkPaint::kStroke_Style);
outlinePaint.setStrokeWidth(0.f);
@ -107,7 +107,7 @@ protected:
SkPaint fillPaint;
fillPaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&fillPaint);
sk_tool_utils::set_portable_typeface_always(&fillPaint);
fillPaint.setTextSize(SkIntToScalar(kPointSize));
fillPaint.setFilterQuality(kLow_SkFilterQuality);
fillPaint.setShader(shader);

2
gm/shadows.cpp
View File

@ -30,7 +30,7 @@ public:
protected:
void onOnceBeforeDraw() override {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
fCirclePath.addCircle(SkIntToScalar(20), SkIntToScalar(20), SkIntToScalar(10) );
fRect.set(SkIntToScalar(10), SkIntToScalar(10),
SkIntToScalar(30), SkIntToScalar(30));

4
gm/simpleaaclip.cpp
View File

@ -64,7 +64,7 @@ protected:
fBasePath.addRoundRect(fBase, SkIntToScalar(5), SkIntToScalar(5));
fRectPath.addRoundRect(fRect, SkIntToScalar(5), SkIntToScalar(5));
INHERITED::setBGColor(0xFFDDDDDD);
INHERITED::setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
void buildRgn(SkAAClip* clip, SkRegion::Op op) {
@ -156,7 +156,7 @@ protected:
SkPaint textPaint;
textPaint.setAntiAlias(true);
sk_tool_utils::set_portable_typeface(&textPaint);
sk_tool_utils::set_portable_typeface_always(&textPaint);
textPaint.setTextSize(SK_Scalar1*24);
int xOff = 0;

2
gm/tinybitmap.cpp
View File

@ -32,7 +32,7 @@ static SkBitmap make_bitmap() {
class TinyBitmapGM : public GM {
public:
TinyBitmapGM() {
this->setBGColor(0xFFDDDDDD);
this->setBGColor(sk_tool_utils::color_to_565(0xFFDDDDDD));
}
protected:

6
tools/sk_tool_utils.cpp
View File

@ -34,10 +34,10 @@ const char* colortype_name(SkColorType ct) {
}
}
void color_to_565(SkColor* color) {
SkPMColor pmColor = SkPreMultiplyColor(*color);
SkColor color_to_565(SkColor color) {
SkPMColor pmColor = SkPreMultiplyColor(color);
U16CPU color16 = SkPixel32ToPixel16(pmColor);
*color = SkPixel16ToColor(color16);
return SkPixel16ToColor(color16);
}
SkTypeface* create_portable_typeface(const char* name, SkTypeface::Style style) {

2
tools/sk_tool_utils.h
View File

@ -24,7 +24,7 @@ class SkTextBlobBuilder;
namespace sk_tool_utils {
const char* colortype_name(SkColorType);
void color_to_565(SkColor* color);
SkColor color_to_565(SkColor color);
/**
* Sets the paint to use a platform-independent text renderer if FLAGS_portableFonts is set.