Skip to content

Commit

Permalink
Move librecovery_ui to a sub-directory
Browse files Browse the repository at this point in the history
This helps to expose librecovery_ui for device specific RecoveryUi.

Bug: 76436783
Test: mma, unit tests pass
Change-Id: Ic6c3d301d5833e4a592e6ea9d9d059bc4e4919be
  • Loading branch information
Tianjie Xu committed Mar 21, 2019
1 parent 9d2a945 commit b5108c3
Show file tree
Hide file tree
Showing 25 changed files with 138 additions and 119 deletions.
76 changes: 5 additions & 71 deletions Android.bp
Original file line number Diff line number Diff line change
Expand Up @@ -26,77 +26,6 @@ cc_defaults {
],
}

cc_library {
name: "librecovery_ui",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"device.cpp",
"screen_ui.cpp",
"ui.cpp",
"vr_ui.cpp",
"wear_ui.cpp"
],

static_libs: [
"libminui",
"libotautil",
"libfstab",
],

shared_libs: [
"libbase",
"libpng",
"libz",
],
}

// Generic device that uses ScreenRecoveryUI.
cc_library_static {
name: "librecovery_ui_default",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"default_device.cpp",
],
}

// The default wear device that uses WearRecoveryUI.
cc_library_static {
name: "librecovery_ui_wear",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"wear_device.cpp",
],
}

// The default VR device that uses VrRecoveryUI.
cc_library_static {
name: "librecovery_ui_vr",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"vr_device.cpp",
],
}

