commit f7f9e50528761022989c4f0cac6a92716b54674f
author Prashant Malani <pmalani@google.com> Thu Mar 10 03:40:20 2016 +0000
committer Prashant Malani <pmalani@google.com> Thu Mar 10 03:40:20 2016 +0000
tree 8f428bb0debeb61b70cfbf5e9b43b58cf5935465
parent 1c7b2230d8aac9f064f68c48b6aa26aca000cc9d

Revert "recovery: More refactoring of WearUI"

This reverts commit 1c7b2230d8aac9f064f68c48b6aa26aca000cc9d.

This change can lead to the derived class indirectly (and incorrectly) calling some functions from the base class, which can lead to unpredictable behavior.

Bug: 27407422
Change-Id: I126a7489b0787dc195e942e2ceea6769de20d70c

screen_ui.h

diff --git a/screen_ui.h b/screen_ui.h
index 386deac..08a5f44 100644
--- a/screen_ui.h
+++ b/screen_ui.h
@@ -68,6 +68,7 @@
     void SetColor(UIElement e);
 
   private:
+    Icon currentIcon;
     int installingFrame;
     const char* locale;
     bool rtl_locale;
@@ -138,8 +139,6 @@
     void LoadBitmap(const char* filename, GRSurface** surface);
     void LoadBitmapArray(const char* filename, int* frames, int* fps, GRSurface*** surface);
     void LoadLocalizedBitmap(const char* filename, GRSurface** surface);
-  protected:
-    Icon currentIcon;
 };
 
 #endif  // RECOVERY_UI_H