diff --git a/core/jni/android/graphics/Paint.cpp b/core/jni/android/graphics/Paint.cpp index 376d6f331d725..cd5c734afc4d5 100644 --- a/core/jni/android/graphics/Paint.cpp +++ b/core/jni/android/graphics/Paint.cpp @@ -724,9 +724,9 @@ namespace PaintGlue { } static void setColorLong(jlong paintHandle, jlong colorSpaceHandle, - jfloat r, jfloat g, jfloat b, jfloat a) { + jlong colorLong) { + SkColor4f color = GraphicsJNI::convertColorLong(colorLong); sk_sp cs = GraphicsJNI::getNativeColorSpace(colorSpaceHandle); - SkColor4f color = SkColor4f{r, g, b, a}; reinterpret_cast(paintHandle)->setColor4f(color, cs.get()); } @@ -995,9 +995,9 @@ namespace PaintGlue { static void setShadowLayer(jlong paintHandle, jfloat radius, jfloat dx, jfloat dy, jlong colorSpaceHandle, - jfloat r, jfloat g, jfloat b, jfloat a) { + jlong colorLong) { + SkColor4f color = GraphicsJNI::convertColorLong(colorLong); sk_sp cs = GraphicsJNI::getNativeColorSpace(colorSpaceHandle); - SkColor4f color = SkColor4f{r, g, b, a}; Paint* paint = reinterpret_cast(paintHandle); if (radius <= 0) { @@ -1087,7 +1087,7 @@ static const JNINativeMethod methods[] = { {"nGetStyle","(J)I", (void*) PaintGlue::getStyle}, {"nSetStyle","(JI)V", (void*) PaintGlue::setStyle}, {"nSetColor","(JI)V", (void*) PaintGlue::setColor}, - {"nSetColor","(JJFFFF)V", (void*) PaintGlue::setColorLong}, + {"nSetColor","(JJJ)V", (void*) PaintGlue::setColorLong}, {"nSetAlpha","(JI)V", (void*) PaintGlue::setAlpha}, {"nGetStrokeWidth","(J)F", (void*) PaintGlue::getStrokeWidth}, {"nSetStrokeWidth","(JF)V", (void*) PaintGlue::setStrokeWidth}, @@ -1130,7 +1130,7 @@ static const JNINativeMethod methods[] = { {"nGetUnderlineThickness","(J)F", (void*) PaintGlue::getUnderlineThickness}, {"nGetStrikeThruPosition","(J)F", (void*) PaintGlue::getStrikeThruPosition}, {"nGetStrikeThruThickness","(J)F", (void*) PaintGlue::getStrikeThruThickness}, - {"nSetShadowLayer", "(JFFFJFFFF)V", (void*)PaintGlue::setShadowLayer}, + {"nSetShadowLayer", "(JFFFJJ)V", (void*)PaintGlue::setShadowLayer}, {"nHasShadowLayer", "(J)Z", (void*)PaintGlue::hasShadowLayer}, {"nEqualsForTextMeasurement", "(JJ)Z", (void*)PaintGlue::equalsForTextMeasurement}, }; diff --git a/graphics/java/android/graphics/Paint.java b/graphics/java/android/graphics/Paint.java index 966e171a9c04e..04385c628c35d 100644 --- a/graphics/java/android/graphics/Paint.java +++ b/graphics/java/android/graphics/Paint.java @@ -1066,12 +1066,8 @@ public class Paint { */ public void setColor(@ColorLong long color) { ColorSpace cs = Color.colorSpace(color); - float r = Color.red(color); - float g = Color.green(color); - float b = Color.blue(color); - float a = Color.alpha(color); - nSetColor(mNativePaint, cs.getNativeInstance(), r, g, b, a); + nSetColor(mNativePaint, cs.getNativeInstance(), color); mColor = color; } @@ -1501,11 +1497,7 @@ public class Paint { */ public void setShadowLayer(float radius, float dx, float dy, @ColorLong long shadowColor) { ColorSpace cs = Color.colorSpace(shadowColor); - float r = Color.red(shadowColor); - float g = Color.green(shadowColor); - float b = Color.blue(shadowColor); - float a = Color.alpha(shadowColor); - nSetShadowLayer(mNativePaint, radius, dx, dy, cs.getNativeInstance(), r, g, b, a); + nSetShadowLayer(mNativePaint, radius, dx, dy, cs.getNativeInstance(), shadowColor); mShadowLayerRadius = radius; mShadowLayerDx = dx; @@ -1533,6 +1525,7 @@ public class Paint { /** * Returns the blur radius of the shadow layer. * @see #setShadowLayer(float,float,float,int) + * @see #setShadowLayer(float,float,float,long) */ public float getShadowLayerRadius() { return mShadowLayerRadius; @@ -1541,6 +1534,7 @@ public class Paint { /** * Returns the x offset of the shadow layer. * @see #setShadowLayer(float,float,float,int) + * @see #setShadowLayer(float,float,float,long) */ public float getShadowLayerDx() { return mShadowLayerDx; @@ -1549,6 +1543,7 @@ public class Paint { /** * Returns the y offset of the shadow layer. * @see #setShadowLayer(float,float,float,int) + * @see #setShadowLayer(float,float,float,long) */ public float getShadowLayerDy() { return mShadowLayerDy; @@ -3138,7 +3133,7 @@ public class Paint { @CriticalNative private static native void nSetShadowLayer(long paintPtr, float radius, float dx, float dy, long colorSpaceHandle, - float r, float g, float b, float a); + @ColorLong long shadowColor); @CriticalNative private static native boolean nHasShadowLayer(long paintPtr); @CriticalNative @@ -3191,7 +3186,7 @@ public class Paint { private static native void nSetFilterBitmap(long paintPtr, boolean filter); @CriticalNative private static native void nSetColor(long paintPtr, long colorSpaceHandle, - float r, float g, float b, float a); + @ColorLong long color); @CriticalNative private static native void nSetColor(long paintPtr, @ColorInt int color); @CriticalNative