This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 218378 - Embedded Webkit Browser, autodetect
Summary: Embedded Webkit Browser, autodetect
Status: VERIFIED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Embedded Browser (show other bugs)
Version: 7.3
Hardware: All All
: P2 normal (vote)
Assignee: Stanislav Aubrecht
URL:
Keywords:
Depends on:
Blocks: 220342
  Show dependency tree
 
Reported: 2012-09-14 13:57 UTC by javydreamercsw
Modified: 2012-10-19 13:43 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
IDE log (286.28 KB, text/plain)
2012-09-14 13:57 UTC, javydreamercsw
Details

Note You need to log in before you can comment on or make changes to this bug.
Description javydreamercsw 2012-09-14 13:57:46 UTC
Product Version = NetBeans IDE Dev (Build 201209140001)
Operating System = Windows 7 version 6.1 running on x86
Java; VM; Vendor = 1.7.0_07
Runtime = Java HotSpot(TM) Client VM 23.3-b01

I noticed it prompts you to browse for the JavaFX 2.2 runtime. All I had to do was point it to the JDK7 directory. Since it is part of JDK7, shouldn't the IDE/plug-in be able to auto configure it? The IDE already knows if it is running on JDK7 right?
Comment 1 javydreamercsw 2012-09-14 13:57:52 UTC
Created attachment 124375 [details]
IDE log
Comment 2 Marian Mirilovic 2012-09-18 16:35:30 UTC
+1
Comment 3 Stanislav Aubrecht 2012-09-19 13:19:34 UTC
Petr, do we want to change the default JavaFX runtime lookup this way?
Comment 4 Petr Jiricka 2012-09-19 13:33:09 UTC
I was thinking about this some time ago, and the points to consider are:
- We don't require just any JDK 7, right now we need the latest JDK 7u6. So the location should not be autodetected if the JDK version is lower than 7u6.
- For NB 7.3 final, this will change again, as we will require a JavaFX runtime from JDK 7u10 or later
- It would be useful to have an option to test with a different version of JavaFX than the one included in the JDK. I see that this is stored under userdir in config/Preferences/org/netbeans/core/browser/webview.properties. Stando, can you confirm that by manually changing this file, I can test with another version of JavaFX?

I also didn't think this was high priority because this only concerns zip builds and most users of FCS versions use the installer, but at the end, why not.
Comment 5 Stanislav Aubrecht 2012-09-19 14:06:17 UTC
(In reply to comment #4)
> - It would be useful to have an option to test with a different version of
> JavaFX than the one included in the JDK. I see that this is stored under
> userdir in config/Preferences/org/netbeans/core/browser/webview.properties.
> Stando, can you confirm that by manually changing this file, I can test with
> another version of JavaFX?
Yes, although menu Tools - Options - General - Edit is probably more convenient.
Comment 6 Petr Jiricka 2012-09-19 16:14:00 UTC
Thanks!
Comment 7 Petr Jiricka 2012-10-15 12:39:29 UTC
We are thinking about no longer bundling the JavaFX runtime, because the penalty in terms of the download size is just too high. Because of this, this issue is becoming more relevant, I am changing it to DEFECT.

I suggest to autodetect the JavaFX runtime from the underlying JDK if the JDK is version 7u10 (currently in Developer Preview) or newer. JavaFX 2.2.4 (which is included in JDK 7u10) will be the minimum supported JavaFX version for the embedded browser. The current latest released version (JavaFX 2.2.1/JDK 7u7) is not good enough.

I also filed a related bug 220113 regarding the wording of the prompt that the embedded browser displays if the required version of JavaFX is not detected.
Comment 8 Stanislav Aubrecht 2012-10-18 10:16:07 UTC
core-main ef7eaa6bf762
Comment 9 Petr Jiricka 2012-10-19 12:35:27 UTC
Thanks, verifying.
Comment 10 Quality Engineering 2012-10-19 13:43:47 UTC
Integrated into 'main-golden', will be available in build *201210191216* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/ef7eaa6bf762
User: S. Aubrecht <saubrecht@netbeans.org>
Log: #218378 - autodetect JavaFX bundled in current JRE