Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 1d8aae3b: am db672b03: Merge "Change how Java Bitmaps are accessed in a few places" into mnc-dev

Browse Source 
* commit '1d8aae3b1924b2340250bdffe491819a26eea5fe':
  Change how Java Bitmaps are accessed in a few places
This commit is contained in:
John Reck
2015-04-30 21:20:51 +00:00
committed by Android Git Automerger
parent c9122d6fe2 1d8aae3b19
commit 0385dc8d12
14 changed files with 85 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/jni/android/graphics/BitmapFactory.cpp
View File

@@ -261,7 +261,7 @@ static jobject doDecode(JNIEnv* env, SkStreamRewindable* stream, jobject padding
SkBitmap* outputBitmap = NULL; SkBitmap* outputBitmap = NULL;
unsigned int existingBufferSize = 0; unsigned int existingBufferSize = 0;
if (javaBitmap != NULL) { if (javaBitmap != NULL) {
outputBitmap = GraphicsJNI::getSkBitmap(env, javaBitmap); outputBitmap = GraphicsJNI::getSkBitmapDeprecated(env, javaBitmap);
if (outputBitmap->isImmutable()) { if (outputBitmap->isImmutable()) {
ALOGW("Unable to reuse an immutable bitmap as an image decoder target."); ALOGW("Unable to reuse an immutable bitmap as an image decoder target.");
javaBitmap = NULL; javaBitmap = NULL;

2
core/jni/android/graphics/BitmapRegionDecoder.cpp
View File

@@ -217,7 +217,7 @@ static jobject nativeDecodeRegion(JNIEnv* env, jobject, jlong brdHandle,
if (tileBitmap != NULL) { if (tileBitmap != NULL) {
// Re-use bitmap. // Re-use bitmap.
bitmap = GraphicsJNI::getSkBitmap(env, tileBitmap); bitmap = GraphicsJNI::getSkBitmapDeprecated(env, tileBitmap);
} }
if (bitmap == NULL) { if (bitmap == NULL) {
bitmap = new SkBitmap; bitmap = new SkBitmap;

15
core/jni/android/graphics/Graphics.cpp
View File

@@ -338,7 +338,7 @@ SkColorType GraphicsJNI::legacyBitmapConfigToColorType(jint legacyConfig) {
return static_cast<SkColorType>(gConfig2ColorType[legacyConfig]); return static_cast<SkColorType>(gConfig2ColorType[legacyConfig]);
} }
SkBitmap* GraphicsJNI::getSkBitmap(JNIEnv* env, jobject bitmap) { SkBitmap* GraphicsJNI::getSkBitmapDeprecated(JNIEnv* env, jobject bitmap) {
SkASSERT(env); SkASSERT(env);
SkASSERT(bitmap); SkASSERT(bitmap);
SkASSERT(env->IsInstanceOf(bitmap, gBitmap_class)); SkASSERT(env->IsInstanceOf(bitmap, gBitmap_class));
@@ -348,6 +348,19 @@ SkBitmap* GraphicsJNI::getSkBitmap(JNIEnv* env, jobject bitmap) {
return b; return b;
} }
void GraphicsJNI::getSkBitmap(JNIEnv* env, jobject bitmap, SkBitmap* outBitmap) {
// TODO: We have to copy from the existing bitmap due to rowBytes not
// being updated on the SkPixelRef at reconfigure time. This is a short term
// problem that will be fixed with the specialized wrapper
*outBitmap = *getSkBitmapDeprecated(env, bitmap);
}
SkPixelRef* GraphicsJNI::getSkPixelRef(JNIEnv* env, jobject bitmap) {
jlong bitmapHandle = env->GetLongField(bitmap, gBitmap_skBitmapPtr);
SkBitmap* b = reinterpret_cast<SkBitmap*>(bitmapHandle);
return b->pixelRef();
}
SkColorType GraphicsJNI::getNativeBitmapColorType(JNIEnv* env, jobject jconfig) { SkColorType GraphicsJNI::getNativeBitmapColorType(JNIEnv* env, jobject jconfig) {
SkASSERT(env); SkASSERT(env);
if (NULL == jconfig) { if (NULL == jconfig) {

4
core/jni/android/graphics/GraphicsJNI.h
View File

@@ -49,7 +49,9 @@ public:
static void point_to_jpointf(const SkPoint& point, JNIEnv*, jobject jpointf); static void point_to_jpointf(const SkPoint& point, JNIEnv*, jobject jpointf);
static android::Canvas* getNativeCanvas(JNIEnv*, jobject canvas); static android::Canvas* getNativeCanvas(JNIEnv*, jobject canvas);
static SkBitmap* getSkBitmap(JNIEnv*, jobject bitmap); static SkBitmap* getSkBitmapDeprecated(JNIEnv*, jobject bitmap);
static void getSkBitmap(JNIEnv*, jobject bitmap, SkBitmap* outBitmap);
static SkPixelRef* getSkPixelRef(JNIEnv*, jobject bitmap);
static SkRegion* getNativeRegion(JNIEnv*, jobject region); static SkRegion* getNativeRegion(JNIEnv*, jobject region);
// Given the 'native' long held by the Rasterizer.java object, return a // Given the 'native' long held by the Rasterizer.java object, return a

19
core/jni/android/graphics/pdf/PdfRenderer.cpp
View File

@@ -243,19 +243,21 @@ static void renderPageBitmap(FPDF_BITMAP bitmap, FPDF_PAGE page, int destLeft, i
} }
static void nativeRenderPage(JNIEnv* env, jclass thiz, jlong documentPtr, jlong pagePtr, static void nativeRenderPage(JNIEnv* env, jclass thiz, jlong documentPtr, jlong pagePtr,
jlong bitmapPtr, jint destLeft, jint destTop, jint destRight, jint destBottom, jobject jbitmap, jint destLeft, jint destTop, jint destRight, jint destBottom,
jlong matrixPtr, jint renderMode) { jlong matrixPtr, jint renderMode) {
FPDF_PAGE page = reinterpret_cast<FPDF_PAGE>(pagePtr); FPDF_PAGE page = reinterpret_cast<FPDF_PAGE>(pagePtr);
SkBitmap* skBitmap = reinterpret_cast<SkBitmap*>(bitmapPtr);
SkMatrix* skMatrix = reinterpret_cast<SkMatrix*>(matrixPtr); SkMatrix* skMatrix = reinterpret_cast<SkMatrix*>(matrixPtr);
skBitmap->lockPixels(); SkBitmap skBitmap;
GraphicsJNI::getSkBitmap(env, jbitmap, &skBitmap);
const int stride = skBitmap->width() * 4; SkAutoLockPixels alp(skBitmap);
FPDF_BITMAP bitmap = FPDFBitmap_CreateEx(skBitmap->width(), skBitmap->height(), const int stride = skBitmap.width() * 4;
FPDFBitmap_BGRA, skBitmap->getPixels(), stride);
FPDF_BITMAP bitmap = FPDFBitmap_CreateEx(skBitmap.width(), skBitmap.height(),
FPDFBitmap_BGRA, skBitmap.getPixels(), stride);
if (!bitmap) { if (!bitmap) {
ALOGE("Erorr creating bitmap"); ALOGE("Erorr creating bitmap");
@@ -278,8 +280,7 @@ static void nativeRenderPage(JNIEnv* env, jclass thiz, jlong documentPtr, jlong
renderPageBitmap(bitmap, page, destLeft, destTop, destRight, renderPageBitmap(bitmap, page, destLeft, destTop, destRight,
destBottom, skMatrix, renderFlags); destBottom, skMatrix, renderFlags);
skBitmap->notifyPixelsChanged(); skBitmap.notifyPixelsChanged();
skBitmap->unlockPixels();
} }
static JNINativeMethod gPdfRenderer_Methods[] = { static JNINativeMethod gPdfRenderer_Methods[] = {
@@ -287,7 +288,7 @@ static JNINativeMethod gPdfRenderer_Methods[] = {
{"nativeClose", "(J)V", (void*) nativeClose}, {"nativeClose", "(J)V", (void*) nativeClose},
{"nativeGetPageCount", "(J)I", (void*) nativeGetPageCount}, {"nativeGetPageCount", "(J)I", (void*) nativeGetPageCount},
{"nativeScaleForPrinting", "(J)Z", (void*) nativeScaleForPrinting}, {"nativeScaleForPrinting", "(J)Z", (void*) nativeScaleForPrinting},
{"nativeRenderPage", "(JJJIIIIJI)V", (void*) nativeRenderPage}, {"nativeRenderPage", "(JJLandroid/graphics/Bitmap;IIIIJI)V", (void*) nativeRenderPage},
{"nativeOpenPageAndGetSize", "(JILandroid/graphics/Point;)J", (void*) nativeOpenPageAndGetSize}, {"nativeOpenPageAndGetSize", "(JILandroid/graphics/Point;)J", (void*) nativeOpenPageAndGetSize},
{"nativeClosePage", "(J)V", (void*) nativeClosePage} {"nativeClosePage", "(J)V", (void*) nativeClosePage}
}; };

18
core/jni/android/opengl/util.cpp
View File

@@ -618,23 +618,25 @@ static int getType(SkColorType colorType)
static jint util_getInternalFormat(JNIEnv *env, jclass clazz, static jint util_getInternalFormat(JNIEnv *env, jclass clazz,
jobject jbitmap) jobject jbitmap)
{ {
SkBitmap const * nativeBitmap = GraphicsJNI::getSkBitmap(env, jbitmap); SkBitmap nativeBitmap;
return getInternalFormat(nativeBitmap->colorType()); GraphicsJNI::getSkBitmap(env, jbitmap, &nativeBitmap);
return getInternalFormat(nativeBitmap.colorType());
} }
static jint util_getType(JNIEnv *env, jclass clazz, static jint util_getType(JNIEnv *env, jclass clazz,
jobject jbitmap) jobject jbitmap)
{ {
SkBitmap const * nativeBitmap = GraphicsJNI::getSkBitmap(env, jbitmap); SkBitmap nativeBitmap;
return getType(nativeBitmap->colorType()); GraphicsJNI::getSkBitmap(env, jbitmap, &nativeBitmap);
return getType(nativeBitmap.colorType());
} }
static jint util_texImage2D(JNIEnv *env, jclass clazz, static jint util_texImage2D(JNIEnv *env, jclass clazz,
jint target, jint level, jint internalformat, jint target, jint level, jint internalformat,
jobject jbitmap, jint type, jint border) jobject jbitmap, jint type, jint border)
{ {
SkBitmap const * nativeBitmap = GraphicsJNI::getSkBitmap(env, jbitmap); SkBitmap bitmap;
const SkBitmap& bitmap(*nativeBitmap); GraphicsJNI::getSkBitmap(env, jbitmap, &bitmap);
SkColorType colorType = bitmap.colorType(); SkColorType colorType = bitmap.colorType();
if (internalformat < 0) { if (internalformat < 0) {
internalformat = getInternalFormat(colorType); internalformat = getInternalFormat(colorType);
@@ -680,8 +682,8 @@ static jint util_texSubImage2D(JNIEnv *env, jclass clazz,
jint target, jint level, jint xoffset, jint yoffset, jint target, jint level, jint xoffset, jint yoffset,
jobject jbitmap, jint format, jint type) jobject jbitmap, jint format, jint type)
{ {
SkBitmap const * nativeBitmap = GraphicsJNI::getSkBitmap(env, jbitmap); SkBitmap bitmap;
const SkBitmap& bitmap(*nativeBitmap); GraphicsJNI::getSkBitmap(env, jbitmap, &bitmap);
SkColorType colorType = bitmap.colorType(); SkColorType colorType = bitmap.colorType();
if (format < 0) { if (format < 0) {
format = getInternalFormat(colorType); format = getInternalFormat(colorType);

4
core/jni/android_graphics_Canvas.cpp
View File

@@ -42,7 +42,7 @@ static void finalizer(JNIEnv* env, jobject clazz, jlong canvasHandle) {
static jlong initRaster(JNIEnv* env, jobject, jobject jbitmap) { static jlong initRaster(JNIEnv* env, jobject, jobject jbitmap) {
SkBitmap* bitmap = nullptr; SkBitmap* bitmap = nullptr;
if (jbitmap != NULL) { if (jbitmap != NULL) {
bitmap = GraphicsJNI::getSkBitmap(env, jbitmap); bitmap = GraphicsJNI::getSkBitmapDeprecated(env, jbitmap);
} }
return reinterpret_cast<jlong>(Canvas::create_canvas( return reinterpret_cast<jlong>(Canvas::create_canvas(
bitmap ? *bitmap : SkBitmap())); bitmap ? *bitmap : SkBitmap()));
@@ -53,7 +53,7 @@ static jlong initRaster(JNIEnv* env, jobject, jobject jbitmap) {
static void setBitmap(JNIEnv* env, jobject, jlong canvasHandle, jobject jbitmap) { static void setBitmap(JNIEnv* env, jobject, jlong canvasHandle, jobject jbitmap) {
SkBitmap* bitmap = nullptr; SkBitmap* bitmap = nullptr;
if (jbitmap != NULL) { if (jbitmap != NULL) {
bitmap = GraphicsJNI::getSkBitmap(env, jbitmap); bitmap = GraphicsJNI::getSkBitmapDeprecated(env, jbitmap);
} }
get_canvas(canvasHandle)->setBitmap(bitmap ? *bitmap : SkBitmap()); get_canvas(canvasHandle)->setBitmap(bitmap ? *bitmap : SkBitmap());
} }

5
core/jni/android_view_PointerIcon.cpp
View File

@@ -80,10 +80,7 @@ status_t android_view_PointerIcon_load(JNIEnv* env, jobject pointerIconObj, jobj
jobject bitmapObj = env->GetObjectField(loadedPointerIconObj, gPointerIconClassInfo.mBitmap); jobject bitmapObj = env->GetObjectField(loadedPointerIconObj, gPointerIconClassInfo.mBitmap);
if (bitmapObj) { if (bitmapObj) {
SkBitmap* bitmap = GraphicsJNI::getSkBitmap(env, bitmapObj); GraphicsJNI::getSkBitmap(env, bitmapObj, &(outPointerIcon->bitmap));
if (bitmap) {
outPointerIcon->bitmap = *bitmap; // use a shared pixel ref
}
env->DeleteLocalRef(bitmapObj); env->DeleteLocalRef(bitmapObj);
} }

13
core/jni/com_google_android_gles_jni_EGLImpl.cpp
View File

@@ -277,8 +277,9 @@ static void jni_eglCreatePixmapSurface(JNIEnv *_env, jobject _this, jobject out_
EGLConfig cnf = getConfig(_env, config); EGLConfig cnf = getConfig(_env, config);
jint* base = 0; jint* base = 0;
SkBitmap const * nativeBitmap = GraphicsJNI::getSkBitmap(_env, native_pixmap); SkBitmap nativeBitmap;
SkPixelRef* ref = nativeBitmap ? nativeBitmap->pixelRef() : 0; GraphicsJNI::getSkBitmap(_env, native_pixmap, &nativeBitmap);
SkPixelRef* ref = nativeBitmap.pixelRef();
if (ref == NULL) { if (ref == NULL) {
jniThrowException(_env, "java/lang/IllegalArgumentException", "Bitmap has no PixelRef"); jniThrowException(_env, "java/lang/IllegalArgumentException", "Bitmap has no PixelRef");
return; return;
@@ -289,10 +290,10 @@ static void jni_eglCreatePixmapSurface(JNIEnv *_env, jobject _this, jobject out_
egl_native_pixmap_t pixmap; egl_native_pixmap_t pixmap;
pixmap.version = sizeof(pixmap); pixmap.version = sizeof(pixmap);
pixmap.width = nativeBitmap->width(); pixmap.width = nativeBitmap.width();
pixmap.height = nativeBitmap->height(); pixmap.height = nativeBitmap.height();
pixmap.stride = nativeBitmap->rowBytes() / nativeBitmap->bytesPerPixel(); pixmap.stride = nativeBitmap.rowBytes() / nativeBitmap.bytesPerPixel();
pixmap.format = convertPixelFormat(nativeBitmap->colorType()); pixmap.format = convertPixelFormat(nativeBitmap.colorType());
pixmap.data = (uint8_t*)ref->pixels(); pixmap.data = (uint8_t*)ref->pixels();
base = beginNativeAttribList(_env, attrib_list); base = beginNativeAttribList(_env, attrib_list);

4
graphics/java/android/graphics/pdf/PdfRenderer.java
View File

@@ -380,7 +380,7 @@ public final class PdfRenderer implements AutoCloseable {
final long transformPtr = (transform != null) ? transform.native_instance : 0; final long transformPtr = (transform != null) ? transform.native_instance : 0;
nativeRenderPage(mNativeDocument, mNativePage, destination.getSkBitmap(), contentLeft, nativeRenderPage(mNativeDocument, mNativePage, destination, contentLeft,
contentTop, contentRight, contentBottom, transformPtr, renderMode); contentTop, contentRight, contentBottom, transformPtr, renderMode);
} }
@@ -425,7 +425,7 @@ public final class PdfRenderer implements AutoCloseable {
private static native void nativeClose(long documentPtr); private static native void nativeClose(long documentPtr);
private static native int nativeGetPageCount(long documentPtr); private static native int nativeGetPageCount(long documentPtr);
private static native boolean nativeScaleForPrinting(long documentPtr); private static native boolean nativeScaleForPrinting(long documentPtr);
private static native void nativeRenderPage(long documentPtr, long pagePtr, long destPtr, private static native void nativeRenderPage(long documentPtr, long pagePtr, Bitmap dest,
int destLeft, int destTop, int destRight, int destBottom, long matrixPtr, int renderMode); int destLeft, int destTop, int destRight, int destBottom, long matrixPtr, int renderMode);
private static native long nativeOpenPageAndGetSize(long documentPtr, int pageIndex, private static native long nativeOpenPageAndGetSize(long documentPtr, int pageIndex,
Point outSize); Point outSize);

9
media/jni/android_media_MediaMetadataRetriever.cpp
View File

@@ -299,15 +299,16 @@ static jobject android_media_MediaMetadataRetriever_getFrameAtTime(JNIEnv *env,
return NULL; return NULL;
} }
SkBitmap *bitmap = GraphicsJNI::getSkBitmap(env, jBitmap); SkBitmap bitmap;
GraphicsJNI::getSkBitmap(env, jBitmap, &bitmap);
bitmap->lockPixels(); bitmap.lockPixels();
rotate((uint16_t*)bitmap->getPixels(), rotate((uint16_t*)bitmap.getPixels(),
(uint16_t*)((char*)videoFrame + sizeof(VideoFrame)), (uint16_t*)((char*)videoFrame + sizeof(VideoFrame)),
videoFrame->mWidth, videoFrame->mWidth,
videoFrame->mHeight, videoFrame->mHeight,
videoFrame->mRotationAngle); videoFrame->mRotationAngle);
bitmap->unlockPixels(); bitmap.unlockPixels();
if (videoFrame->mDisplayWidth != videoFrame->mWidth || if (videoFrame->mDisplayWidth != videoFrame->mWidth ||
videoFrame->mDisplayHeight != videoFrame->mHeight) { videoFrame->mDisplayHeight != videoFrame->mHeight) {

35
native/graphics/jni/bitmap.cpp
View File

@@ -27,18 +27,16 @@ int AndroidBitmap_getInfo(JNIEnv* env, jobject jbitmap,
return ANDROID_BITMAP_RESULT_BAD_PARAMETER; return ANDROID_BITMAP_RESULT_BAD_PARAMETER;
} }
SkBitmap* bm = GraphicsJNI::getSkBitmap(env, jbitmap); SkBitmap bm;
if (NULL == bm) { GraphicsJNI::getSkBitmap(env, jbitmap, &bm);
return ANDROID_BITMAP_RESULT_JNI_EXCEPTION;
}
if (info) { if (info) {
info->width = bm->width(); info->width = bm.width();
info->height = bm->height(); info->height = bm.height();
info->stride = bm->rowBytes(); info->stride = bm.rowBytes();
info->flags = 0; info->flags = 0;
switch (bm->colorType()) { switch (bm.colorType()) {
case kN32_SkColorType: case kN32_SkColorType:
info->format = ANDROID_BITMAP_FORMAT_RGBA_8888; info->format = ANDROID_BITMAP_FORMAT_RGBA_8888;
break; break;
@@ -64,17 +62,18 @@ int AndroidBitmap_lockPixels(JNIEnv* env, jobject jbitmap, void** addrPtr) {
return ANDROID_BITMAP_RESULT_BAD_PARAMETER; return ANDROID_BITMAP_RESULT_BAD_PARAMETER;
} }
SkBitmap* bm = GraphicsJNI::getSkBitmap(env, jbitmap); SkPixelRef* pixelRef = GraphicsJNI::getSkPixelRef(env, jbitmap);
if (NULL == bm) { if (!pixelRef) {
return ANDROID_BITMAP_RESULT_JNI_EXCEPTION; return ANDROID_BITMAP_RESULT_JNI_EXCEPTION;
} }
bm->lockPixels(); pixelRef->lockPixels();
void* addr = bm->getPixels(); void* addr = pixelRef->pixels();
if (NULL == addr) { if (NULL == addr) {
bm->unlockPixels(); pixelRef->unlockPixels();
return ANDROID_BITMAP_RESULT_ALLOCATION_FAILED; return ANDROID_BITMAP_RESULT_ALLOCATION_FAILED;
} }
pixelRef->ref();
if (addrPtr) { if (addrPtr) {
*addrPtr = addr; *addrPtr = addr;
@@ -87,8 +86,8 @@ int AndroidBitmap_unlockPixels(JNIEnv* env, jobject jbitmap) {
return ANDROID_BITMAP_RESULT_BAD_PARAMETER; return ANDROID_BITMAP_RESULT_BAD_PARAMETER;
} }
SkBitmap* bm = GraphicsJNI::getSkBitmap(env, jbitmap); SkPixelRef* pixelRef = GraphicsJNI::getSkPixelRef(env, jbitmap);
if (NULL == bm) { if (!pixelRef) {
return ANDROID_BITMAP_RESULT_JNI_EXCEPTION; return ANDROID_BITMAP_RESULT_JNI_EXCEPTION;
} }
@@ -96,9 +95,11 @@ int AndroidBitmap_unlockPixels(JNIEnv* env, jobject jbitmap) {
// bitmaps. Note that this will slow down read-only accesses to the // bitmaps. Note that this will slow down read-only accesses to the
// bitmaps, but the NDK methods are primarily intended to be used for // bitmaps, but the NDK methods are primarily intended to be used for
// writes. // writes.
bm->notifyPixelsChanged(); pixelRef->notifyPixelsChanged();
pixelRef->unlockPixels();
pixelRef->unref();
bm->unlockPixels();
return ANDROID_BITMAP_RESULT_SUCCESS; return ANDROID_BITMAP_RESULT_SUCCESS;
} }

25
rs/jni/android_renderscript_RenderScript.cpp
View File

@@ -1102,9 +1102,8 @@ static jlong
nAllocationCreateFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong type, jint mip, nAllocationCreateFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong type, jint mip,
jobject jbitmap, jint usage) jobject jbitmap, jint usage)
{ {
SkBitmap const * nativeBitmap = SkBitmap bitmap;
GraphicsJNI::getSkBitmap(_env, jbitmap); GraphicsJNI::getSkBitmap(_env, jbitmap, &bitmap);
const SkBitmap& bitmap(*nativeBitmap);
bitmap.lockPixels(); bitmap.lockPixels();
const void* ptr = bitmap.getPixels(); const void* ptr = bitmap.getPixels();
@@ -1119,9 +1118,8 @@ static jlong
nAllocationCreateBitmapBackedAllocation(JNIEnv *_env, jobject _this, jlong con, jlong type, nAllocationCreateBitmapBackedAllocation(JNIEnv *_env, jobject _this, jlong con, jlong type,
jint mip, jobject jbitmap, jint usage) jint mip, jobject jbitmap, jint usage)
{ {
SkBitmap const * nativeBitmap = SkBitmap bitmap;
GraphicsJNI::getSkBitmap(_env, jbitmap); GraphicsJNI::getSkBitmap(_env, jbitmap, &bitmap);
const SkBitmap& bitmap(*nativeBitmap);
bitmap.lockPixels(); bitmap.lockPixels();
const void* ptr = bitmap.getPixels(); const void* ptr = bitmap.getPixels();
@@ -1136,9 +1134,8 @@ static jlong
nAllocationCubeCreateFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong type, jint mip, nAllocationCubeCreateFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong type, jint mip,
jobject jbitmap, jint usage) jobject jbitmap, jint usage)
{ {
SkBitmap const * nativeBitmap = SkBitmap bitmap;
GraphicsJNI::getSkBitmap(_env, jbitmap); GraphicsJNI::getSkBitmap(_env, jbitmap, &bitmap);
const SkBitmap& bitmap(*nativeBitmap);
bitmap.lockPixels(); bitmap.lockPixels();
const void* ptr = bitmap.getPixels(); const void* ptr = bitmap.getPixels();
@@ -1152,9 +1149,8 @@ nAllocationCubeCreateFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong ty
static void static void
nAllocationCopyFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong alloc, jobject jbitmap) nAllocationCopyFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong alloc, jobject jbitmap)
{ {
SkBitmap const * nativeBitmap = SkBitmap bitmap;
GraphicsJNI::getSkBitmap(_env, jbitmap); GraphicsJNI::getSkBitmap(_env, jbitmap, &bitmap);
const SkBitmap& bitmap(*nativeBitmap);
int w = bitmap.width(); int w = bitmap.width();
int h = bitmap.height(); int h = bitmap.height();
@@ -1169,9 +1165,8 @@ nAllocationCopyFromBitmap(JNIEnv *_env, jobject _this, jlong con, jlong alloc, j
static void static void
nAllocationCopyToBitmap(JNIEnv *_env, jobject _this, jlong con, jlong alloc, jobject jbitmap) nAllocationCopyToBitmap(JNIEnv *_env, jobject _this, jlong con, jlong alloc, jobject jbitmap)
{ {
SkBitmap const * nativeBitmap = SkBitmap bitmap;
GraphicsJNI::getSkBitmap(_env, jbitmap); GraphicsJNI::getSkBitmap(_env, jbitmap, &bitmap);
const SkBitmap& bitmap(*nativeBitmap);
bitmap.lockPixels(); bitmap.lockPixels();
void* ptr = bitmap.getPixels(); void* ptr = bitmap.getPixels();

2
services/core/jni/com_android_server_AssetAtlasService.cpp
View File

@@ -64,7 +64,7 @@ namespace android {
static jboolean com_android_server_AssetAtlasService_upload(JNIEnv* env, jobject, static jboolean com_android_server_AssetAtlasService_upload(JNIEnv* env, jobject,
jobject graphicBuffer, jobject bitmapHandle) { jobject graphicBuffer, jobject bitmapHandle) {
SkBitmap& bitmap = *GraphicsJNI::getSkBitmap(env, bitmapHandle); SkBitmap& bitmap = *GraphicsJNI::getSkBitmapDeprecated(env, bitmapHandle);
SkAutoLockPixels alp(bitmap); SkAutoLockPixels alp(bitmap);
// The goal of this method is to copy the bitmap into the GraphicBuffer // The goal of this method is to copy the bitmap into the GraphicBuffer