2006-09-25 Anders Carlsson <acarlsson@apple.com>
authorandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 25 Sep 2006 08:08:39 +0000 (08:08 +0000)
committerandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 25 Sep 2006 08:08:39 +0000 (08:08 +0000)
        Reviewed by Maciej.

        * css/MediaList.cpp:
        Remove unused header.

        * dom/Document.cpp:
        (WebCore::Document::clear):
        Use clear instead of iterating through the list of event listeners, removing each one.
        (Doing that causes a crash when running the layout tests with GuardMalloc turned on)

        * html/HTMLKeygenElement.cpp:
        (WebCore::HTMLKeygenElement::HTMLKeygenElement):
        (WebCore::HTMLKeygenElement::appendFormData):
        Update due to changes in the SSL key generator.

        * ksvg2/svg/SVGAngle.cpp:
        Remove unused header.

        * ksvg2/svg/SVGAnimationElement.cpp:
        (SVGAnimationElement::parseMappedAttribute):
        Rename methods and pass Strings instead of DeprecatedStrings to
        parseSeparatedList.

        * ksvg2/svg/SVGFEColorMatrixElement.cpp:
        * ksvg2/svg/SVGFEDiffuseLightingElement.cpp:
        * ksvg2/svg/SVGFEDisplacementMapElement.cpp:
        * ksvg2/svg/SVGFEMergeElement.cpp:
        * ksvg2/svg/SVGFEOffsetElement.cpp:
        Remove unused headers.

        * ksvg2/svg/SVGHelper.cpp:
        (WebCore::SVGHelper::parseSeparatedList):
        * ksvg2/svg/SVGHelper.h:
        Rename ParseSeperatedList to parseSeparatedList, make it take a String instead of a
        DeprecatedString and have it use Vector<String> instead of DeprecatedStringList.

        * ksvg2/svg/SVGLengthList.cpp:
        (WebCore::SVGLengthList::parse):
        * ksvg2/svg/SVGLengthList.h:
        Use Vector<String> instead of DeprecatedStringList.

        * loader/Cache.cpp:
        (WebCore::Cache::requestImage):
        (WebCore::Cache::requestStyleSheet):
        (WebCore::Cache::requestScript):
        (WebCore::Cache::requestXSLStyleSheet):
        (WebCore::Cache::requestXBLDocument):
        (WebCore::Cache::remove):
        (WebCore::Cache::get):
        Use a String as the key in the cache set instead of a RefPtr<StringImpl>.

        * loader/DocLoader.cpp:
        (WebCore::DocLoader::needReload):
        * loader/DocLoader.h:
        Use a HashSet instead of DeprecatedStringList.

        * page/FrameView.h:
        Remove declarations that have no functions.

        * platform/SSLKeyGenerator.h:
        * platform/mac/SSLKeyGeneratorMac.mm:
        (WebCore::supportedKeySizes):
        (WebCore::signedPublicKeyAndChallengeString):
        Convert to use Vector<String> instead of DeprecatedStringList, and String instead of
        DeprecatedString.

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

21 files changed:
WebCore/ChangeLog
WebCore/css/MediaList.cpp
WebCore/dom/Document.cpp
WebCore/html/HTMLKeygenElement.cpp
WebCore/ksvg2/svg/SVGAngle.cpp
WebCore/ksvg2/svg/SVGAnimationElement.cpp
WebCore/ksvg2/svg/SVGFEColorMatrixElement.cpp
WebCore/ksvg2/svg/SVGFEDiffuseLightingElement.cpp
WebCore/ksvg2/svg/SVGFEDisplacementMapElement.cpp
WebCore/ksvg2/svg/SVGFEMergeElement.cpp
WebCore/ksvg2/svg/SVGFEOffsetElement.cpp
WebCore/ksvg2/svg/SVGHelper.cpp
WebCore/ksvg2/svg/SVGHelper.h
WebCore/ksvg2/svg/SVGLengthList.cpp
WebCore/ksvg2/svg/SVGLengthList.h
WebCore/loader/Cache.cpp
WebCore/loader/DocLoader.cpp
WebCore/loader/DocLoader.h
WebCore/page/FrameView.h
WebCore/platform/SSLKeyGenerator.h
WebCore/platform/mac/SSLKeyGeneratorMac.mm

