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

Side by Side Diff: third_party/WebKit/Source/platform/graphics/GeneratedImage.cpp

Issue 1789063005: Add sk_sp helpers and switch Blink SkShader clients to the new APIs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fromSkSp, review comments Created 4 years, 9 months 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 29 matching lines...) Expand all
40 void GeneratedImage::drawPattern(GraphicsContext& destContext, const FloatRect& srcRect, const FloatSize& scale, 40 void GeneratedImage::drawPattern(GraphicsContext& destContext, const FloatRect& srcRect, const FloatSize& scale,
41 const FloatPoint& phase, SkXfermode::Mode compositeOp, const FloatRect& dest Rect, 41 const FloatPoint& phase, SkXfermode::Mode compositeOp, const FloatRect& dest Rect,
42 const FloatSize& repeatSpacing) 42 const FloatSize& repeatSpacing)
43 { 43 {
44 FloatRect tileRect = srcRect; 44 FloatRect tileRect = srcRect;
45 tileRect.expand(FloatSize(repeatSpacing)); 45 tileRect.expand(FloatSize(repeatSpacing));
46 46
47 SkPictureBuilder builder(tileRect, nullptr, &destContext); 47 SkPictureBuilder builder(tileRect, nullptr, &destContext);
48 builder.context().beginRecording(tileRect); 48 builder.context().beginRecording(tileRect);
49 drawTile(builder.context(), srcRect); 49 drawTile(builder.context(), srcRect);
50 RefPtr<const SkPicture> tilePicture = builder.endRecording(); 50 RefPtr<SkPicture> tilePicture = builder.endRecording();
51 51
52 AffineTransform patternTransform; 52 AffineTransform patternTransform;
53 patternTransform.translate(phase.x(), phase.y()); 53 patternTransform.translate(phase.x(), phase.y());
54 patternTransform.scale(scale.width(), scale.height()); 54 patternTransform.scale(scale.width(), scale.height());
55 patternTransform.translate(tileRect.x(), tileRect.y()); 55 patternTransform.translate(tileRect.x(), tileRect.y());
56 56
57 RefPtr<Pattern> picturePattern = Pattern::createPicturePattern(tilePicture); 57 RefPtr<Pattern> picturePattern = Pattern::createPicturePattern(tilePicture.r elease());
Stephen White 2016/03/18 14:48:27 Out of curiosity, is this release() necessary? Or
f(malita) 2016/03/18 15:11:53 The latter: it would work fine without a release,
58 picturePattern->setPatternSpaceTransform(patternTransform); 58 picturePattern->setPatternSpaceTransform(patternTransform);
59 59
60 SkPaint fillPaint = destContext.fillPaint(); 60 SkPaint fillPaint = destContext.fillPaint();
61 picturePattern->applyToPaint(fillPaint); 61 picturePattern->applyToPaint(fillPaint);
62 fillPaint.setColor(SK_ColorBLACK); 62 fillPaint.setColor(SK_ColorBLACK);
63 fillPaint.setXfermodeMode(compositeOp); 63 fillPaint.setXfermodeMode(compositeOp);
64 64
65 destContext.drawRect(destRect, fillPaint); 65 destContext.drawRect(destRect, fillPaint);
66 } 66 }
67 67
68 PassRefPtr<SkImage> GeneratedImage::imageForCurrentFrame() 68 PassRefPtr<SkImage> GeneratedImage::imageForCurrentFrame()
69 { 69 {
70 return nullptr; 70 return nullptr;
71 } 71 }
72 72
73 } // namespace blink 73 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698