summaryrefslogtreecommitdiffstats
path: root/src/com/android
diff options
context:
space:
mode:
authorMihail Dumitrescu <mdumitrescu@google.com>2014-01-03 12:40:22 +0000
committerDanesh M <daneshm90@gmail.com>2014-06-06 22:54:23 -0700
commit70b1a016070f28088b446846326191d1cb10f244 (patch)
tree16b16443bc4722f526b058dfc21bb2dff50031a8 /src/com/android
parent5b52f725a69f2e5d2ebc751163d2ef71df8ad2d7 (diff)
downloadandroid_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.java11
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;
}
}
}