Merge "[HWUI]: fix residual line on FrameBuffer" into lmp-dev (f3e85b5b) · Commits · e / os / android_frameworks_base

libs/hwui/OpenGLRenderer.cpp

100644 → 100755

+19 −5

Original line number	Diff line number	Diff line
		@@ -259,7 +259,7 @@ void OpenGLRenderer::syncState() {
		}
		}

		void OpenGLRenderer::startTilingCurrentClip(bool opaque) {
		void OpenGLRenderer::startTilingCurrentClip(bool opaque, bool expand) {
		if (!mSuppressTiling) {
		const Snapshot* snapshot = currentSnapshot();

		@@ -268,16 +268,29 @@ void OpenGLRenderer::startTilingCurrentClip(bool opaque) {
		clip = &(snapshot->layer->clipRect);
		}

		startTiling(*clip, getViewportHeight(), opaque);
		startTiling(*clip, getViewportHeight(), opaque, expand);
		}
		}

		void OpenGLRenderer::startTiling(const Rect& clip, int windowHeight, bool opaque) {
		void OpenGLRenderer::startTiling(const Rect& clip, int windowHeight, bool opaque, bool expand) {
		if (!mSuppressTiling) {
		if(expand) {
		// Expand the startTiling region by 1
		int leftNotZero = (clip.left > 0) ? 1 : 0;
		int topNotZero = (windowHeight - clip.bottom > 0) ? 1 : 0;

		mCaches.startTiling(
		clip.left - leftNotZero,
		windowHeight - clip.bottom - topNotZero,
		clip.right - clip.left + leftNotZero + 1,
		clip.bottom - clip.top + topNotZero + 1,
		opaque);
		} else {
		mCaches.startTiling(clip.left, windowHeight - clip.bottom,
		clip.right - clip.left, clip.bottom - clip.top, opaque);
		}
		}
		}

		void OpenGLRenderer::endTiling() {
		if (!mSuppressTiling) mCaches.endTiling();
		@@ -821,7 +834,8 @@ bool OpenGLRenderer::createFboLayer(Layer* layer, Rect& bounds, Rect& clip) {
		glFramebufferTexture2D(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D,
		layer->getTexture(), 0);

		startTilingCurrentClip(true);
		// Expand the startTiling region by 1
		startTilingCurrentClip(true, true);

		// Clear the FBO, expand the clear region by 1 to get nice bilinear filtering
		mCaches.enableScissor();

libs/hwui/OpenGLRenderer.h

100644 → 100755

+2 −2

Original line number	Diff line number	Diff line
		@@ -491,14 +491,14 @@ private:
		* This method needs to be invoked every time getTargetFbo() is
		* bound again.
		*/
		void startTilingCurrentClip(bool opaque = false);
		void startTilingCurrentClip(bool opaque = false, bool expand = false);

		/**
		* Tells the GPU what part of the screen is about to be redrawn.
		* This method needs to be invoked every time getTargetFbo() is
		* bound again.
		*/
		void startTiling(const Rect& clip, int windowHeight, bool opaque = false);
		void startTiling(const Rect& clip, int windowHeight, bool opaque = false, bool expand = false);

		/**
		* Tells the GPU that we are done drawing the frame or that we