take a QString as identifier in QWebFrame::addToJSWindowObject.
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Jan 2008 12:25:17 +0000 (12:25 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Jan 2008 12:25:17 +0000 (12:25 +0000)
Signed-off-by: Simon
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29159 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/qt/Api/qwebframe.cpp
WebKit/qt/Api/qwebframe.h
WebKit/qt/ChangeLog
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/qt/DumpRenderTree.cpp

index c70ac2f66e1a16806795d62ff7d6cd6e28324ac0..e12ab005b1a6f574d4dc41dfb985b3ef03c48c87 100644 (file)
@@ -139,7 +139,7 @@ QWebFrame::~QWebFrame()
     delete d;
 }
 
-void QWebFrame::addToJSWindowObject(const QByteArray &name, QObject *object)
+void QWebFrame::addToJSWindowObject(const QString &name, QObject *object)
 {
       KJS::JSLock lock;
       KJS::Window *window = KJS::Window::retrieveWindow(d->frame.get());
@@ -153,7 +153,7 @@ void QWebFrame::addToJSWindowObject(const QByteArray &name, QObject *object)
         KJS::Bindings::Instance::createRuntimeObject(KJS::Bindings::Instance::QtLanguage,
                                                      object, root);
 
-      window->put(window->globalExec(), KJS::Identifier(name.constData()), runtimeObject);
+      window->put(window->globalExec(), KJS::Identifier((const KJS::UChar *) name.constData(), name.length()), runtimeObject);
 }
 
 
index 4cf11f9ed80a7d0d26eba5f95588403135c669eb..31ce84444ee529112fa067e1170d757cfc6a9d27 100644 (file)
@@ -64,7 +64,7 @@ public:
     void setHtml(const QByteArray &html, const QUrl &baseUrl = QUrl());
     void setContent(const QByteArray &data, const QString &mimeType = QString(), const QUrl &baseUrl = QUrl());
 
-    void addToJSWindowObject(const QByteArray &name, QObject *object);
+    void addToJSWindowObject(const QString &name, QObject *object);
     QString markup() const;
     QString innerText() const;
     QString renderTreeDump() const;
index cf869a5b73070d3294416085c9dbaf94df5ac1d5..1ff8e43bd7c77f3984db191be0be7c6049435952 100644 (file)
@@ -1,3 +1,13 @@
+2008-01-04  Lars Knoll  <lars@trolltech.com>
+
+        Reviewed by Simon.
+
+        take a QString as identifier in QWebFrame::addToJSWindowObject.
+
+        * Api/qwebframe.cpp:
+        (QWebFrame::addToJSWindowObject):
+        * Api/qwebframe.h:
+
 2008-01-04  Simon Hausmann  <hausmann@webkit.org>
 
         Reviewed by Lars.
index 7d887a9200cb6b04e68153c25a2b0d71792073c3..0bba4809f75e5adc3de99f5b42fb3e68c9299237 100644 (file)
@@ -1,3 +1,12 @@
+2008-01-04  Lars Knoll  <lars@trolltech.com>
+
+        Reviewed by Simon.
+
+        take a QString as identifier in QWebFrame::addToJSWindowObject.
+
+        * DumpRenderTree/qt/DumpRenderTree.cpp:
+        (WebCore::DumpRenderTree::initJSObjects):
+
 2008-01-04  Alp Toker  <alp@atoker.com>
 
         GTK+ DRT build fix for breakage introduced in r29149.
index 05be4bc1b6689fa7e859a9e69fb3cd1e8742f835..40716faa538475c55aada55de9b79d931ade4314 100644 (file)
@@ -182,9 +182,9 @@ void DumpRenderTree::initJSObjects()
 {
     QWebFrame *frame = qobject_cast<QWebFrame*>(sender());
     Q_ASSERT(frame);
-    frame->addToJSWindowObject("layoutTestController", m_controller);
-    frame->addToJSWindowObject("eventSender", m_eventSender);
-    frame->addToJSWindowObject("textInputController", m_textInputController);
+    frame->addToJSWindowObject(QLatin1String("layoutTestController"), m_controller);
+    frame->addToJSWindowObject(QLatin1String("eventSender"), m_eventSender);
+    frame->addToJSWindowObject(QLatin1String("textInputController"), m_textInputController);
 }