[Merge] lp:~abreu-alexandre/webbrowser-app/host-support-devtools into lp:webbrowser-app
Alexandre Abreu
alexandre.abreu at canonical.com
Mon Oct 6 14:12:39 UTC 2014
updated
Diff comments:
> === modified file 'src/Ubuntu/Web/UbuntuWebContext.qml'
> --- src/Ubuntu/Web/UbuntuWebContext.qml 2014-08-11 15:13:50 +0000
> +++ src/Ubuntu/Web/UbuntuWebContext.qml 2014-10-06 13:57:05 +0000
> @@ -82,4 +82,5 @@
>
> devtoolsEnabled: webviewDevtoolsDebugPort !== -1
> devtoolsPort: webviewDevtoolsDebugPort
> + devtoolsIp: webviewDevtoolsDebugHost
> }
>
> === modified file 'src/Ubuntu/Web/plugin.cpp'
> --- src/Ubuntu/Web/plugin.cpp 2014-09-26 13:30:46 +0000
> +++ src/Ubuntu/Web/plugin.cpp 2014-10-06 13:57:05 +0000
> @@ -83,6 +83,18 @@
> return port > 0 ? port : DEVTOOLS_INVALID_PORT;
> }
>
> +
> +static QString getDevtoolsHost()
> +{
> + QString host;
> + const char* DEVTOOLS_HOST_ENV_VAR = "UBUNTU_WEBVIEW_DEVTOOLS_HOST";
> +
> + if (qEnvironmentVariableIsSet(DEVTOOLS_HOST_ENV_VAR)) {
> + host = qgetenv(DEVTOOLS_HOST_ENV_VAR);
> + }
> + return host;
> +}
> +
> void UbuntuBrowserPlugin::initializeEngine(QQmlEngine* engine, const char* uri)
> {
> Q_UNUSED(uri);
> @@ -104,6 +116,7 @@
> context->setContextProperty("webviewDevtoolsDebugPort", getDevtoolsPort());
>
> engine->addImageProvider("favicon", new FaviconImageProvider());
> + context->setContextProperty("webviewDevtoolsDebugHost", getDevtoolsHost());
you are right, updated
> }
>
> void UbuntuBrowserPlugin::registerTypes(const char* uri)
>
> === modified file 'src/app/browserapplication.cpp'
> --- src/app/browserapplication.cpp 2014-09-15 17:08:41 +0000
> +++ src/app/browserapplication.cpp 2014-10-06 13:57:05 +0000
> @@ -77,6 +77,18 @@
> return port;
> }
>
> +QString BrowserApplication::inspectorHost() const
> +{
> + QString host;
> + Q_FOREACH(QHostAddress address, QNetworkInterface::allAddresses()) {
> + if (!address.isLoopback() && (address.protocol() == QAbstractSocket::IPv4Protocol)) {
> + host = address.toString();
> + break;
> + }
> + }
> + return host;
> +}
> +
> QString BrowserApplication::appId() const
> {
> Q_FOREACH(const QString& argument, m_arguments) {
> @@ -123,8 +135,10 @@
> #endif
>
> QString devtoolsPort = inspectorPort();
> + QString devtoolsHost = inspectorHost();
> bool inspectorEnabled = !devtoolsPort.isEmpty();
> if (inspectorEnabled) {
> + qputenv("UBUNTU_WEBVIEW_DEVTOOLS_HOST", devtoolsHost.toUtf8());
> qputenv("UBUNTU_WEBVIEW_DEVTOOLS_PORT", devtoolsPort.toUtf8());
> }
>
>
> === modified file 'src/app/browserapplication.h'
> --- src/app/browserapplication.h 2014-07-03 19:49:54 +0000
> +++ src/app/browserapplication.h 2014-10-06 13:57:05 +0000
> @@ -59,6 +59,7 @@
> private:
> QString appId() const;
> QString inspectorPort() const;
> + QString inspectorHost() const;
>
> WebBrowserWindow *m_webbrowserWindowProxy;
> };
>
--
https://code.launchpad.net/~abreu-alexandre/webbrowser-app/host-support-devtools/+merge/236965
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list