Bug 93462

Summary: [Qt] Add unit test for QObject bindings for scriptable plugins
Product: WebKit Reporter: Simon Hausmann <hausmann>
Component: New BugsAssignee: Simon Hausmann <hausmann>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 60842    
Attachments:
Description Flags
Patch kenneth: review+

Description Simon Hausmann 2012-08-08 04:44:57 PDT 
[Qt] Add unit test for QObject bindings for scriptable plugins
Comment 1 Simon Hausmann 2012-08-08 04:48:01 PDT 
Created attachment 157182 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2012-08-08 05:04:17 PDT 
Comment on attachment 157182 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=157182&action=review

> Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2189
> +    TestPluginWidget()
> +    { }

why on separate line?

> Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2192
> +    int slotWithReturnValue() { return 42; }

Lovely number :-)

> Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2206
> +    virtual QObject *createPlugin(const QString &, const QUrl &, const QStringList &, const QStringList &)

why not webkit style?
Comment 3 Simon Hausmann 2012-08-08 06:12:20 PDT 
(In reply to comment #2)
> (From update of attachment 157182 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=157182&action=review
> 
> > Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2189
> > +    TestPluginWidget()
> > +    { }
> 
> why on separate line?

OOps.

> > Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2192
> > +    int slotWithReturnValue() { return 42; }
> 
> Lovely number :-)
> 
> > Source/WebKit/qt/tests/qobjectbridge/tst_qobjectbridge.cpp:2206
> > +    virtual QObject *createPlugin(const QString &, const QUrl &, const QStringList &, const QStringList &)
> 
> why not webkit style?

Thanks, will fix these before landing.
Comment 4 Simon Hausmann 2012-08-08 06:14:37 PDT 
Committed r125029: <http://trac.webkit.org/changeset/125029>