ui: Refactor the declaration orders.
By grouping similar kinds together, in an order of types, constants, ctor/dtor, all other methods and data members. Also rename ScreenRecoveryUI::density_ to ScreenRecoveryUI::kDensity to align with others. Test: mmma bootable/recovery Change-Id: I1ba2d15c05ba7be8c39762f3d9dadf1fb2130de4
This commit is contained in:
parent
fa4f014063
commit
7577965ba1
3 changed files with 103 additions and 76 deletions
|
@ -54,7 +54,7 @@ ScreenRecoveryUI::ScreenRecoveryUI()
|
|||
: kMarginWidth(RECOVERY_UI_MARGIN_WIDTH),
|
||||
kMarginHeight(RECOVERY_UI_MARGIN_HEIGHT),
|
||||
kAnimationFps(RECOVERY_UI_ANIMATION_FPS),
|
||||
density_(static_cast<float>(android::base::GetIntProperty("ro.sf.lcd_density", 160)) / 160.f),
|
||||
kDensity(static_cast<float>(android::base::GetIntProperty("ro.sf.lcd_density", 160)) / 160.f),
|
||||
currentIcon(NONE),
|
||||
progressBarType(EMPTY),
|
||||
progressScopeStart(0),
|
||||
|
@ -105,7 +105,7 @@ GRSurface* ScreenRecoveryUI::GetCurrentText() const {
|
|||
}
|
||||
|
||||
int ScreenRecoveryUI::PixelsFromDp(int dp) const {
|
||||
return dp * density_;
|
||||
return dp * kDensity;
|
||||
}
|
||||
|
||||
// Here's the intended layout:
|
||||
|
|
103
screen_ui.h
103
screen_ui.h
|
@ -32,6 +32,17 @@ struct GRSurface;
|
|||
// (shows an icon + a progress bar, text logging, menu, etc.)
|
||||
class ScreenRecoveryUI : public RecoveryUI {
|
||||
public:
|
||||
enum UIElement {
|
||||
HEADER,
|
||||
MENU,
|
||||
MENU_SEL_BG,
|
||||
MENU_SEL_BG_ACTIVE,
|
||||
MENU_SEL_FG,
|
||||
LOG,
|
||||
TEXT_FILL,
|
||||
INFO
|
||||
};
|
||||
|
||||
ScreenRecoveryUI();
|
||||
|
||||
bool Init(const std::string& locale) override;
|
||||
|
@ -67,16 +78,6 @@ class ScreenRecoveryUI : public RecoveryUI {
|
|||
|
||||
void Redraw();
|
||||
|
||||
enum UIElement {
|
||||
HEADER,
|
||||
MENU,
|
||||
MENU_SEL_BG,
|
||||
MENU_SEL_BG_ACTIVE,
|
||||
MENU_SEL_FG,
|
||||
LOG,
|
||||
TEXT_FILL,
|
||||
INFO
|
||||
};
|
||||
void SetColor(UIElement e) const;
|
||||
|
||||
protected:
|
||||
|
@ -89,7 +90,47 @@ class ScreenRecoveryUI : public RecoveryUI {
|
|||
const int kAnimationFps;
|
||||
|
||||
// The scale factor from dp to pixels. 1.0 for mdpi, 4.0 for xxxhdpi.
|
||||
const float density_;
|
||||
const float kDensity;
|
||||
|
||||
virtual bool InitTextParams();
|
||||
|
||||
virtual void draw_background_locked();
|
||||
virtual void draw_foreground_locked();
|
||||
virtual void draw_screen_locked();
|
||||
virtual void update_screen_locked();
|
||||
virtual void update_progress_locked();
|
||||
|
||||
GRSurface* GetCurrentFrame() const;
|
||||
GRSurface* GetCurrentText() const;
|
||||
|
||||
static void* ProgressThreadStartRoutine(void* data);
|
||||
void ProgressThreadLoop();
|
||||
|
||||
virtual void ShowFile(FILE*);
|
||||
virtual void PrintV(const char*, bool, va_list);
|
||||
void PutChar(char);
|
||||
void ClearText();
|
||||
|
||||
void LoadAnimation();
|
||||
void LoadBitmap(const char* filename, GRSurface** surface);
|
||||
void LoadLocalizedBitmap(const char* filename, GRSurface** surface);
|
||||
|
||||
int PixelsFromDp(int dp) const;
|
||||
virtual int GetAnimationBaseline() const;
|
||||
virtual int GetProgressBaseline() const;
|
||||
virtual int GetTextBaseline() const;
|
||||
|
||||
// Draws a highlight bar at (x, y) - (x + width, y + height).
|
||||
virtual void DrawHighlightBar(int x, int y, int width, int height) const;
|
||||
// Draws a horizontal rule at Y. Returns the offset it should be moving along Y-axis.
|
||||
virtual int DrawHorizontalRule(int y) const;
|
||||
// Draws a line of text. Returns the offset it should be moving along Y-axis.
|
||||
virtual int DrawTextLine(int x, int y, const char* line, bool bold) const;
|
||||
// Draws multiple text lines. Returns the offset it should be moving along Y-axis.
|
||||
int DrawTextLines(int x, int y, const char* const* lines) const;
|
||||
// Similar to DrawTextLines() to draw multiple text lines, but additionally wraps long lines.
|
||||
// Returns the offset it should be moving along Y-axis.
|
||||
int DrawWrappedTextLines(int x, int y, const char* const* lines) const;
|
||||
|
||||
Icon currentIcon;
|
||||
|
||||
|
@ -151,46 +192,6 @@ class ScreenRecoveryUI : public RecoveryUI {
|
|||
int char_height_;
|
||||
|
||||
pthread_mutex_t updateMutex;
|
||||
|
||||
virtual bool InitTextParams();
|
||||
|
||||
virtual void draw_background_locked();
|
||||
virtual void draw_foreground_locked();
|
||||
virtual void draw_screen_locked();
|
||||
virtual void update_screen_locked();
|
||||
virtual void update_progress_locked();
|
||||
|
||||
GRSurface* GetCurrentFrame() const;
|
||||
GRSurface* GetCurrentText() const;
|
||||
|
||||
static void* ProgressThreadStartRoutine(void* data);
|
||||
void ProgressThreadLoop();
|
||||
|
||||
virtual void ShowFile(FILE*);
|
||||
virtual void PrintV(const char*, bool, va_list);
|
||||
void PutChar(char);
|
||||
void ClearText();
|
||||
|
||||
void LoadAnimation();
|
||||
void LoadBitmap(const char* filename, GRSurface** surface);
|
||||
void LoadLocalizedBitmap(const char* filename, GRSurface** surface);
|
||||
|
||||
int PixelsFromDp(int dp) const;
|
||||
virtual int GetAnimationBaseline() const;
|
||||
virtual int GetProgressBaseline() const;
|
||||
virtual int GetTextBaseline() const;
|
||||
|
||||
// Draws a highlight bar at (x, y) - (x + width, y + height).
|
||||
virtual void DrawHighlightBar(int x, int y, int width, int height) const;
|
||||
// Draws a horizontal rule at Y. Returns the offset it should be moving along Y-axis.
|
||||
virtual int DrawHorizontalRule(int y) const;
|
||||
// Draws a line of text. Returns the offset it should be moving along Y-axis.
|
||||
virtual int DrawTextLine(int x, int y, const char* line, bool bold) const;
|
||||
// Draws multiple text lines. Returns the offset it should be moving along Y-axis.
|
||||
int DrawTextLines(int x, int y, const char* const* lines) const;
|
||||
// Similar to DrawTextLines() to draw multiple text lines, but additionally wraps long lines.
|
||||
// Returns the offset it should be moving along Y-axis.
|
||||
int DrawWrappedTextLines(int x, int y, const char* const* lines) const;
|
||||
};
|
||||
|
||||
#endif // RECOVERY_UI_H
|
||||
|
|
72
ui.h
72
ui.h
|
@ -26,6 +26,27 @@
|
|||
// Abstract class for controlling the user interface during recovery.
|
||||
class RecoveryUI {
|
||||
public:
|
||||
enum Icon {
|
||||
NONE,
|
||||
INSTALLING_UPDATE,
|
||||
ERASING,
|
||||
NO_COMMAND,
|
||||
ERROR
|
||||
};
|
||||
|
||||
enum ProgressType {
|
||||
EMPTY,
|
||||
INDETERMINATE,
|
||||
DETERMINATE
|
||||
};
|
||||
|
||||
enum KeyAction {
|
||||
ENQUEUE,
|
||||
TOGGLE,
|
||||
REBOOT,
|
||||
IGNORE
|
||||
};
|
||||
|
||||
RecoveryUI();
|
||||
|
||||
virtual ~RecoveryUI() {}
|
||||
|
@ -38,12 +59,10 @@ class RecoveryUI {
|
|||
virtual void SetStage(int current, int max) = 0;
|
||||
|
||||
// Sets the overall recovery state ("background image").
|
||||
enum Icon { NONE, INSTALLING_UPDATE, ERASING, NO_COMMAND, ERROR };
|
||||
virtual void SetBackground(Icon icon) = 0;
|
||||
virtual void SetSystemUpdateText(bool security_update) = 0;
|
||||
|
||||
// --- progress indicator ---
|
||||
enum ProgressType { EMPTY, INDETERMINATE, DETERMINATE };
|
||||
virtual void SetProgressType(ProgressType determinate) = 0;
|
||||
|
||||
// Shows a progress bar and define the scope of the next operation:
|
||||
|
@ -94,7 +113,6 @@ class RecoveryUI {
|
|||
// Called on each key press, even while operations are in progress. Return value indicates whether
|
||||
// an immediate operation should be triggered (toggling the display, rebooting the device), or if
|
||||
// the key should be enqueued for use by the main thread.
|
||||
enum KeyAction { ENQUEUE, TOGGLE, REBOOT, IGNORE };
|
||||
virtual KeyAction CheckKey(int key, bool is_long_press);
|
||||
|
||||
// Called when a key is held down long enough to have been a long-press (but before the key is
|
||||
|
@ -139,10 +157,37 @@ class RecoveryUI {
|
|||
bool touch_screen_allowed_;
|
||||
|
||||
private:
|
||||
enum class ScreensaverState {
|
||||
DISABLED,
|
||||
NORMAL,
|
||||
DIMMED,
|
||||
OFF
|
||||
};
|
||||
|
||||
struct key_timer_t {
|
||||
RecoveryUI* ui;
|
||||
int key_code;
|
||||
int count;
|
||||
};
|
||||
|
||||
// The sensitivity when detecting a swipe.
|
||||
const int kTouchLowThreshold;
|
||||
const int kTouchHighThreshold;
|
||||
|
||||
void OnKeyDetected(int key_code);
|
||||
void OnTouchDetected(int dx, int dy);
|
||||
int OnInputEvent(int fd, uint32_t epevents);
|
||||
void ProcessKey(int key_code, int updown);
|
||||
|
||||
bool IsUsbConnected();
|
||||
|
||||
static void* time_key_helper(void* cookie);
|
||||
void time_key(int key_code, int count);
|
||||
|
||||
void SetLocale(const std::string&);
|
||||
|
||||
bool InitScreensaver();
|
||||
|
||||
// Key event input queue
|
||||
pthread_mutex_t key_queue_mutex;
|
||||
pthread_cond_t key_queue_cond;
|
||||
|
@ -172,33 +217,14 @@ class RecoveryUI {
|
|||
bool touch_swiping_;
|
||||
bool is_bootreason_recovery_ui_;
|
||||
|
||||
struct key_timer_t {
|
||||
RecoveryUI* ui;
|
||||
int key_code;
|
||||
int count;
|
||||
};
|
||||
|
||||
pthread_t input_thread_;
|
||||
|
||||
void OnKeyDetected(int key_code);
|
||||
void OnTouchDetected(int dx, int dy);
|
||||
int OnInputEvent(int fd, uint32_t epevents);
|
||||
void ProcessKey(int key_code, int updown);
|
||||
|
||||
bool IsUsbConnected();
|
||||
|
||||
static void* time_key_helper(void* cookie);
|
||||
void time_key(int key_code, int count);
|
||||
|
||||
void SetLocale(const std::string&);
|
||||
|
||||
enum class ScreensaverState { DISABLED, NORMAL, DIMMED, OFF };
|
||||
ScreensaverState screensaver_state_;
|
||||
|
||||
// The following two contain the absolute values computed from brightness_normal_ and
|
||||
// brightness_dimmed_ respectively.
|
||||
unsigned int brightness_normal_value_;
|
||||
unsigned int brightness_dimmed_value_;
|
||||
bool InitScreensaver();
|
||||
};
|
||||
|
||||
#endif // RECOVERY_UI_H
|
||||
|
|
Loading…
Reference in a new issue