summaryrefslogtreecommitdiff
path: root/rs/java/android/renderscript/ProgramFragmentFixedFunction.java
diff options
context:
space:
mode:
authorRobert Carr <racarr@google.com>2016-02-02 19:16:04 -0800
committerRobert Carr <racarr@google.com>2016-02-02 22:44:53 -0800
commite352b18f17789897ba1d160b011e81e9b182f0ec (patch)
tree571844bf69c6901948a3eb547c3604701dd5d107 /rs/java/android/renderscript/ProgramFragmentFixedFunction.java
parentd11c15d418d776d1db33f7664fad2671de0e6669 (diff)
Remove obsolete conditional preventing surface placement.
This conditional seems obsolete and is preventing surface changes for dialogs on secondary displays. To show obsolesence, we can break down our cases based on display as follows: 1. Primary Display In this case we have a home stack, and we cannot enter this conditional. 2. Secondary Display (no home stack) a. Private displays - We allow layout without a homestack when type == TYPE_PRIVATE_PRESENTATION b. Other secondary displays - As TYPE_PRIVATE_PRESENTATION can only be on private displays (see addWindow) we always enter this conditional for taskless Windows (e.g. Dialogs) on secondary displays, and refuse to apply their surface changes. We can see if we remove the conditional, 1 will be invariant. 2a will remain invariant, except we allow layout when type != TYPE_PRIVATE_PRESENTATION. If there is a non private presentation window on a private display, an invariant has malfunctioned and we don't know what to do here (we don't enforce all the addWindow invariants at each surface change). In case 2b we will allow surface changes (where previously blocking them), and this seems correct, after all we wish to allow dialogs to appear on secondary displays right? Bug: 26154242 Change-Id: Ia8cbb079616a80f41b131cafe46c7408a558e307
Diffstat (limited to 'rs/java/android/renderscript/ProgramFragmentFixedFunction.java')
0 files changed, 0 insertions, 0 deletions