Remove unused ctor in GrTextureDomain
Change-Id: I82b4add39f87133157f94745902df0b3a1a23680 Reviewed-on: https://skia-review.googlesource.com/22063 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
This commit is contained in:
parent
a8565e502d
commit
bca23b8634
@ -30,44 +30,6 @@ static bool can_ignore_rect(GrTextureProxy* proxy, const SkRect& domain) {
|
|||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
static bool can_ignore_rect(GrTexture* tex, const SkRect& domain) {
|
|
||||||
// This logic is relying on the instantiated size of 'tex'. In the deferred world it
|
|
||||||
// will have to change so this logic only fires for kExact texture proxies. This shouldn't
|
|
||||||
// change the actual behavior of Ganesh since shaders shouldn't be accessing pixels outside
|
|
||||||
// of the content rectangle.
|
|
||||||
const SkIRect kFullRect = SkIRect::MakeWH(tex->width(), tex->height());
|
|
||||||
|
|
||||||
return domain.contains(kFullRect);
|
|
||||||
}
|
|
||||||
|
|
||||||
GrTextureDomain::GrTextureDomain(GrTexture* tex, const SkRect& domain, Mode mode, int index)
|
|
||||||
: fMode(mode)
|
|
||||||
, fIndex(index) {
|
|
||||||
|
|
||||||
if (kIgnore_Mode == fMode) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (kClamp_Mode == mode && can_ignore_rect(tex, domain)) {
|
|
||||||
fMode = kIgnore_Mode;
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
const SkRect kFullRect = SkRect::MakeIWH(tex->width(), tex->height());
|
|
||||||
|
|
||||||
// We don't currently handle domains that are empty or don't intersect the texture.
|
|
||||||
// It is OK if the domain rect is a line or point, but it should not be inverted. We do not
|
|
||||||
// handle rects that do not intersect the [0..1]x[0..1] rect.
|
|
||||||
SkASSERT(domain.fLeft <= domain.fRight);
|
|
||||||
SkASSERT(domain.fTop <= domain.fBottom);
|
|
||||||
fDomain.fLeft = SkScalarPin(domain.fLeft, 0.0f, kFullRect.fRight);
|
|
||||||
fDomain.fRight = SkScalarPin(domain.fRight, fDomain.fLeft, kFullRect.fRight);
|
|
||||||
fDomain.fTop = SkScalarPin(domain.fTop, 0.0f, kFullRect.fBottom);
|
|
||||||
fDomain.fBottom = SkScalarPin(domain.fBottom, fDomain.fTop, kFullRect.fBottom);
|
|
||||||
SkASSERT(fDomain.fLeft <= fDomain.fRight);
|
|
||||||
SkASSERT(fDomain.fTop <= fDomain.fBottom);
|
|
||||||
}
|
|
||||||
|
|
||||||
GrTextureDomain::GrTextureDomain(GrTextureProxy* proxy, const SkRect& domain, Mode mode, int index)
|
GrTextureDomain::GrTextureDomain(GrTextureProxy* proxy, const SkRect& domain, Mode mode, int index)
|
||||||
: fMode(mode)
|
: fMode(mode)
|
||||||
, fIndex(index) {
|
, fIndex(index) {
|
||||||
|
@ -53,8 +53,6 @@ public:
|
|||||||
* @param index Pass a value >= 0 if using multiple texture domains in the same effect.
|
* @param index Pass a value >= 0 if using multiple texture domains in the same effect.
|
||||||
* It is used to keep inserted variables from causing name collisions.
|
* It is used to keep inserted variables from causing name collisions.
|
||||||
*/
|
*/
|
||||||
GrTextureDomain(GrTexture*, const SkRect& domain, Mode, int index = -1);
|
|
||||||
|
|
||||||
GrTextureDomain(GrTextureProxy*, const SkRect& domain, Mode, int index = -1);
|
GrTextureDomain(GrTextureProxy*, const SkRect& domain, Mode, int index = -1);
|
||||||
|
|
||||||
const SkRect& domain() const { return fDomain; }
|
const SkRect& domain() const { return fDomain; }
|
||||||
|
Loading…
Reference in New Issue
Block a user