index fb77b93..4528cbd 100644 (file)
@@ -1,4 +1,72 @@
-2006-09-25  Dave Hyatt  <hyat@apple.com>
+2006-09-25  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Maciej.
+
+        * css/MediaList.cpp:
+        Remove unused header.
+        
+        * dom/Document.cpp:
+        (WebCore::Document::clear):
+        Use clear instead of iterating through the list of event listeners, removing each one.
+        (Doing that causes a crash when running the layout tests with GuardMalloc turned on)
+        
+        * html/HTMLKeygenElement.cpp:
+        (WebCore::HTMLKeygenElement::HTMLKeygenElement):
+        (WebCore::HTMLKeygenElement::appendFormData):
+        Update due to changes in the SSL key generator.
+        
+        * ksvg2/svg/SVGAngle.cpp:
+        Remove unused header.
+        
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (SVGAnimationElement::parseMappedAttribute):
+        Rename methods and pass Strings instead of DeprecatedStrings to
+        parseSeparatedList.
+        
+        * ksvg2/svg/SVGFEColorMatrixElement.cpp:
+        * ksvg2/svg/SVGFEDiffuseLightingElement.cpp:
+        * ksvg2/svg/SVGFEDisplacementMapElement.cpp:
+        * ksvg2/svg/SVGFEMergeElement.cpp:
+        * ksvg2/svg/SVGFEOffsetElement.cpp:
+        Remove unused headers.
+        
+        * ksvg2/svg/SVGHelper.cpp:
+        (WebCore::SVGHelper::parseSeparatedList):
+        * ksvg2/svg/SVGHelper.h:
+        Rename ParseSeperatedList to parseSeparatedList, make it take a String instead of a 
+        DeprecatedString and have it use Vector<String> instead of DeprecatedStringList.
+        
+        * ksvg2/svg/SVGLengthList.cpp:
+        (WebCore::SVGLengthList::parse):
+        * ksvg2/svg/SVGLengthList.h:
+        Use Vector<String> instead of DeprecatedStringList.
+        
+        * loader/Cache.cpp:
+        (WebCore::Cache::requestImage):
+        (WebCore::Cache::requestStyleSheet):
+        (WebCore::Cache::requestScript):
+        (WebCore::Cache::requestXSLStyleSheet):
+        (WebCore::Cache::requestXBLDocument):
+        (WebCore::Cache::remove):
+        (WebCore::Cache::get):
+        Use a String as the key in the cache set instead of a RefPtr<StringImpl>.
+        
+        * loader/DocLoader.cpp:
+        (WebCore::DocLoader::needReload):
+        * loader/DocLoader.h:
+        Use a HashSet instead of DeprecatedStringList.
+        
+        * page/FrameView.h:
+        Remove declarations that have no functions.
+        
+        * platform/SSLKeyGenerator.h:
+        * platform/mac/SSLKeyGeneratorMac.mm:
+        (WebCore::supportedKeySizes):
+        (WebCore::signedPublicKeyAndChallengeString):
+        Convert to use Vector<String> instead of DeprecatedStringList, and String instead of
+        DeprecatedString.
+
+2006-09-25  Dave Hyatt  <hyatt@apple.com>
 
         Shunt the RenderTheme world transform adoption between
         Cairo and Windows into getWindowsContext and move the method
index ec7aed0..357acdb 100644 (file)
@@ -24,7 +24,6 @@
 
 #include "CSSRule.h"
 #include "CSSStyleSheet.h"
-#include "DeprecatedStringList.h"
 #include "ExceptionCode.h"
 #include "MediaQuery.h"
 #include "cssparser.h"
index d2f61c0..b3ded69 100644 (file)
@@ -1364,9 +1364,7 @@ void Document::clear()
 
     removeChildren();
 
-    RegisteredEventListenerList::iterator end = m_windowEventListeners.end();
-    for (RegisteredEventListenerList::iterator it = m_windowEventListeners.begin(); it != end; ++it)
-        m_windowEventListeners.remove(it);
+    m_windowEventListeners.clear();
 }
 
 void Document::setURL(const DeprecatedString& url)
