diff options
| author | John Reck <jreck@google.com> | 2014-07-16 13:29:45 -0700 |
|---|---|---|
| committer | John Reck <jreck@google.com> | 2014-07-16 13:55:14 -0700 |
| commit | 0a97330b98dd633b58dcfff405d94476c89e867d (patch) | |
| tree | 857c44bf9e505e947cc441b234d7606d5df53aa0 /libs | |
| parent | 945654188ef8b5ee1c404bd950a9ef45defa43d1 (diff) | |
| download | frameworks_base-0a97330b98dd633b58dcfff405d94476c89e867d.zip frameworks_base-0a97330b98dd633b58dcfff405d94476c89e867d.tar.gz frameworks_base-0a97330b98dd633b58dcfff405d94476c89e867d.tar.bz2 | |
Fix root RenderNode damage calculation
Bug: 15888445
Change-Id: I281ec9271c9889673dcdfcb6d31e341a7b47b7de
Diffstat (limited to 'libs')
| -rw-r--r-- | libs/hwui/RenderNode.h | 4 | ||||
| -rw-r--r-- | libs/hwui/renderthread/CanvasContext.cpp | 5 |
2 files changed, 6 insertions, 3 deletions
diff --git a/libs/hwui/RenderNode.h b/libs/hwui/RenderNode.h index 54fa143..8cc65b2 100644 --- a/libs/hwui/RenderNode.h +++ b/libs/hwui/RenderNode.h @@ -173,9 +173,6 @@ public: // UI thread only! ANDROID_API void addAnimator(const sp<BaseRenderNodeAnimator>& animator); -protected: - virtual void damageSelf(TreeInfo& info); - private: typedef key_value_pair_t<float, DrawRenderNodeOp*> ZDrawRenderNodeOpPair; @@ -250,6 +247,7 @@ private: void prepareLayer(TreeInfo& info); void pushLayerUpdate(TreeInfo& info); void deleteDisplayListData(); + void damageSelf(TreeInfo& info); void incParentRefCount() { mParentCount++; } void decParentRefCount(); diff --git a/libs/hwui/renderthread/CanvasContext.cpp b/libs/hwui/renderthread/CanvasContext.cpp index 57279b7..a4ac262 100644 --- a/libs/hwui/renderthread/CanvasContext.cpp +++ b/libs/hwui/renderthread/CanvasContext.cpp @@ -185,6 +185,11 @@ void CanvasContext::draw() { } else if (!mDirtyRegionsEnabled || mHaveNewSurface) { dirty.setEmpty(); } else { + if (!dirty.intersect(0, 0, width, height)) { + ALOGW("Dirty " RECT_STRING " doesn't intersect with 0 0 %d %d ?", + SK_RECT_ARGS(dirty), width, height); + dirty.setEmpty(); + } profiler().unionDirty(&dirty); } |