Reviewed by Kevin Decker
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Feb 2007 01:35:33 +0000 (01:35 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Feb 2007 01:35:33 +0000 (01:35 +0000)
        - fixed all places in WebKit where _web_userVisibleString was used where
        _web_originalDataAsString should have been used instead.

        * History/WebURLsWithTitles.m:
        (+[WebURLsWithTitles writeURLs:andTitles:toPasteboard:]):
        use _web_originalDataAsString when writing since these aren't displayed to the user
        (+[WebURLsWithTitles URLsFromPasteboard:]):
        use _web_URLWithDataAsString when reading, to match what we used when writing

        * Misc/WebNSPasteboardExtras.mm:
        (-[NSPasteboard _web_writeURL:andTitle:types:]):
        use _web_originalDataAsString when writing the NSURL type; continue using
        _web_userVisibleString when writing the plain text type

        * WebView/WebHTMLView.mm:
        (-[WebHTMLView _documentFragmentWithPaths:]):
        added comment about why _web_userVisibleString is appropriate here
        (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]):
        use _web_originalDataAsString when setting the href attribute of an anchor tag

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

WebKit/ChangeLog
WebKit/History/WebURLsWithTitles.m
WebKit/Misc/WebNSPasteboardExtras.mm
WebKit/WebView/WebHTMLView.mm

index e1bc7b6..7b1010a 100644 (file)
@@ -1,3 +1,27 @@
+2007-02-07  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Kevin Decker
+
+        - fixed all places in WebKit where _web_userVisibleString was used where
+        _web_originalDataAsString should have been used instead.
+
+        * History/WebURLsWithTitles.m:
+        (+[WebURLsWithTitles writeURLs:andTitles:toPasteboard:]):
+        use _web_originalDataAsString when writing since these aren't displayed to the user
+        (+[WebURLsWithTitles URLsFromPasteboard:]):
+        use _web_URLWithDataAsString when reading, to match what we used when writing
+
+        * Misc/WebNSPasteboardExtras.mm:
+        (-[NSPasteboard _web_writeURL:andTitle:types:]):
+        use _web_originalDataAsString when writing the NSURL type; continue using
+        _web_userVisibleString when writing the plain text type
+
+        * WebView/WebHTMLView.mm:
+        (-[WebHTMLView _documentFragmentWithPaths:]):
+        added comment about why _web_userVisibleString is appropriate here
+        (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]):
+        use _web_originalDataAsString when setting the href attribute of an anchor tag
+
 2007-02-07  David Harrison  <harrison@apple.com>
 
         Reviewed by Adam.
@@ -7,7 +31,7 @@
         Problem is that AppKit recently changed NSControl to trigger a full redraw if the control has a focus ring.
         WebHTMLView is a subclass of NSControl, but the focus ring type was the default value, though we actually
         draw no focus ring. Fix is to formally set our focus ring type.
-        
+
         * WebView/WebHTMLView.mm:
         (-[WebHTMLView initWithFrame:]):
         Send [self setFocusRingType:NSFocusRingTypeNone].
index c8d216b..df9ef75 100644 (file)
@@ -67,7 +67,7 @@
     URLStrings = [NSMutableArray arrayWithCapacity:count];
     titlesOrEmptyStrings = [NSMutableArray arrayWithCapacity:count];
     for (index = 0; index < count; ++index) {
-        [URLStrings addObject:[[URLs objectAtIndex:index] _web_userVisibleString]];
+        [URLStrings addObject:[[URLs objectAtIndex:index] _web_originalDataAsString]];
         [titlesOrEmptyStrings addObject:(titles == nil) ? @"" : [[titles objectAtIndex:index] _webkit_stringByTrimmingWhitespace]];
     }
 
@@ -98,7 +98,7 @@
     count = [URLStrings count];
     URLs = [NSMutableArray arrayWithCapacity:count];
     for (index = 0; index < count; ++index) {
-        [URLs addObject:[NSURL _web_URLWithUserTypedString:[URLStrings objectAtIndex:index]]];
+        [URLs addObject:[NSURL _web_URLWithDataAsString:[URLStrings objectAtIndex:index]]];
     }
 
     return URLs;
index b4807a9..270d8f9 100644 (file)
@@ -188,7 +188,7 @@ static NSArray *_writableTypesForImageWithArchive (void)
         [URL writeToPasteboard:self];
     }
     if ([types containsObject:WebURLPboardType]) {
-        [self setString:[URL _web_userVisibleString] forType:WebURLPboardType];
+        [self setString:[URL _web_originalDataAsString] forType:WebURLPboardType];
     }
     if ([types containsObject:WebURLNamePboardType]) {
         [self setString:title forType:WebURLNamePboardType];
index 89c6dce..849b2f7 100644 (file)
@@ -377,7 +377,8 @@ struct WebHTMLViewInterpretKeyEventsParameters {
                 [resource release];
             }
         } else {
-            // Non-image file types
+            // Non-image file types; _web_userVisibleString is appropriate here because this will
+            // be pasted as visible text.
             NSString *url = [[[NSURL fileURLWithPath:path] _webkit_canonicalize] _web_userVisibleString];
             [domNodes addObject:[[[self _frame] DOMDocument] createTextNode: url]];
         }
@@ -526,7 +527,7 @@ static NSURL* uniqueURLWithRelativePart(NSString *relativePart)
         ASSERT(document);
         if (document) {
             DOMHTMLAnchorElement* anchor = (DOMHTMLAnchorElement*)[document createElement:@"a"];
-            NSString *URLString = [URL _web_userVisibleString];
+            NSString *URLString = [URL _web_originalDataAsString];
             NSString *URLTitleString = [pasteboard stringForType:WebURLNamePboardType];
             DOMText* text = [document createTextNode:URLTitleString];
             [anchor setHref:URLString];