index c527728..32cda85 100644 (file)
@@ -44,11 +44,13 @@ using namespace HTMLNames;
 HTMLKeygenElement::HTMLKeygenElement(Document* doc, HTMLFormElement* f)
     : HTMLSelectElement(keygenTag, doc, f)
 {
-    DeprecatedStringList keys = supportedKeySizes();
-    for (DeprecatedStringList::Iterator i = keys.begin(); i != keys.end(); ++i) {
+    Vector<String> keys = supportedKeySizes();
+        
+    Vector<String>::const_iterator end = keys.end();
+    for (Vector<String>::const_iterator it = keys.begin(); it != end; ++it) {
         HTMLOptionElement* o = new HTMLOptionElement(doc, form());
         addChild(o);
-        o->addChild(new Text(doc, String(*i)));
+        o->addChild(new Text(doc, *it));
     }
 }
 
@@ -74,7 +76,7 @@ bool HTMLKeygenElement::appendFormData(FormDataList& encoded_values, bool)
     // Only RSA is supported at this time.
     if (!m_keyType.isNull() && !equalIgnoringCase(m_keyType, "rsa"))
         return false;
-    DeprecatedString value = signedPublicKeyAndChallengeString(selectedIndex(), m_challenge.deprecatedString(), document()->baseURL());
+    String value = signedPublicKeyAndChallengeString(selectedIndex(), m_challenge, document()->baseURL());
     if (value.isNull())
         return false;
     encoded_values.appendData(name(), value.utf8());
index 97fc7f9..d92fe04 100644 (file)
@@ -26,6 +26,7 @@
 
 #include <ksvg2/ksvg.h>
 
+#include "DeprecatedString.h"
 #include "SVGAngle.h"
 #include "SVGHelper.h"
 
index b37aaeb..8829da5 100644 (file)
@@ -138,7 +138,7 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
         RefPtr<SVGStringList> temp = new SVGStringList();
 
         // Feed data into list
-        SVGHelper::ParseSeperatedList(temp.get(), value.deprecatedString(), ';');
+        SVGHelper::parseSeparatedList(temp.get(), value, ';');
 
         ExceptionCode ec = 0;
 
@@ -252,17 +252,17 @@ void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
     else if (attr->name() == SVGNames::valuesAttr)
     {
         m_values = new SVGStringList();
-        SVGHelper::ParseSeperatedList(m_values.get(), value.deprecatedString(), ';');
+        SVGHelper::parseSeparatedList(m_values.get(), value, ';');
     }
     else if (attr->name() == SVGNames::keyTimesAttr)
     {
         m_keyTimes = new SVGStringList();
-        SVGHelper::ParseSeperatedList(m_keyTimes.get(), value.deprecatedString(), ';');
+        SVGHelper::parseSeparatedList(m_keyTimes.get(), value, ';');
     }
     else if (attr->name() == SVGNames::keySplinesAttr)
     {
         m_keySplines = new SVGStringList();
-        SVGHelper::ParseSeperatedList(m_keySplines.get(), value.deprecatedString(), ';');
+        SVGHelper::parseSeparatedList(m_keySplines.get(), value, ';');
     }
     else if (attr->name() == SVGNames::fromAttr)
         m_from = value.deprecatedString();
index 8a2b03e..fb94a44 100644 (file)
@@ -22,8 +22,6 @@
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "DeprecatedStringList.h"
-
 #include "Attr.h"
 
 #include "KCanvasFilters.h"
index 4e60ee4..7902585 100644 (file)
@@ -19,8 +19,6 @@
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "DeprecatedStringList.h"
-
 #include "Attr.h"
 
 #include <kcanvas/KCanvasFilters.h>
index 07bcc80..b9b30c3 100644 (file)
@@ -20,8 +20,6 @@
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "DeprecatedStringList.h"
-
 #include "Attr.h"
 
 #include <kcanvas/KCanvasFilters.h>
index 9a360e9..f487f3a 100644 (file)
@@ -22,8 +22,6 @@
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "DeprecatedStringList.h"
-
 #include "Attr.h"
 
 #include <kcanvas/KCanvasFilters.h>
index bc3d8bc..8c107c3 100644 (file)
@@ -22,8 +22,6 @@
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "DeprecatedStringList.h"
-
 #include "Attr.h"
 
 #include "KCanvasFilters.h"
index fddb065..e5493ce 100644 (file)
@@ -77,18 +77,17 @@ float SVGHelper::PercentageOfViewport(float value, const SVGElement* viewportEle
     return 0.0;
 }
 
-void SVGHelper::ParseSeperatedList(SVGStringList *list, const DeprecatedString &data, const DeprecatedChar &delimiter)
+void SVGHelper::parseSeparatedList(SVGStringList *list, const String& data, UChar delimiter)
 {
     // TODO : more error checking/reporting
     ExceptionCode ec = 0;
     list->clear(ec);
 
-    DeprecatedStringList substrings = DeprecatedStringList::split(delimiter, data);
+    Vector<String> substrings = String(data).split(delimiter);
     
-    DeprecatedStringList::ConstIterator it = substrings.begin();
-    DeprecatedStringList::ConstIterator end = substrings.end();
-    for (; it != end; ++it)
-        list->appendItem(String(*it), ec);
+    Vector<String>::const_iterator end = substrings.end();
+    for (Vector<String>::const_iterator it = substrings.begin(); it != end; ++it)
+        list->appendItem(*it, ec);
 }
 
 }
