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

Commit 45bd213f authored by Jinsuk Kim's avatar Jinsuk Kim Committed by Android Git Automerger
Browse files

am 0ac8928e: Fix bugs in active source command handling in HdmiControl 

* commit '0ac8928ec1b5065d28c08198c6c0b9d98b9639b0':
  Fix bugs in active source command handling in HdmiControl
parents fdb568c0 7c3a9563

services/core/java/com/android/server/hdmi/ActiveSourceHandler.java

+3 −5
Original line number Diff line number Diff line
@@ -71,18 +71,16 @@ final class ActiveSourceHandler { 
            tv.startNewDeviceAction(newActive);

        }



        ActiveSource current = tv.getActiveSource();

        if (!tv.isProhibitMode()) {

            tv.updateActiveSource(newActive);

            if (!current.equals(newActive)) {

            boolean notifyInputChange = (mCallback == null);

            tv.updateActiveInput(newActive.physicalAddress, notifyInputChange);

            }

            invokeCallback(HdmiControlManager.RESULT_SUCCESS);

        } else {

            // TV is in a mode that should keep its current source/input from

            // being changed for its operation. Reclaim the active source

            // or switch the port back to the one used for the current mode.

            ActiveSource current = tv.getActiveSource();

            if (current.logicalAddress == getSourceAddress()) {

                HdmiCecMessage activeSourceCommand = HdmiCecMessageBuilder.buildActiveSource(

                        current.logicalAddress, current.physicalAddress);

services/core/java/com/android/server/hdmi/HdmiCecLocalDeviceTv.java

+3 −2
Original line number Diff line number Diff line
@@ -238,9 +238,9 @@ final class HdmiCecLocalDeviceTv extends HdmiCecLocalDevice { 
        if (path == getActivePath()) {

            return;

        }

        setPrevPortId(getActivePortId());

        int portId = mService.pathToPortId(path);

        setActivePath(path);

        setPrevPortId(portId);

        // TODO: Handle PAP/PIP case.

        // Show OSD port change banner

        if (notifyInputChange) {
@@ -440,9 +440,10 @@ final class HdmiCecLocalDeviceTv extends HdmiCecLocalDevice { 
        if (isTailOfActivePath(path, getActivePath())) {

            removeAction(RoutingControlAction.class);

            int newPath = mService.portIdToPath(getActivePortId());

            setActivePath(newPath);

            mService.sendCecCommand(HdmiCecMessageBuilder.buildRoutingChange(

                    mAddress, getActivePath(), newPath));

            addAndStartAction(new RoutingControlAction(this, getActivePortId(), false, null));

            addAndStartAction(new RoutingControlAction(this, newPath, false, null));

        }

    }

services/core/java/com/android/server/hdmi/RoutingControlAction.java

+2 −2
Original line number Diff line number Diff line
@@ -101,12 +101,12 @@ final class RoutingControlAction extends FeatureAction { 
            // If the routing path doesn't belong to the currently active one, we should

            // ignore it since it might have come from other routing change sequence.

            int routingPath = HdmiUtils.twoBytesToInt(params);

            if (HdmiUtils.isInActiveRoutingPath(mCurrentRoutingPath, routingPath)) {

            if (!HdmiUtils.isInActiveRoutingPath(mCurrentRoutingPath, routingPath)) {

                return true;

            }

            mCurrentRoutingPath = routingPath;

            // Stop possible previous routing change sequence if in progress.

            removeAction(RoutingControlAction.class);

            removeActionExcept(RoutingControlAction.class, this);

            addTimer(mState, TIMEOUT_ROUTING_INFORMATION_MS);

            return true;

        } else if (mState == STATE_WAIT_FOR_REPORT_POWER_STATUS