Title: [104066] trunk/Source/WebKit/mac
Revision
104066
Author
[email protected]
Date
2012-01-04 14:02:53 -0800 (Wed, 04 Jan 2012)

Log Message

Ambiguous if-then-else in Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm
https://bugs.webkit.org/show_bug.cgi?id=75447

Reviewed by Brian Weinstein.

Resolve the ambiguity.

* Misc/WebNSPasteboardExtras.mm:
(-[NSPasteboard _web_writeImage:element:URL:title:archive:types:source:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit/mac/ChangeLog (104065 => 104066)


--- trunk/Source/WebKit/mac/ChangeLog	2012-01-04 22:01:14 UTC (rev 104065)
+++ trunk/Source/WebKit/mac/ChangeLog	2012-01-04 22:02:53 UTC (rev 104066)
@@ -1,3 +1,15 @@
+2012-01-04  Anders Carlsson  <[email protected]>
+
+        Ambiguous if-then-else in Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm
+        https://bugs.webkit.org/show_bug.cgi?id=75447
+
+        Reviewed by Brian Weinstein.
+
+        Resolve the ambiguity.
+
+        * Misc/WebNSPasteboardExtras.mm:
+        (-[NSPasteboard _web_writeImage:element:URL:title:archive:types:source:]):
+
 2012-01-04  Mihnea Ovidenie  <[email protected]>
 
         [Mac]Fix build after changeset 103997

Modified: trunk/Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm (104065 => 104066)


--- trunk/Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm	2012-01-04 22:01:14 UTC (rev 104065)
+++ trunk/Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm	2012-01-04 22:02:53 UTC (rev 104066)
@@ -248,14 +248,15 @@
             [self setData:[element _imageTIFFRepresentation] forType:NSTIFFPboardType];
     }
     
-    if (archive)
+    if (archive) {
         if ([types containsObject:WebArchivePboardType])
             [self setData:[archive data] forType:WebArchivePboardType];
-    else {
-        // We should not have declared types that we aren't going to write (4031826).
-        ASSERT(![types containsObject:NSRTFDPboardType]);
-        ASSERT(![types containsObject:WebArchivePboardType]);
+        return;
     }
+
+    // We should not have declared types that we aren't going to write (4031826).
+    ASSERT(![types containsObject:NSRTFDPboardType]);
+    ASSERT(![types containsObject:WebArchivePboardType]);
 }
 
 - (id)_web_declareAndWriteDragImageForElement:(DOMElement *)element
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to