index cf8b07b..80c0b77 100644 (file)
@@ -24,7 +24,7 @@
 #define KSVG_SVGHelper_H
 #ifdef SVG_SUPPORT
 
-#include "DeprecatedString.h"
+#include "PlatformString.h"
 
 namespace WebCore {
 
@@ -43,7 +43,7 @@ namespace WebCore {
     public:
         static float PercentageOfViewport(float value, const SVGElement *viewportElement, LengthMode mode);
 
-        static void ParseSeperatedList(SVGStringList *list, const DeprecatedString &data, const DeprecatedChar &delimiter = ',');
+        static void parseSeparatedList(SVGStringList *list, const String &data, UChar delimiter = ',');
     };
 
     // Lazy creation, template-based
index 5903645..2f64a2d 100644 (file)
@@ -41,15 +41,15 @@ SVGLengthList::~SVGLengthList()
 {
 }
 
-void SVGLengthList::parse(const DeprecatedString& value, const SVGStyledElement* context, LengthMode mode)
+void SVGLengthList::parse(const String& value, const SVGStyledElement* context, LengthMode mode)
 {
     ExceptionCode ec = 0;
 
-    DeprecatedStringList lengths = DeprecatedStringList::split(' ', value);
-    for (unsigned int i = 0; i < lengths.count(); i++) {
+    Vector<String> lengths = value.split(' ');
+    Vector<String>::const_iterator end = lengths.end();
+    for (Vector<String>::const_iterator it = lengths.begin(); it != end; ++it) {
         SVGLength* length = new SVGLength(context, mode);
-        String str(lengths[i]);
-        length->setValueAsString(str);
+        length->setValueAsString(*it);
         appendItem(length, ec);
     }
 }
index 9a746b8..856be06 100644 (file)
@@ -36,7 +36,7 @@ namespace WebCore
         SVGLengthList();
         virtual ~SVGLengthList();
 
-        void parse(const DeprecatedString& value, const SVGStyledElement* context, LengthMode mode);
+        void parse(const String& value, const SVGStyledElement* context, LengthMode mode);
     };
 
 } // namespace WebCore
