commit 5344cd98e69f92e70d52969b1851c9d8f9e81853
author Jinsuk Kim <jinsukkim@google.com> Thu Jul 03 18:02:01 2014 +0900
committer Jinsuk Kim <jinsukkim@google.com> Fri Jul 04 13:57:17 2014 +0900
tree c9a13d30a8ab0de0eb9e36e9c4047e88b11e8b71
parent 0f94217e9043e9dee329fa7167ee5be75eef0c23

Launch routing control in HdmiControlService at boot up

Change-Id: I918c2a0da26a7cda20ee703dfeeacae276ec8373

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

diff --git a/services/core/java/com/android/server/hdmi/HdmiCecLocalDeviceTv.java b/services/core/java/com/android/server/hdmi/HdmiCecLocalDeviceTv.java
index eb80b20..810e670 100644
--- a/services/core/java/com/android/server/hdmi/HdmiCecLocalDeviceTv.java
+++ b/services/core/java/com/android/server/hdmi/HdmiCecLocalDeviceTv.java
@@ -98,9 +98,8 @@
                 mAddress, mService.getPhysicalAddress(), mDeviceType));
         mService.sendCecCommand(HdmiCecMessageBuilder.buildDeviceVendorIdCommand(
                 mAddress, mService.getVendorId()));
-
+        launchRoutingControl(true);
         launchDeviceDiscovery();
-        // TODO: Start routing control action
     }
 
     /**
@@ -945,24 +944,33 @@
         }
     }
 
+    /**
+     * Launch routing control process.
+     *
+     * @param routingForBootup true if routing control is initiated due to One Touch Play
+     *        or TV power on
+     */
     @ServiceThreadOnly
-    void routingAtEnableTime() {
+    void launchRoutingControl(boolean routingForBootup) {
         assertRunOnServiceThread();
         // Seq #24
         if (getActivePortId() != HdmiConstants.INVALID_PORT_ID) {
-            // TODO: Check if TV was not powered on due to <Text/Image View On>,
-            //       TV is not in Preset Installation mode, not in initial setup mode, not
-            //       in Software updating mode, not in service mode, for following actions.
-            removeAction(RoutingControlAction.class);
-            int newPath = mService.portIdToPath(getActivePortId());
-            mService.sendCecCommand(
-                    HdmiCecMessageBuilder.buildRoutingChange(mAddress, getActivePath(), newPath));
-            addAndStartAction(new RoutingControlAction(this, getActivePortId(), false, null));
+            if (!routingForBootup && !isProhibitMode()) {
+                removeAction(RoutingControlAction.class);
+                int newPath = mService.portIdToPath(getActivePortId());
+                setActivePath(newPath);
+                mService.sendCecCommand(HdmiCecMessageBuilder.buildRoutingChange(mAddress,
+                        getActivePath(), newPath));
+                addAndStartAction(new RoutingControlAction(this, getActivePortId(),
+                        routingForBootup, null));
+            }
         } else {
             int activePath = mService.getPhysicalAddress();
             setActivePath(activePath);
-            // TODO: Do following only when TV was not powered on due to <Text/Image View On>.
-            mService.sendCecCommand(HdmiCecMessageBuilder.buildActiveSource(mAddress, activePath));
+            if (!routingForBootup) {
+                mService.sendCecCommand(HdmiCecMessageBuilder.buildActiveSource(mAddress,
+                        activePath));
+            }
         }
     }
 

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

diff --git a/services/core/java/com/android/server/hdmi/HdmiControlService.java b/services/core/java/com/android/server/hdmi/HdmiControlService.java
index 6dd3250..1fdb3ec 100644
--- a/services/core/java/com/android/server/hdmi/HdmiControlService.java
+++ b/services/core/java/com/android/server/hdmi/HdmiControlService.java
@@ -828,7 +828,7 @@
                     if (mMhlController != null) {
                         mMhlController.setOption(HdmiCec.OPTION_MHL_ENABLE, value);
                     }
-                    tv.routingAtEnableTime();
+                    tv.launchRoutingControl(false);
                 }
             });
         }