Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5cf6e2b1 authored by Gavin Corkery's avatar Gavin Corkery Committed by Automerger Merge Worker
Browse files

Merge changes from topic "remove-duplicate-br-calls" am: 9b795b5f am: f2879207 am: 43230a92 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1508497

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ied17ebbc7e9baa94e68a7c5e1212bbd036b58724
parents ad94c459 43230a92

cmds/dumpstate/dumpstate.cpp

+10 −18
Original line number Diff line number Diff line
@@ -2864,31 +2864,23 @@ Dumpstate::RunStatus Dumpstate::RunInternal(int32_t calling_uid, 
    // duration is logged into MYLOG instead.

    PrintHeader();



    // TODO(b/158737089) reduce code repetition in if branches

    if (options_->telephony_only) {

    bool is_dumpstate_restricted = options_->telephony_only

                                   || options_->wifi_only

                                   || options_->limited_only;

    if (!is_dumpstate_restricted) {

        // Invoke critical dumpsys first to preserve system state, before doing anything else.

        RunDumpsysCritical();

    }

    MaybeTakeEarlyScreenshot();

    onUiIntensiveBugreportDumpsFinished(calling_uid);

    MaybeCheckUserConsent(calling_uid, calling_package);

    if (options_->telephony_only) {

        DumpstateTelephonyOnly(calling_package);

    } else if (options_->wifi_only) {

        MaybeTakeEarlyScreenshot();

        onUiIntensiveBugreportDumpsFinished(calling_uid);

        MaybeCheckUserConsent(calling_uid, calling_package);

        DumpstateWifiOnly();

    } else if (options_->limited_only) {

        MaybeTakeEarlyScreenshot();

        onUiIntensiveBugreportDumpsFinished(calling_uid);

        MaybeCheckUserConsent(calling_uid, calling_package);

        DumpstateLimitedOnly();

    } else {

        // Invoke critical dumpsys first to preserve system state, before doing anything else.

        RunDumpsysCritical();



        // Take screenshot and get consent only after critical dumpsys has finished.

        MaybeTakeEarlyScreenshot();

        onUiIntensiveBugreportDumpsFinished(calling_uid);

        MaybeCheckUserConsent(calling_uid, calling_package);



        // Dump state for the default case. This also drops root.

        RunStatus s = DumpstateDefaultAfterCritical();

        if (s != RunStatus::OK) {