diff options
author | Mihail Dumitrescu <mdumitrescu@google.com> | 2014-01-03 12:40:22 +0000 |
---|---|---|
committer | Danesh M <daneshm90@gmail.com> | 2014-06-06 22:54:23 -0700 |
commit | 70b1a016070f28088b446846326191d1cb10f244 (patch) | |
tree | 16b16443bc4722f526b058dfc21bb2dff50031a8 /src/com/android | |
parent | 5b52f725a69f2e5d2ebc751163d2ef71df8ad2d7 (diff) | |
download | android_packages_apps_Trebuchet-70b1a016070f28088b446846326191d1cb10f244.tar.gz android_packages_apps_Trebuchet-70b1a016070f28088b446846326191d1cb10f244.tar.bz2 android_packages_apps_Trebuchet-70b1a016070f28088b446846326191d1cb10f244.zip |
Remove a non-obvious fall through in a switch statement.
This CL does not modify the result of calling onInterceptHoverEvent()
assuming isEventOverFolder() has no side effects. The only change is
that isEventOverFolder will now be called once rather than twice when
action == MotionEvent.ACTION_HOVER_ENTER and isOverFolder == true.
Change-Id: Icc28ab880cf5094e49f51d72035891e924cb2031
Diffstat (limited to 'src/com/android')
-rw-r--r-- | src/com/android/launcher3/DragLayer.java | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/com/android/launcher3/DragLayer.java b/src/com/android/launcher3/DragLayer.java index 261e61fdc..2d3acad58 100644 --- a/src/com/android/launcher3/DragLayer.java +++ b/src/com/android/launcher3/DragLayer.java @@ -227,22 +227,19 @@ public class DragLayer extends FrameLayout implements ViewGroup.OnHierarchyChang sendTapOutsideFolderAccessibilityEvent(currentFolder.isEditingName()); mHoverPointClosesFolder = true; return true; - } else if (isOverFolder) { - mHoverPointClosesFolder = false; - } else { - return true; } + mHoverPointClosesFolder = false; + break; case MotionEvent.ACTION_HOVER_MOVE: isOverFolder = isEventOverFolder(currentFolder, ev); if (!isOverFolder && !mHoverPointClosesFolder) { sendTapOutsideFolderAccessibilityEvent(currentFolder.isEditingName()); mHoverPointClosesFolder = true; return true; - } else if (isOverFolder) { - mHoverPointClosesFolder = false; - } else { + } else if (!isOverFolder) { return true; } + mHoverPointClosesFolder = false; } } } |