Revert "recovery: Ask to reboot before installing anything after A/B update" (f3a00356) · Commits · e / os / android_bootable_recovery

install/install.cpp

+1 −11

Original line number	Diff line number	Diff line
		@@ -63,7 +63,6 @@

		using namespace std::chrono_literals;

		bool ask_to_ab_reboot(Device* device);
		bool ask_to_continue_unverified(Device* device);
		bool ask_to_continue_downgrade(Device* device);

		@@ -392,14 +391,6 @@ static InstallResult TryUpdateBinary(Package* package, bool* wipe_cache,
		Reboot("recovery");
		};

		static bool ab_package_installed = false;
		if (ab_package_installed) {
		if (ask_to_ab_reboot(device)) {
		reboot_to_recovery();
		}
		return INSTALL_ERROR;
		}

		if (package_is_ab) {
		CHECK(package->GetType() == PackageType::kFile);
		}
		@@ -585,9 +576,8 @@ static InstallResult TryUpdateBinary(Package* package, bool* wipe_cache,
		LOG(FATAL) << "Invalid status code " << status;
		}
		if (package_is_ab) {
		ab_package_installed = true;
		PerformPowerwashIfRequired(zip, device);
		if (!ui->IsSideloadAutoReboot() && ask_to_ab_reboot(device)) {
		if (!ui->IsSideloadAutoReboot()) {
		reboot_to_recovery();
		}
		}

+0 −6

Original line number	Diff line number	Diff line
		@@ -175,12 +175,6 @@ static bool yes_no(Device* device, const char* question1, const char* question2)
		return (chosen_item == 1);
		}

		bool ask_to_ab_reboot(Device* device) {
		device->GetUI()->SetProgressType(RecoveryUI::EMPTY);
		return yes_no(device, "To install additional packages, you need to reboot recovery first",
		"Do you want to reboot to recovery now?");
		}

		bool ask_to_continue_unverified(Device* device) {
		if (get_build_type() == "user") {
		return false;