cc_library_static {
name: "librecovery_fastboot",
recovery_available: true,
Expand All @@ -113,6 +42,7 @@ cc_library_static {
"libbootloader_message",
"libcutils",
"liblog",
"librecovery_ui",
],

static_libs: [
Expand Down Expand Up @@ -180,6 +110,10 @@ cc_library_static {
"roots.cpp",
],

shared_libs: [
"librecovery_ui",
],

include_dirs: [
"system/vold",
],
Expand Down
2 changes: 1 addition & 1 deletion adb_install.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
#include "common.h"
#include "fuse_sideload.h"
#include "install.h"
#include "ui.h"
#include "recovery_ui/ui.h"

int apply_from_adb(bool* wipe_cache) {
// Save the usb state to restore after the sideload operation.
Expand Down
3 changes: 1 addition & 2 deletions fastboot/fastboot.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@
#include <android-base/properties.h>
#include <bootloader_message/bootloader_message.h>

#include "device.h"
#include "ui.h"
#include "recovery_ui/ui.h"

static const std::vector<std::pair<std::string, Device::BuiltinAction>> kFastbootMenuActions{
{ "Reboot system now", Device::REBOOT },
Expand Down
2 changes: 1 addition & 1 deletion fastboot/fastboot.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@
#include <string>
#include <vector>

#include "device.h"
#include "recovery_ui/device.h"

Device::BuiltinAction StartFastboot(Device* device, const std::vector<std::string>& args);
4 changes: 2 additions & 2 deletions fuse_sdcard_install.h
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

#pragma once

#include "device.h"
#include "ui.h"
#include "recovery_ui/device.h"
#include "recovery_ui/ui.h"

int ApplyFromSdcard(Device* device, bool* wipe_cache, RecoveryUI* ui);
2 changes: 1 addition & 1 deletion install.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@
#include "otautil/thermalutil.h"
#include "package.h"
#include "private/install.h"
#include "recovery_ui/ui.h"
#include "roots.h"
#include "ui.h"
#include "verifier.h"

using namespace std::chrono_literals;
Expand Down
5 changes: 2 additions & 3 deletions recovery.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@

#include "adb_install.h"
#include "common.h"
#include "device.h"
#include "fsck_unshare_blocks.h"
#include "fuse_sdcard_install.h"
#include "install.h"
Expand All @@ -62,9 +61,9 @@
#include "otautil/paths.h"
#include "otautil/sysutil.h"
#include "package.h"
#include "recovery_ui/screen_ui.h"
#include "recovery_ui/ui.h"
#include "roots.h"
#include "screen_ui.h"
#include "ui.h"

static constexpr const char* CACHE_LOG_DIR = "/cache/recovery";
static constexpr const char* COMMAND_FILE = "/cache/recovery/command";
Expand Down
2 changes: 1 addition & 1 deletion recovery.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@
#include <string>
#include <vector>

#include "device.h"
#include "recovery_ui/device.h"

Device::BuiltinAction start_recovery(Device* device, const std::vector<std::string>& args);
6 changes: 3 additions & 3 deletions recovery_main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -49,16 +49,16 @@
#include <selinux/selinux.h>

#include "common.h"
#include "device.h"
#include "fastboot/fastboot.h"
#include "logging.h"
#include "minadbd/minadbd.h"
#include "otautil/paths.h"
#include "otautil/sysutil.h"
#include "recovery.h"
#include "recovery_ui/device.h"
#include "recovery_ui/stub_ui.h"
#include "recovery_ui/ui.h"
#include "roots.h"
#include "stub_ui.h"
#include "ui.h"

static constexpr const char* COMMAND_FILE = "/cache/recovery/command";
static constexpr const char* LOCALE_FILE = "/cache/recovery/last_locale";
Expand Down
91 changes: 91 additions & 0 deletions recovery_ui/Android.bp
Original file line number Diff line number Diff line change
@@ -0,0 +1,91 @@
// Copyright (C) 2019 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

cc_library {
name: "librecovery_ui",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"device.cpp",
"screen_ui.cpp",
"ui.cpp",
"vr_ui.cpp",
"wear_ui.cpp",
],

export_include_dirs: ["include"],

static_libs: [
"libminui",
"libotautil",
],

shared_libs: [
"libbase",
"libpng",
"libz",
],
}

// Generic device that uses ScreenRecoveryUI.
cc_library_static {
name: "librecovery_ui_default",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"default_device.cpp",
],

export_include_dirs: ["include"],
}

// The default wear device that uses WearRecoveryUI.
cc_library_static {
name: "librecovery_ui_wear",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"wear_device.cpp",
],

export_include_dirs: ["include"],
}

// The default VR device that uses VrRecoveryUI.
cc_library_static {
name: "librecovery_ui_vr",
recovery_available: true,

defaults: [
"recovery_defaults",
],

srcs: [
"vr_device.cpp",
],

export_include_dirs: ["include"],
}
4 changes: 2 additions & 2 deletions default_device.cpp → recovery_ui/default_device.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
* limitations under the License.
*/

#include "device.h"
#include "screen_ui.h"
#include "recovery_ui/device.h"
#include "recovery_ui/screen_ui.h"

Device* make_device() {
return new Device(new ScreenRecoveryUI);
Expand Down
4 changes: 2 additions & 2 deletions device.cpp → recovery_ui/device.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
* limitations under the License.
*/

#include "device.h"
#include "recovery_ui/device.h"

#include <algorithm>
#include <string>
Expand All @@ -23,7 +23,7 @@

#include <android-base/logging.h>

#include "ui.h"
#include "recovery_ui/ui.h"

static std::vector<std::pair<std::string, Device::BuiltinAction>> g_menu_actions{
{ "Reboot system now", Device::REBOOT },
Expand Down
2 changes: 1 addition & 1 deletion device.h → recovery_ui/include/recovery_ui/device.h
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ class Device {

// Called when recovery starts up (after the UI has been obtained and initialized and after the
// arguments have been parsed, but before anything else).
virtual void StartRecovery() {};
virtual void StartRecovery() {}

// Called from the main thread when recovery is at the main menu and waiting for input, and a key
// is pressed. (Note that "at" the main menu does not necessarily mean the menu is visible;
Expand Down
File renamed without changes.
File renamed without changes.
12 changes: 6 additions & 6 deletions ui.h → recovery_ui/include/recovery_ui/ui.h
Original file line number Diff line number Diff line change
Expand Up @@ -35,20 +35,20 @@ class RecoveryUI {
INSTALLING_UPDATE,
ERASING,
NO_COMMAND,
ERROR
ERROR,
};

enum ProgressType {
EMPTY,
INDETERMINATE,
DETERMINATE
DETERMINATE,
};

enum KeyAction {
ENQUEUE,
TOGGLE,
REBOOT,
IGNORE
IGNORE,
};

enum class KeyError : int {
Expand All @@ -60,8 +60,8 @@ class RecoveryUI {

virtual ~RecoveryUI();

// Initializes the object; called before anything else. UI texts will be initialized according to
// the given locale. Returns true on success.
// Initializes the object; called before anything else. UI texts will be initialized according
// to the given locale. Returns true on success.
virtual bool Init(const std::string& locale);

virtual std::string GetLocale() const = 0;
Expand Down Expand Up @@ -211,7 +211,7 @@ class RecoveryUI {
DISABLED,
NORMAL,
DIMMED,
OFF
OFF,
};

// The sensitivity when detecting a swipe.
Expand Down
File renamed without changes.
File renamed without changes.
Loading

0 comments on commit b5108c3

Please sign in to comment.