Changeset 149028 in webkit

Timestamp:

Apr 24, 2013 5:17:32 AM (11 years ago)

Author:

kangil.han@samsung.com

Message:

[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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/SpatialNavigation.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+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>
 

trunk/Source/WebCore/page/SpatialNavigation.cpp

@@ -235 +235 @@
     // ... and variants of the above cases.
     return ((bStart >= aStart && bStart <= aEnd)
-            || (bEnd >= aStart && bEnd <= aEnd)
             || (bMiddle >= aStart && bMiddle <= aEnd)
             || (bEnd >= aStart && bEnd <= aEnd));