Title: [149028] trunk/Source/WebCore
Revision
149028
Author
[email protected]
Date
2013-04-24 05:17:32 -0700 (Wed, 24 Apr 2013)

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

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (149027 => 149028)


--- trunk/Source/WebCore/ChangeLog	2013-04-24 12:01:37 UTC (rev 149027)
+++ trunk/Source/WebCore/ChangeLog	2013-04-24 12:17:32 UTC (rev 149028)
@@ -1,3 +1,15 @@
+2013-04-24  Kangil Han  <[email protected]>
+
+        [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  <[email protected]>
 
         [EFL] Fix build warnings caused by missing images attributes for default buttons

Modified: trunk/Source/WebCore/page/SpatialNavigation.cpp (149027 => 149028)


--- trunk/Source/WebCore/page/SpatialNavigation.cpp	2013-04-24 12:01:37 UTC (rev 149027)
+++ trunk/Source/WebCore/page/SpatialNavigation.cpp	2013-04-24 12:17:32 UTC (rev 149028)
@@ -234,7 +234,6 @@
     //
     // ... and variants of the above cases.
     return ((bStart >= aStart && bStart <= aEnd)
-            || (bEnd >= aStart && bEnd <= aEnd)
             || (bMiddle >= aStart && bMiddle <= aEnd)
             || (bEnd >= aStart && bEnd <= aEnd));
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to