2011-06-08 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
authorcaio.oliveira@openbossa.org <caio.oliveira@openbossa.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jun 2011 18:12:15 +0000 (18:12 +0000)
committercaio.oliveira@openbossa.org <caio.oliveira@openbossa.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jun 2011 18:12:15 +0000 (18:12 +0000)
        Reviewed by Andreas Kling.

        [Qt] Remove dead code from our JSC bridge
        https://bugs.webkit.org/show_bug.cgi?id=62287

        The m_defaultMethod member is not used since r36848 and QtMethod
        was replaced by QtRuntimeMethod and subclasses in r29729.

        * bridge/qt/qt_instance.cpp:
        (JSC::Bindings::QtInstance::removeCachedMethod):
        (JSC::Bindings::QtInstance::visitAggregate):
        * bridge/qt/qt_instance.h:
        * bridge/qt/qt_runtime.h:

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

Source/WebCore/ChangeLog
Source/WebCore/bridge/qt/qt_instance.cpp
Source/WebCore/bridge/qt/qt_instance.h
Source/WebCore/bridge/qt/qt_runtime.h

index d6b8453..4f79375 100644 (file)
@@ -1,3 +1,19 @@
+2011-06-08  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
+
+        Reviewed by Andreas Kling.
+
+        [Qt] Remove dead code from our JSC bridge
+        https://bugs.webkit.org/show_bug.cgi?id=62287
+
+        The m_defaultMethod member is not used since r36848 and QtMethod
+        was replaced by QtRuntimeMethod and subclasses in r29729.
+
+        * bridge/qt/qt_instance.cpp:
+        (JSC::Bindings::QtInstance::removeCachedMethod):
+        (JSC::Bindings::QtInstance::visitAggregate):
+        * bridge/qt/qt_instance.h:
+        * bridge/qt/qt_runtime.h:
+
 2011-06-08  Ryan Sleevi  <rsleevi@chromium.org>
 
         Reviewed by Tony Chang.
index df39bbc..df278c9 100644 (file)
@@ -151,9 +151,6 @@ void QtInstance::put(JSObject* object, ExecState* exec, const Identifier& proper
 
 void QtInstance::removeCachedMethod(JSObject* method)
 {
-    if (m_defaultMethod.get() == method)
-        m_defaultMethod.clear();
-
     for (QHash<QByteArray, WriteBarrier<JSObject> >::Iterator it = m_methods.begin(), end = m_methods.end(); it != end; ++it) {
         if (it.value().get() == method) {
             m_methods.erase(it);
@@ -190,8 +187,6 @@ RuntimeObject* QtInstance::newRuntimeObject(ExecState* exec)
 
 void QtInstance::visitAggregate(SlotVisitor& visitor)
 {
-    if (m_defaultMethod)
-        visitor.append(&m_defaultMethod);
     for (QHash<QByteArray, WriteBarrier<JSObject> >::Iterator it = m_methods.begin(), end = m_methods.end(); it != end; ++it)
         visitor.append(&it.value());
 }
index 6935022..3ee0caf 100644 (file)
@@ -98,7 +98,6 @@ private:
     QObject* m_hashkey;
     mutable QHash<QByteArray, WriteBarrier<JSObject> > m_methods;
     mutable QHash<QString, QtField*> m_fields;
-    mutable WriteBarrier<QtRuntimeMetaMethod> m_defaultMethod;
     QScriptEngine::ValueOwnership m_ownership;
 };
 
index 4138d88..4d28873 100644 (file)
@@ -73,28 +73,6 @@ private:
 };
 
 
-class QtMethod : public Method
-{
-public:
-    QtMethod(const QMetaObject *mo, int i, const QByteArray &ident, int numParameters)
-        : m_metaObject(mo),
-          m_index(i),
-          m_identifier(ident),
-          m_nParams(numParameters)
-        { }
-
-    virtual const char* name() const { return m_identifier.constData(); }
-    virtual int numParameters() const { return m_nParams; }
-
-private:
-    friend class QtInstance;
-    const QMetaObject *m_metaObject;
-    int m_index;
-    QByteArray m_identifier;
-    int m_nParams;
-};
-
-
 template <typename T> class QtArray : public Array
 {
 public: