Title: [173442] trunk/Source/WebKit2
Revision
173442
Author
m...@apple.com
Date
2014-09-09 14:43:57 -0700 (Tue, 09 Sep 2014)

Log Message

[Cocoa] -[WKBackForwardList backList] contains an unnecessary nil check
https://bugs.webkit.org/show_bug.cgi?id=136680

Reviewed by Anders Carlsson.

* UIProcess/API/Cocoa/WKBackForwardList.mm:
(-[WKBackForwardList backList]): Removed nil check.
(-[WKBackForwardList forwardList]): Ditto.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (173441 => 173442)


--- trunk/Source/WebKit2/ChangeLog	2014-09-09 21:03:29 UTC (rev 173441)
+++ trunk/Source/WebKit2/ChangeLog	2014-09-09 21:43:57 UTC (rev 173442)
@@ -1,3 +1,14 @@
+2014-09-09  Dan Bernstein  <m...@apple.com>
+
+        [Cocoa] -[WKBackForwardList backList] contains an unnecessary nil check
+        https://bugs.webkit.org/show_bug.cgi?id=136680
+
+        Reviewed by Anders Carlsson.
+
+        * UIProcess/API/Cocoa/WKBackForwardList.mm:
+        (-[WKBackForwardList backList]): Removed nil check.
+        (-[WKBackForwardList forwardList]): Ditto.
+
 2014-09-09  Tim Horton  <timothy_hor...@apple.com>
 
         Make DrawingArea a real MessageReceiver

Modified: trunk/Source/WebKit2/UIProcess/API/Cocoa/WKBackForwardList.mm (173441 => 173442)


--- trunk/Source/WebKit2/UIProcess/API/Cocoa/WKBackForwardList.mm	2014-09-09 21:03:29 UTC (rev 173441)
+++ trunk/Source/WebKit2/UIProcess/API/Cocoa/WKBackForwardList.mm	2014-09-09 21:43:57 UTC (rev 173442)
@@ -75,18 +75,12 @@
 - (NSArray *)backList
 {
     RefPtr<API::Array> list = _list->backList();
-    if (!list)
-        return nil;
-
     return [wrapper(*list.release().leakRef()) autorelease];
 }
 
 - (NSArray *)forwardList
 {
     RefPtr<API::Array> list = _list->forwardList();
-    if (!list)
-        return nil;
-
     return [wrapper(*list.release().leakRef()) autorelease];
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to