Bug 121984 - [Qt] Fix build after r156487
Summary: [Qt] Fix build after r156487
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-09-26 14:35 PDT by Julien Brianceau
Modified: 2013-09-26 15:17 PDT (History)
4 users (show)

See Also:


Attachments
Use Weak instead of PassWeak in WebCore/bridge/qt/qt_runtime.cpp (1.17 KB, patch)
2013-09-26 14:44 PDT, Julien Brianceau
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Brianceau 2013-09-26 14:35:48 PDT
Fix Qt build after r156487 (http://trac.webkit.org/changeset/156487)

Seen with our sh4 build bot (http://build.webkit.org/builders/Qt%20Linux%20SH4%20Release/builds/30703):

/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/Source/WebCore/bridge/qt/qt_runtime.cpp: In member function 'OpaqueJSValue* JSC::Bindings::QtRuntimeMethod::jsObjectRef(JSContextRef, const OpaqueJSValue**)':
/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/Source/WebCore/bridge/qt/qt_runtime.cpp:1304:18: error: 'PassWeak' was not declared in this scope
/local/wkit/slavebuildbot_LOCAL/workspace_LOCAL/qt-linux-sh4-release/build/Source/WebCore/bridge/qt/qt_runtime.cpp:1304:35: error: expected primary-expression before '>' token
make[2]: *** [.obj/release-shared/bridge/qt/qt_runtime.o] Error 1
make[2]: *** Waiting for unfinished jobs....
Comment 1 Julien Brianceau 2013-09-26 14:44:15 PDT
Created attachment 212751 [details]
Use Weak instead of PassWeak in WebCore/bridge/qt/qt_runtime.cpp
Comment 2 WebKit Commit Bot 2013-09-26 15:17:55 PDT
Comment on attachment 212751 [details]
Use Weak instead of PassWeak in WebCore/bridge/qt/qt_runtime.cpp

Clearing flags on attachment: 212751

Committed r156503: <http://trac.webkit.org/changeset/156503>
Comment 3 WebKit Commit Bot 2013-09-26 15:17:57 PDT
All reviewed patches have been landed.  Closing bug.