aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAaron McCarthy <aaron.mccarthy@jollamobile.com>2015-03-10 15:40:44 +1000
committerAaron McCarthy <mccarthy.aaron@gmail.com>2015-04-02 02:06:13 +0000
commitbcba471151c00c726aa6825b80a25816425168de (patch)
treea47b6a57e498dc3ae9db1a94d9706e800ad52310
parentcc8b455d00dfa30a6c4ddc98ffd8a275c36159b9 (diff)
Add note clarifying return value of Qt.openUrlExternally()
On some platforms (notably wayland and xcb) the Qt.openUrlExternally() call returns as soon as the request is passed to the underlying system. The call may still fail, but this status is not reported to the application. Change-Id: I66936440f162154f08ab5abcf5b449844930a81b Reviewed-by: Valerio Valerio <valerio.valerio@jollamobile.com> Reviewed-by: Topi Reiniƶ <topi.reinio@digia.com>
-rw-r--r--src/qml/qml/v8/qqmlbuiltinfunctions.cpp11
1 files changed, 9 insertions, 2 deletions
diff --git a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
index f222d59494..d4e0d53cf5 100644
--- a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
+++ b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
@@ -803,8 +803,15 @@ ReturnedValue QtObject::method_formatDateTime(QV4::CallContext *ctx)
}
/*!
-\qmlmethod bool Qt::openUrlExternally(url target)
-Attempts to open the specified \c target url in an external application, based on the user's desktop preferences. Returns true if it succeeds, and false otherwise.
+ \qmlmethod bool Qt::openUrlExternally(url target)
+
+ Attempts to open the specified \c target url in an external application, based on the user's
+ desktop preferences. Returns true if it succeeds, and false otherwise.
+
+ \warning A return value of \c true indicates that the application has successfully requested
+ the operating system to open the URL in an external application. The external application may
+ still fail to launch or fail to open the requested URL. This result will not be reported back
+ to the application.
*/
ReturnedValue QtObject::method_openUrlExternally(QV4::CallContext *ctx)
{