tree 6e657db944c37bdd820d591da4d490d475c86114
parent 44bb9d7f6bb4add074b6c3128ac162091f80a58f
author chaviw <chaviw@google.com> 1559167070 -0700
committer Chavi Weingarten <chaviw@google.com> 1559347929 +0000

Only updateBoundsSurface when surface size has changed.

Currently, updateBoundsSurface was getting called when the surface
changed, not just when the size changed. This meant it could be calling
setWindowCrop and deferTransaction when no size had changed. If size
hadn't changed, there was a high possibility that no new frames would be
submitted by the client, causing the deferTransaction to wait forever.

Since the deferTransaction was still waiting, SurfaceFlinger would wake
up every vsync to check if it should call doTransaction for the deferred
transaction. This caused 60Hz composition even when frames were rendered
slower.

Fixes: 132110524
Test: SF doesn't compose 30fps app at 60Hz
Change-Id: Icf3a99b34c288575438bfcd05e9077ea7919b4ca