index fd881e7..0264045 100644 (file)
@@ -57,7 +57,7 @@ struct LRUList {
 
 static bool cacheDisabled;
 
-typedef HashMap<RefPtr<StringImpl>, CachedResource*> CacheMap;
+typedef HashMap<String, CachedResource*> CacheMap;
 
 static CacheMap* cache = 0;
 
@@ -147,7 +147,7 @@ CachedImage* Cache::requestImage(DocLoader* dl, const KURL& url, bool reload, ti
 
     CachedResource *o = 0;
     if (!reload)
-        o = cache->get(String(url.url()).impl());
+        o = cache->get(url.url());
     if (!o) {
 #ifdef CACHE_DEBUG
         kdDebug(6060) << "Cache: new: " << url.url() << endl;
@@ -157,7 +157,7 @@ CachedImage* Cache::requestImage(DocLoader* dl, const KURL& url, bool reload, ti
         if (cacheDisabled)
             im->setFree(true);
         else {
-            cache->set(String(url.url()).impl(), im);
+            cache->set(url.url(), im);
             moveToHeadOfLRUList(im);
         }
         o = im;
@@ -193,7 +193,7 @@ CachedCSSStyleSheet* Cache::requestStyleSheet(DocLoader* dl, const String& url,
 
     // Checking if the URL is malformed is lots of extra work for little benefit.
 
-    CachedResource *o = cache->get(String(kurl.url()).impl());
+    CachedResource *o = cache->get(kurl.url());
     if (!o) {
 #ifdef CACHE_DEBUG
         kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
@@ -202,7 +202,7 @@ CachedCSSStyleSheet* Cache::requestStyleSheet(DocLoader* dl, const String& url,
         if (cacheDisabled)
             sheet->setFree(true);
         else {
-            cache->set(String(kurl.url()).impl(), sheet);
+            cache->set(kurl.url(), sheet);
             moveToHeadOfLRUList(sheet);
         }
         o = sheet;
@@ -243,7 +243,7 @@ CachedScript* Cache::requestScript(DocLoader* dl, const String& url, bool reload
 
     // Checking if the URL is malformed is lots of extra work for little benefit.
 
-    CachedResource *o = cache->get(String(kurl.url()).impl());
+    CachedResource *o = cache->get(kurl.url());
     if (!o)
     {
 #ifdef CACHE_DEBUG
@@ -253,7 +253,7 @@ CachedScript* Cache::requestScript(DocLoader* dl, const String& url, bool reload
         if (cacheDisabled)
             script->setFree(true);
         else {
-            cache->set(String(kurl.url()).impl(), script);
+            cache->set(kurl.url(), script);
             moveToHeadOfLRUList(script);
         }
         o = script;
@@ -296,7 +296,7 @@ CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String& ur
     
     // Checking if the URL is malformed is lots of extra work for little benefit.
     
-    CachedResource *o = cache->get(String(kurl.url()).impl());
+    CachedResource *o = cache->get(kurl.url());
     if (!o) {
 #ifdef CACHE_DEBUG
         kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
@@ -305,7 +305,7 @@ CachedXSLStyleSheet* Cache::requestXSLStyleSheet(DocLoader* dl, const String& ur
         if (cacheDisabled)
             doc->setFree(true);
         else {
-            cache->set(String(kurl.url()).impl(), doc);
+            cache->set(kurl.url(), doc);
             moveToHeadOfLRUList(doc);
         }
         o = doc;
@@ -348,7 +348,7 @@ CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String& url, b
     
     // Checking if the URL is malformed is lots of extra work for little benefit.
     
-    CachedResource *o = cache->get(String(kurl.url()).impl());
+    CachedResource *o = cache->get(kurl.url());
     if (!o) {
 #ifdef CACHE_DEBUG
         kdDebug(6060) << "Cache: new: " << kurl.url() << endl;
@@ -357,7 +357,7 @@ CachedXBLDocument* Cache::requestXBLDocument(DocLoader* dl, const String& url, b
         if (cacheDisabled)
             doc->setFree(true);
         else {
-            cache->set(String(kurl.url()).impl(), doc);
+            cache->set(kurl.url(), doc);
             moveToHeadOfLRUList(doc);
         }
         o = doc;
@@ -424,7 +424,7 @@ void Cache::remove(CachedResource *object)
   // drops down to zero
   object->setFree(true);
 
-  cache->remove(object->url().impl());
+  cache->remove(object->url());
   removeFromLRUList(object);
 
   HashSet<DocLoader*>::iterator end = docloaders->end();
@@ -616,7 +616,7 @@ void Cache::setCacheDisabled(bool disabled)
 
 CachedResource* Cache::get(const String& s)
 {
-    return (cache && s.impl()) ? cache->get(s.impl()) : 0;
+    return (cache && s.impl()) ? cache->get(s) : 0;
 }
 
 }
index 58259b8..553391a 100644 (file)
@@ -71,7 +71,7 @@ bool DocLoader::needReload(const KURL& fullURL)
           CachedResource* existing = Cache::get(fullURL.url());
           if (existing && existing->isExpired()) {
              Cache::remove(existing);
-             m_reloadedURLs.append(fullURL.url());
+             m_reloadedURLs.add(fullURL.url());
              reload = true;
           }
        }
@@ -80,7 +80,7 @@ bool DocLoader::needReload(const KURL& fullURL)
           CachedResource* existing = Cache::get(fullURL.url());
           if (existing)
              Cache::remove(existing);
-          m_reloadedURLs.append(fullURL.url());
+          m_reloadedURLs.add(fullURL.url());
           reload = true;
        }
     }
index 4696a40..38c692d 100644 (file)
 #define DocLoader_h
 
 #include "CachePolicy.h"
-#include "DeprecatedStringList.h"
 #include "Settings.h"
 #include "StringHash.h"
 #include <wtf/HashMap.h>
+#include <wtf/HashSet.h>
 
 namespace WebCore {
     class CachedCSSStyleSheet;
@@ -88,7 +88,7 @@ namespace WebCore {
         friend class Document;
         friend class HTMLImageLoader;
 
-        DeprecatedStringList m_reloadedURLs;
+        HashSet<String> m_reloadedURLs;
         mutable HashMap<String, CachedResource*> m_docObjects;
         time_t m_expireDate;
         CachePolicy m_cachePolicy;
index 43b84b9..29a4178 100644 (file)
@@ -224,9 +224,6 @@ private:
 
     void restoreScrollBar();
 
-    DeprecatedStringList formCompletionItems(const String& name) const;
-    void addFormCompletionItem(const String& name, const String& value);
-
     MouseEventWithHitTestResults prepareMouseEvent(bool readonly, bool active, bool mouseMove, const PlatformMouseEvent&);
 
     bool dispatchMouseEvent(const AtomicString& eventType, Node* target,
index 145d4ef..8472d68 100644 (file)
 #ifndef SSLKeyGenerator_h
 #define SSLKeyGenerator_h
 
-#include "DeprecatedStringList.h"
+#include <wtf/Vector.h>
+#include "PlatformString.h"
 
 namespace WebCore {
 
     class KURL;
 
-    DeprecatedStringList supportedKeySizes();
-    DeprecatedString signedPublicKeyAndChallengeString(unsigned keySizeIndex, const DeprecatedString& challengeString, const KURL&);
+    Vector<String> supportedKeySizes();
+    String signedPublicKeyAndChallengeString(unsigned keySizeIndex, const String& challengeString, const KURL&);
 
 }
 
index 5115780..5c4161a 100644 (file)
 
 namespace WebCore {
 
-DeprecatedStringList supportedKeySizes()
+Vector<String> supportedKeySizes()
 { 
     NSEnumerator *enumerator = [[[WebCoreKeyGenerator sharedGenerator] strengthMenuItemTitles] objectEnumerator];
-    DeprecatedStringList supportedKeySizes = DeprecatedStringList(); 
+    Vector<String> supportedKeySizes;
     NSString *string;
     while ((string = [enumerator nextObject]) != nil) {
-        supportedKeySizes.append(DeprecatedString::fromNSString(string));
+        supportedKeySizes.append(string);
     }
     return supportedKeySizes;
 }
 
-DeprecatedString signedPublicKeyAndChallengeString(unsigned keySizeIndex, const DeprecatedString &challengeString, const KURL &url)
+String signedPublicKeyAndChallengeString(unsigned keySizeIndex, const String& challengeString, const KURL& url)
 {   
-    return DeprecatedString::fromNSString([[WebCoreKeyGenerator sharedGenerator] signedPublicKeyAndChallengeStringWithStrengthIndex:keySizeIndex 
-                                                                                                                 challenge:challengeString.getNSString()
-                                                                                                                   pageURL:url.getNSURL()]);
+    return [[WebCoreKeyGenerator sharedGenerator] signedPublicKeyAndChallengeStringWithStrengthIndex:keySizeIndex 
+                                                                                           challenge:challengeString
+                                                                                             pageURL:url.getNSURL()];
 }
 
 }