commit ceb67d1f8f61e15e2c752d15c4077e5194118d1c
author Chris Forbes <chrisforbes@google.com> Tue Apr 11 12:20:00 2017 -0700
committer Chris Forbes <chrisforbes@google.com> Tue Apr 11 14:24:28 2017 -0700
tree 6ad15cc9e71998be01a8ebcbacd26a960249da2e
parent 016d73c2d8f8fae544f09a6eebe827791f499470

hwc2: Remove Display::{mIsVirtual,setVirtual}

Trivial cleanup.

Now that mType is available, we don't need a separate flag for whether
the display is virtual. The remaining effect of setVirtual was to set
the display connected, so just use setConnected.

Bug: b/36597125
Test: build
Change-Id: Ia1fa4a324cc07b0d6c124f4f50966c8b0cbe4827

services/surfaceflinger/DisplayHardware/HWC2.cpp

diff --git a/services/surfaceflinger/DisplayHardware/HWC2.cpp b/services/surfaceflinger/DisplayHardware/HWC2.cpp
index 402009a..62c5aff 100644
--- a/services/surfaceflinger/DisplayHardware/HWC2.cpp
+++ b/services/surfaceflinger/DisplayHardware/HWC2.cpp
@@ -244,7 +244,7 @@
         ALOGE("Failed to get display by id");
         return Error::BadDisplay;
     }
-    (*outDisplay)->setVirtual();
+    (*outDisplay)->setConnected(true);
     return Error::None;
 }
 
@@ -531,7 +531,6 @@
   : mDevice(device),
     mId(id),
     mIsConnected(false),
-    mIsVirtual(false),
     mType(DisplayType::Invalid)
 {
     ALOGV("Created display %" PRIu64, id);
@@ -553,7 +552,7 @@
 Display::~Display()
 {
     ALOGV("Destroyed display %" PRIu64, mId);
-    if (mIsVirtual) {
+    if (mType == DisplayType::Virtual) {
         mDevice.destroyVirtualDisplay(mId);
     }
 }