Don't enforce broadcast timeouts for PRE_BOOT_COMPLETED broadcasts.

They're designed to handle one time upgrade code, which may
take a significant amount of time. Third party code cannot
receive this broadcast so the timing should be able to be
kept under control.

Bug: 2713849
Change-Id: I3733b6530198bebd9b8c2ba6f9f27a7999336b8d

Author: jham

services/java/com/android/server/am/ActivityManagerService.java

@@ -1106,7 +1106,12 @@ public void handleMessage(Message msg) {
                     mHandler.sendMessageDelayed(nmsg, BROADCAST_TIMEOUT);
                     return;
                 }
-                broadcastTimeout();
+                // Only process broadcast timeouts if the system is ready. That way
+                // PRE_BOOT_COMPLETED broadcasts can't timeout as they are intended
+                // to do heavy lifting for system up
+                if (mSystemReady) {
+                    broadcastTimeout();
+                }
             } break;
             case PAUSE_TIMEOUT_MSG: {
                 IBinder token = (IBinder)msg.obj;
@@ -13116,10 +13121,15 @@ private final void processNextBroadcast(boolean fromMsg) {
 
                 // Ensure that even if something goes awry with the timeout
                 // detection, we catch "hung" broadcasts here, discard them,
-                // and continue to make progress.  
+                // and continue to make progress.
+                //
+                // This is only done if the system is ready so that PRE_BOOT_COMPLETED
+                // receivers don't get executed with with timeouts. They're intended for
+                // one time heavy lifting after system upgrades and can take
+                // significant amounts of time.
                 int numReceivers = (r.receivers != null) ? r.receivers.size() : 0;
-                long now = SystemClock.uptimeMillis();
-                if (r.dispatchTime > 0) {
+                if (mSystemReady && r.dispatchTime > 0) {
+                    long now = SystemClock.uptimeMillis();
                     if ((numReceivers > 0) &&
                             (now > r.dispatchTime + (2*BROADCAST_TIMEOUT*numReceivers))) {
                         Slog.w(TAG, "Hung broadcast discarded after timeout failure:"