diff --git a/tools/layoutlib/bridge/src/android/view/AttachInfo_Accessor.java b/tools/layoutlib/bridge/src/android/view/AttachInfo_Accessor.java index 4901f72b23d6b..94f3f546d0c37 100644 --- a/tools/layoutlib/bridge/src/android/view/AttachInfo_Accessor.java +++ b/tools/layoutlib/bridge/src/android/view/AttachInfo_Accessor.java @@ -45,4 +45,10 @@ public class AttachInfo_Accessor { public static void dispatchOnPreDraw(View view) { view.mAttachInfo.mTreeObserver.dispatchOnPreDraw(); } + + public static void detachFromWindow(View view) { + if (view != null) { + view.dispatchDetachedFromWindow(); + } + } } diff --git a/tools/layoutlib/bridge/src/android/view/HandlerActionQueue_Delegate.java b/tools/layoutlib/bridge/src/android/view/HandlerActionQueue_Delegate.java new file mode 100644 index 0000000000000..e580ed0e14f75 --- /dev/null +++ b/tools/layoutlib/bridge/src/android/view/HandlerActionQueue_Delegate.java @@ -0,0 +1,37 @@ +/* + * Copyright (C) 2016 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package android.view; + +import com.android.tools.layoutlib.annotations.LayoutlibDelegate; + +/** + * Delegate used to provide new implementation of a select few methods of + * {@link HandlerActionQueue} + * + * Through the layoutlib_create tool, the original methods of ViewRootImpl.RunQueue have been + * replaced by calls to methods of the same name in this delegate class. + * + */ +public class HandlerActionQueue_Delegate { + + @LayoutlibDelegate + /*package*/ static void postDelayed(HandlerActionQueue thisQueue, Runnable action, long + delayMillis) { + // The actual HandlerActionQueue is never run and therefore never cleared. This method + // avoids runnables to be added to the RunQueue so they do not leak resources. + } +} diff --git a/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/BridgeRenderSession.java b/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/BridgeRenderSession.java index 2ac212c312c09..fea633e7036db 100644 --- a/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/BridgeRenderSession.java +++ b/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/BridgeRenderSession.java @@ -208,6 +208,9 @@ public class BridgeRenderSession extends RenderSession { @Override public void dispose() { + if (mSession != null) { + mSession.dispose(); + } } /*package*/ BridgeRenderSession(RenderSessionImpl scene, Result lastResult) { diff --git a/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/impl/RenderSessionImpl.java b/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/impl/RenderSessionImpl.java index ec50cfe556516..99af226e4c4c1 100644 --- a/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/impl/RenderSessionImpl.java +++ b/tools/layoutlib/bridge/src/com/android/layoutlib/bridge/impl/RenderSessionImpl.java @@ -1396,4 +1396,21 @@ public class RenderSessionImpl extends RenderAction { public RenderSession getSession() { return mScene; } + + public void dispose() { + AttachInfo_Accessor.detachFromWindow(mViewRoot); + if (mCanvas != null) { + mCanvas.release(); + mCanvas = null; + } + if (mViewInfoList != null) { + mViewInfoList.clear(); + } + if (mSystemViewInfoList != null) { + mSystemViewInfoList.clear(); + } + mImage = null; + mViewRoot = null; + mContentRoot = null; + } } diff --git a/tools/layoutlib/bridge/tests/src/com/android/layoutlib/bridge/intensive/Main.java b/tools/layoutlib/bridge/tests/src/com/android/layoutlib/bridge/intensive/Main.java index fe16a3ed8459a..6b23da71861a8 100644 --- a/tools/layoutlib/bridge/tests/src/com/android/layoutlib/bridge/intensive/Main.java +++ b/tools/layoutlib/bridge/tests/src/com/android/layoutlib/bridge/intensive/Main.java @@ -291,7 +291,6 @@ public class Main { @Test public void testActivity() throws ClassNotFoundException { renderAndVerify("activity.xml", "activity.png"); - } /** Test allwidgets.xml */ @@ -431,6 +430,8 @@ public class Main { ImageUtils.requireSimilar(goldenImagePath, session.getImage()); } catch (IOException e) { getLogger().error(e, e.getMessage()); + } finally { + session.dispose(); } } diff --git a/tools/layoutlib/create/src/com/android/tools/layoutlib/create/CreateInfo.java b/tools/layoutlib/create/src/com/android/tools/layoutlib/create/CreateInfo.java index 9e390f68f128f..8a23e4b9d3dad 100644 --- a/tools/layoutlib/create/src/com/android/tools/layoutlib/create/CreateInfo.java +++ b/tools/layoutlib/create/src/com/android/tools/layoutlib/create/CreateInfo.java @@ -178,6 +178,7 @@ public final class CreateInfo implements ICreateInfo { "android.view.Choreographer#scheduleVsyncLocked", "android.view.Display#updateDisplayInfoLocked", "android.view.Display#getWindowManager", + "android.view.HandlerActionQueue#postDelayed", "android.view.LayoutInflater#rInflate", "android.view.LayoutInflater#parseInclude", "android.view.View#getWindowToken",