Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Williams2019-02-27 18:42:45 +0000
committerEric Williams2019-03-11 12:51:19 +0000
commitb5fb9da32d06cd10c4d1a76aaac00439682396ef (patch)
tree4da361276ec7be7d5be798e2cbaf4b42598edd2f
parent38233cc8b4b9945e7affab239ee2d77242e77c5f (diff)
downloadeclipse.platform.swt-b5fb9da32d06cd10c4d1a76aaac00439682396ef.tar.gz
eclipse.platform.swt-b5fb9da32d06cd10c4d1a76aaac00439682396ef.tar.xz
eclipse.platform.swt-b5fb9da32d06cd10c4d1a76aaac00439682396ef.zip
Bug 543908: [GTK4] Get ControlExample to launch and run on GTK4
Disable Browser loading on GTK4. Change-Id: I9ebbb35dd3fe6402e4732ad7fb9e123b9fb3b4c6 Signed-off-by: Eric Williams <ericwill@redhat.com>
-rw-r--r--bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java11
-rw-r--r--bundles/org.eclipse.swt/Eclipse SWT Browser/gtk/org/eclipse/swt/browser/BrowserFactory.java1
-rw-r--r--bundles/org.eclipse.swt/Eclipse SWT WebKit/gtk/org/eclipse/swt/browser/WebKit.java1
3 files changed, 8 insertions, 5 deletions
diff --git a/bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java b/bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java
index dc20229953..d344a70817 100644
--- a/bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java
@@ -104,17 +104,18 @@ public Browser (Composite parent, int style) {
}
dispose ();
- String errMsg = " because no underlying browser available.\n";
+ String errMsg = " because there is no underlying browser available.\n";
switch (SWT.getPlatform()) {
case "gtk":
- errMsg = errMsg + " Please ensure Webkit with its Gtk 3.x bindings installed. Webkit2 API level preferred.\n";
+ errMsg = errMsg + "Please ensure that WebKit with its GTK 3.x bindings is installed (WebKit2 API level is preferred)."
+ + " Additionally, please note that GTK4 does not currently have Browser support.\n";
break;
case "cocoa":
- errMsg = errMsg + " SWT failed to load webkit library.\n";
+ errMsg = errMsg + "SWT failed to load the WebKit library.\n";
break;
case "win32":
- errMsg = errMsg + " SWT uses either IE or Webkit. Either SWT.WEBKIT flag is passed and Webkit library was not "
- + "loaded properly by SWT or SWT failed to load IE.\n";
+ errMsg = errMsg + "SWT uses either IE or WebKit. Either the SWT.WEBKIT flag is passed and the WebKit library was not "
+ + "loaded properly by SWT, or SWT failed to load IE.\n";
break;
default:
break;
diff --git a/bundles/org.eclipse.swt/Eclipse SWT Browser/gtk/org/eclipse/swt/browser/BrowserFactory.java b/bundles/org.eclipse.swt/Eclipse SWT Browser/gtk/org/eclipse/swt/browser/BrowserFactory.java
index c48367c1dd..f9a1891dbb 100644
--- a/bundles/org.eclipse.swt/Eclipse SWT Browser/gtk/org/eclipse/swt/browser/BrowserFactory.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT Browser/gtk/org/eclipse/swt/browser/BrowserFactory.java
@@ -19,6 +19,7 @@ class BrowserFactory {
WebBrowser createWebBrowser (int style) {
if (OS.IsWin32) return null;
+ if (GTK.GTK4) return null;
boolean webkitInstalled = WebKit.IsInstalled ();
if (!webkitInstalled) return null;
diff --git a/bundles/org.eclipse.swt/Eclipse SWT WebKit/gtk/org/eclipse/swt/browser/WebKit.java b/bundles/org.eclipse.swt/Eclipse SWT WebKit/gtk/org/eclipse/swt/browser/WebKit.java
index 5cb1b17313..be699439d2 100644
--- a/bundles/org.eclipse.swt/Eclipse SWT WebKit/gtk/org/eclipse/swt/browser/WebKit.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT WebKit/gtk/org/eclipse/swt/browser/WebKit.java
@@ -698,6 +698,7 @@ static Browser FindBrowser (long /*int*/ webView) {
}
static boolean IsInstalled () {
+ if (GTK.GTK4) return false;
if (!WebKitGTK.LibraryLoaded) return false;
// TODO webkit_check_version() should take care of the following, but for some
// reason this symbol is missing from the latest build. If it is present in

Back to the top