[Minor code cleanup] Remove duplicated condition check
authorkangil.han@samsung.com <kangil.han@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Apr 2013 12:17:32 +0000 (12:17 +0000)
committerkangil.han@samsung.com <kangil.han@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Apr 2013 12:17:32 +0000 (12:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=115082

Reviewed by Andreas Kling.

areRectsPartiallyAligned() has duplicated condition check, so remove it.

* page/SpatialNavigation.cpp:
(WebCore::areRectsPartiallyAligned):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@149028 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/SpatialNavigation.cpp

index bd7a825..48e789e 100644 (file)
@@ -1,3 +1,15 @@
+2013-04-24  Kangil Han  <kangil.han@samsung.com>
+
+        [Minor code cleanup] Remove duplicated condition check
+        https://bugs.webkit.org/show_bug.cgi?id=115082
+
+        Reviewed by Andreas Kling.
+
+        areRectsPartiallyAligned() has duplicated condition check, so remove it.
+
+        * page/SpatialNavigation.cpp:
+        (WebCore::areRectsPartiallyAligned):
+
 2013-04-24  Krzysztof Wolanski  <k.wolanski@samsung.com>
 
         [EFL] Fix build warnings caused by missing images attributes for default buttons
index e7dc8fb..c4f1f04 100644 (file)
@@ -234,7 +234,6 @@ static bool areRectsPartiallyAligned(FocusDirection direction, const LayoutRect&
     //
     // ... and variants of the above cases.
     return ((bStart >= aStart && bStart <= aEnd)
-            || (bEnd >= aStart && bEnd <= aEnd)
             || (bMiddle >= aStart && bMiddle <= aEnd)
             || (bEnd >= aStart && bEnd <= aEnd));
 }