diff options
author | Nebojsa Cvetkovic <nebkat@gmail.com> | 2013-06-14 11:13:16 +0100 |
---|---|---|
committer | Nebojsa Cvetkovic <nebkat@gmail.com> | 2013-06-14 11:13:16 +0100 |
commit | 7b71954ab635a85cfd6adf3b83c7bdaf614cec26 (patch) | |
tree | e2ab093f6a3ca8a4473685ac5d531ef8c52478ee | |
parent | 02faeef789f7837ed7ae95bde00aec804915f8bd (diff) | |
download | packages_apps_trebuchet-7b71954ab635a85cfd6adf3b83c7bdaf614cec26.zip packages_apps_trebuchet-7b71954ab635a85cfd6adf3b83c7bdaf614cec26.tar.gz packages_apps_trebuchet-7b71954ab635a85cfd6adf3b83c7bdaf614cec26.tar.bz2 |
Revert "Trebuchet: Fix double tab on home button"
This reverts commit 6a198e07c88224bfbbed4db41e2e664554903c2b.
-rw-r--r-- | src/com/cyanogenmod/trebuchet/Launcher.java | 63 |
1 files changed, 8 insertions, 55 deletions
diff --git a/src/com/cyanogenmod/trebuchet/Launcher.java b/src/com/cyanogenmod/trebuchet/Launcher.java index 7208a9a..7569573 100644 --- a/src/com/cyanogenmod/trebuchet/Launcher.java +++ b/src/com/cyanogenmod/trebuchet/Launcher.java @@ -258,9 +258,6 @@ public final class Launcher extends Activity private boolean mWaitingForResult; private boolean mOnResumeNeedsLoad; - private final List<Runnable> mNewPendingIntents = new ArrayList<Runnable>(); - private boolean mOnNewIntentProcessing; - // Keep track of whether the user has left launcher private static boolean sPausedFromUserAction = false; @@ -1562,20 +1559,15 @@ public final class Launcher extends Activity public void run() { if (mWorkspace == null) { // Can be cases where mWorkspace is null, this prevents a NPE - synchronized (mNewPendingIntents) { - mOnNewIntentProcessing = false; - } return; } Folder openFolder = mWorkspace.getOpenFolder(); // In all these cases, only animate if we're already on home mWorkspace.exitWidgetResizeMode(); - boolean waitForPendingTranstions = true; if (alreadyOnHome && mState == State.WORKSPACE && !mWorkspace.isTouchActive() && openFolder == null) { mWorkspace.moveToDefaultScreen(true); mHotseat.moveToDefaultScreen(true); - waitForPendingTranstions = false; } closeFolder(); @@ -1584,20 +1576,9 @@ public final class Launcher extends Activity // If we are already on home, then just animate back to the workspace, // otherwise, just wait until onResume to set the state back to Workspace if (alreadyOnHome) { - showWorkspace(true, new Runnable() { - @Override - public void run() { - processNewPendingIntents(); - } - }); - // onCompleteRunnable is not called if there is no pending animations - // so we must ensure that processNewPendingIntents is called. - if (!waitForPendingTranstions) { - processNewPendingIntents(); - } + showWorkspace(true); } else { mOnResumeState = State.WORKSPACE; - processNewPendingIntents(); } final View v = getWindow().peekDecorView(); @@ -1614,43 +1595,15 @@ public final class Launcher extends Activity } }; - boolean markAsPending = false; - synchronized (mNewPendingIntents) { - if (mOnNewIntentProcessing) { - mNewPendingIntents.add(processIntent); - markAsPending = true; - } else { - mOnNewIntentProcessing = true; - } - } - - if (!markAsPending) { - if (alreadyOnHome && !mWorkspace.hasWindowFocus()) { - // Delay processing of the intent to allow the status bar animation to finish - // first in order to avoid janky animations. - mWorkspace.postDelayed(processIntent, 350); - } else { - // Process the intent immediately. - processIntent.run(); - } - } - - } - } - - private void processNewPendingIntents() { - Runnable newIntent = null; - synchronized (mNewPendingIntents) { - if (mNewPendingIntents.size() > 0) { - if (mWorkspace != null) { - newIntent = mNewPendingIntents.remove(0); - mWorkspace.post(newIntent); - } else { - mOnNewIntentProcessing = false; - } + if (alreadyOnHome && !mWorkspace.hasWindowFocus()) { + // Delay processing of the intent to allow the status bar animation to finish + // first in order to avoid janky animations. + mWorkspace.postDelayed(processIntent, 350); } else { - mOnNewIntentProcessing = false; + // Process the intent immediately. + processIntent.run(); } + } } |