Reviewed by Dave Hyatt.
authorharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Apr 2005 00:05:24 +0000 (00:05 +0000)
committerharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Apr 2005 00:05:24 +0000 (00:05 +0000)
        <rdar://problem/4084106> Remove NSAccessibilityForegroundColorTextAttributeWrapper

        * kwq/KWQAccObject.mm:
        (AXAttributeStringSetStyle):
        Use NSAccessibilityForegroundColorTextAttribute directly.

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

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQAccObject.mm

index 2729973a118a2962f0b3a58750e54e5507017acb..effb8b4f74b78611df71e9a302120f2a9a64193b 100644 (file)
@@ -1,3 +1,13 @@
+2005-04-08  David Harrison  <harrison@apple.com>
+
+        Reviewed by Dave Hyatt.
+
+        <rdar://problem/4084106> Remove NSAccessibilityForegroundColorTextAttributeWrapper
+
+        * kwq/KWQAccObject.mm:
+        (AXAttributeStringSetStyle):
+        Use NSAccessibilityForegroundColorTextAttribute directly.
+
 2005-04-05  David Hyatt  <hyatt@apple.com>
 
        Fix for 4077106, make sure that mouse wheeling in overflow sections uses 40 rather than 10 as the base line
index 948ee53604294e5c3cf6973ca1d6a165d300858d..34065fe37a6d22a25d6316daa6535c3c435e8fc4 100644 (file)
@@ -1277,35 +1277,6 @@ static void AXAttributeStringSetFont(NSMutableAttributedString *attrString, NSSt
     
 }
 
-// NOTE: This wrapper is to get the NSAccessibilityForegroundColorTextAttribute string because
-// AppKit changed the name from NSAccessibilityForegoundColorTextAttribute w/o binary compatibliity.
-// It should be removed once everyone is building on the newer AppKit per <rdar://problem/4014691>.
-static NSString * NSAccessibilityForegroundColorTextAttributeWrapper(void)
-{
-    static NSString * axForegroundColorTextAttribute = nil;
-    
-    // find the symbol just once
-    if (axForegroundColorTextAttribute == nil) {
-        // check for the new name
-        NSSymbol axSymbol = NSLookupAndBindSymbol("_NSAccessibilityForegroundColorTextAttribute");
-        
-        // fall back to the old name
-        if (axSymbol == nil)
-            axSymbol = NSLookupAndBindSymbol("_NSAccessibilityForegoundColorTextAttribute");
-        ASSERT(axSymbol);
-        
-        // get the addreess of variable address
-        NSString ** axSymbolAddr = (NSString**) NSAddressOfSymbol(axSymbol);
-        
-        // get and save the string address
-        axForegroundColorTextAttribute = *axSymbolAddr;
-        ASSERT(axForegroundColorTextAttribute);
-    }
-    
-    // return the symbol we can use
-    return axForegroundColorTextAttribute;
-}
-
 static void AXAttributeStringSetStyle(NSMutableAttributedString *attrString, RenderObject *renderer, NSRange range)
 {
     RenderStyle *style = renderer->style();
@@ -1314,7 +1285,7 @@ static void AXAttributeStringSetStyle(NSMutableAttributedString *attrString, Ren
     AXAttributeStringSetFont(attrString, NSAccessibilityFontTextAttribute, style->font().getNSFont(), range);
 
     // set basic colors
-    AXAttributeStringSetColor(attrString, NSAccessibilityForegroundColorTextAttributeWrapper(), style->color().getNSColor(), range);
+    AXAttributeStringSetColor(attrString, NSAccessibilityForegroundColorTextAttribute, style->color().getNSColor(), range);
     AXAttributeStringSetColor(attrString, NSAccessibilityBackgroundColorTextAttribute, style->backgroundColor().getNSColor(), range);
 
     // set super/sub scripting