summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--core/jni/android_view_GLES20Canvas.cpp2
-rw-r--r--libs/hwui/DisplayListRenderer.cpp9
-rw-r--r--libs/hwui/DisplayListRenderer.h27
-rw-r--r--libs/hwui/OpenGLDebugRenderer.cpp6
-rw-r--r--libs/hwui/OpenGLDebugRenderer.h1
-rw-r--r--libs/hwui/OpenGLRenderer.cpp9
-rw-r--r--libs/hwui/OpenGLRenderer.h3
7 files changed, 49 insertions, 8 deletions
diff --git a/core/jni/android_view_GLES20Canvas.cpp b/core/jni/android_view_GLES20Canvas.cpp
index 6d848a4..cb261d4 100644
--- a/core/jni/android_view_GLES20Canvas.cpp
+++ b/core/jni/android_view_GLES20Canvas.cpp
@@ -421,7 +421,7 @@ static void android_view_GLES20Canvas_destroyDisplayList(JNIEnv* env,
static void android_view_GLES20Canvas_drawDisplayList(JNIEnv* env,
jobject canvas, OpenGLRenderer* renderer, DisplayList* displayList) {
- displayList->replay(*renderer);
+ renderer->drawDisplayList(displayList);
}
#endif // USE_OPENGL_RENDERER
diff --git a/libs/hwui/DisplayListRenderer.cpp b/libs/hwui/DisplayListRenderer.cpp
index 23ccef6..770e596 100644
--- a/libs/hwui/DisplayListRenderer.cpp
+++ b/libs/hwui/DisplayListRenderer.cpp
@@ -226,6 +226,10 @@ void DisplayList::replay(OpenGLRenderer& renderer) {
(SkRegion::Op) getInt());
}
break;
+ case DrawDisplayList: {
+ renderer.drawDisplayList(getDisplayList());
+ }
+ break;
case DrawBitmap: {
renderer.drawBitmap(getBitmap(), getFloat(), getFloat(), getPaint());
}
@@ -453,6 +457,11 @@ bool DisplayListRenderer::clipRect(float left, float top, float right, float bot
return OpenGLRenderer::clipRect(left, top, right, bottom, op);
}
+void DisplayListRenderer::drawDisplayList(DisplayList* displayList) {
+ addOp(DisplayList::DrawDisplayList);
+ addDisplayList(displayList);
+}
+
void DisplayListRenderer::drawBitmap(SkBitmap* bitmap, float left, float top,
SkPaint* paint) {
addOp(DisplayList::DrawBitmap);
diff --git a/libs/hwui/DisplayListRenderer.h b/libs/hwui/DisplayListRenderer.h
index fd69fab..b608381 100644
--- a/libs/hwui/DisplayListRenderer.h
+++ b/libs/hwui/DisplayListRenderer.h
@@ -92,6 +92,7 @@ public:
SetMatrix,
ConcatMatrix,
ClipRect,
+ DrawDisplayList,
DrawBitmap,
DrawBitmapMatrix,
DrawBitmapRect,
@@ -160,6 +161,10 @@ private:
return (SkPaint*) getInt();
}
+ DisplayList* getDisplayList() {
+ return (DisplayList*) getInt();
+ }
+
inline float getFloat() {
return mReader.readScalar();
}
@@ -235,6 +240,7 @@ public:
bool clipRect(float left, float top, float right, float bottom, SkRegion::Op op);
+ void drawDisplayList(DisplayList* displayList);
void drawBitmap(SkBitmap* bitmap, float left, float top, SkPaint* paint);
void drawBitmap(SkBitmap* bitmap, SkMatrix* matrix, SkPaint* paint);
void drawBitmap(SkBitmap* bitmap, float srcLeft, float srcTop,
@@ -348,16 +354,25 @@ private:
inline void addPaint(SkPaint* paint) {
if (paint == NULL) {
- addInt((int)NULL);
+ addInt((int) NULL);
return;
}
+
SkPaint *paintCopy = mPaintMap.valueFor(paint);
if (paintCopy == NULL || paintCopy->getGenerationID() != paint->getGenerationID()) {
paintCopy = new SkPaint(*paint);
mPaintMap.add(paint, paintCopy);
mPaints.add(paintCopy);
}
- addInt((int)paintCopy);
+
+ addInt((int) paintCopy);
+ }
+
+ inline void addDisplayList(DisplayList* displayList) {
+ // TODO: To be safe, the display list should be ref-counted in the
+ // resources cache, but we rely on the caller (UI toolkit) to
+ // do the right thing for now
+ addInt((int) displayList);
}
inline void addMatrix(SkMatrix* matrix) {
@@ -371,21 +386,21 @@ private:
// correctly, such as creating the bitmap from scratch, drawing with it, changing its
// contents, and drawing again. The only fix would be to always copy it the first time,
// which doesn't seem worth the extra cycles for this unlikely case.
- addInt((int)bitmap);
+ addInt((int) bitmap);
mBitmapResources.add(bitmap);
Caches& caches = Caches::getInstance();
caches.resourceCache.incrementRefcount(bitmap);
}
inline void addShader(SkiaShader* shader) {
- addInt((int)shader);
+ addInt((int) shader);
mShaderResources.add(shader);
Caches& caches = Caches::getInstance();
caches.resourceCache.incrementRefcount(shader);
}
inline void addColorFilter(SkiaColorFilter* colorFilter) {
- addInt((int)colorFilter);
+ addInt((int) colorFilter);
mFilterResources.add(colorFilter);
Caches& caches = Caches::getInstance();
caches.resourceCache.incrementRefcount(colorFilter);
@@ -398,7 +413,7 @@ private:
Vector<SkiaColorFilter*> mFilterResources;
Vector<SkPaint*> mPaints;
- DefaultKeyedVector<SkPaint *, SkPaint *> mPaintMap;
+ DefaultKeyedVector<SkPaint*, SkPaint*> mPaintMap;
Vector<SkMatrix*> mMatrices;
PathHeap* mPathHeap;
diff --git a/libs/hwui/OpenGLDebugRenderer.cpp b/libs/hwui/OpenGLDebugRenderer.cpp
index fe75ca2..1cf3d20 100644
--- a/libs/hwui/OpenGLDebugRenderer.cpp
+++ b/libs/hwui/OpenGLDebugRenderer.cpp
@@ -48,6 +48,12 @@ int OpenGLDebugRenderer::saveLayer(float left, float top, float right, float bot
return OpenGLRenderer::saveLayer(left, top, right, bottom, p, flags);
}
+void OpenGLDebugRenderer::drawDisplayList(DisplayList* displayList) {
+ mPrimitivesCount++;
+ StopWatch w("drawDisplayList");
+ OpenGLRenderer::drawDisplayList(displayList);
+}
+
void OpenGLDebugRenderer::drawBitmap(SkBitmap* bitmap, float left, float top,
SkPaint* paint) {
mPrimitivesCount++;
diff --git a/libs/hwui/OpenGLDebugRenderer.h b/libs/hwui/OpenGLDebugRenderer.h
index 7787ff1..ee34d73 100644
--- a/libs/hwui/OpenGLDebugRenderer.h
+++ b/libs/hwui/OpenGLDebugRenderer.h
@@ -40,6 +40,7 @@ public:
int saveLayer(float left, float top, float right, float bottom,
SkPaint* p, int flags);
+ void drawDisplayList(DisplayList* displayList);
void drawBitmap(SkBitmap* bitmap, float left, float top, SkPaint* paint);
void drawBitmap(SkBitmap* bitmap, SkMatrix* matrix, SkPaint* paint);
void drawBitmap(SkBitmap* bitmap, float srcLeft, float srcTop,
diff --git a/libs/hwui/OpenGLRenderer.cpp b/libs/hwui/OpenGLRenderer.cpp
index cad991d..9a6618b 100644
--- a/libs/hwui/OpenGLRenderer.cpp
+++ b/libs/hwui/OpenGLRenderer.cpp
@@ -29,6 +29,7 @@
#include <ui/Rect.h>
#include "OpenGLRenderer.h"
+#include "DisplayListRenderer.h"
namespace android {
namespace uirenderer {
@@ -827,6 +828,14 @@ bool OpenGLRenderer::clipRect(float left, float top, float right, float bottom,
// Drawing
///////////////////////////////////////////////////////////////////////////////
+void OpenGLRenderer::drawDisplayList(DisplayList* displayList) {
+ // All the usual checks and setup operations (quickReject, setupDraw, etc.)
+ // will be performed by the display list itself
+ if (displayList) {
+ displayList->replay(*this);
+ }
+}
+
void OpenGLRenderer::drawBitmap(SkBitmap* bitmap, float left, float top, SkPaint* paint) {
const float right = left + bitmap->width();
const float bottom = top + bitmap->height();
diff --git a/libs/hwui/OpenGLRenderer.h b/libs/hwui/OpenGLRenderer.h
index 93c2e6c..3340b4a 100644
--- a/libs/hwui/OpenGLRenderer.h
+++ b/libs/hwui/OpenGLRenderer.h
@@ -58,7 +58,7 @@ namespace uirenderer {
// Renderer
///////////////////////////////////////////////////////////////////////////////
-class DisplayListRenderer;
+class DisplayList;
/**
* OpenGL renderer used to draw accelerated 2D graphics. The API is a
@@ -100,6 +100,7 @@ public:
bool quickReject(float left, float top, float right, float bottom);
virtual bool clipRect(float left, float top, float right, float bottom, SkRegion::Op op);
+ virtual void drawDisplayList(DisplayList* displayList);
virtual void drawBitmap(SkBitmap* bitmap, float left, float top, SkPaint* paint);
virtual void drawBitmap(SkBitmap* bitmap, SkMatrix* matrix, SkPaint* paint);
virtual void drawBitmap(SkBitmap* bitmap, float srcLeft, float srcTop,