yinxianwei / chromedevtools

Automatically exported from code.google.com/p/chromedevtools
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Make Chromium Dev Tools Java5 - compatible #17

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Currently Chromium Developer Tools require Java6 to properly build and run. 
This has some unfortunate implications on Mac as Java6 is 64-bit only on 
Mac and thus the only SWT variant is Cocoa64 that is lacking in several 
departments - i.e. it does not support embedding Mozilla browser, has 
higher memory requirement, etc.

Attached is a patch that enables the plugins to build and run on Java5. We 
did some pleriminary changes and found no problems.

Changes in the patch are:
1. Manifests were updated to include Java5 as valid run-time environment.
2. ChromiumIOException was introduced as java.io.IOException does not 
support ctors with Exception. I opted for introducing a new subclass so 
there is little changes to application behavior.
3. Handshaker::runnableFeature type was changed to FutureTask from 
RunnableTask - it was always its actual type. RunnableTask is not available 
in Java5 and is not a superclass of the FutureTask.

Original issue reported on code.google.com by eostrouk...@gmail.com on 28 Jan 2010 at 8:48

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by peter.ry...@gmail.com on 7 Feb 2010 at 7:49

GoogleCodeExporter commented 8 years ago
Fixed in HEAD / r311

Original comment by peter.ry...@gmail.com on 7 Mar 2010 at 8:35

GoogleCodeExporter commented 8 years ago
Eguene

Could you please update on this issue as it's almost year now since you 
submitted it and we are considering switching to Java6 again.

Thank you,
Peter

Original comment by peter.ry...@gmail.com on 1 Dec 2010 at 10:18

GoogleCodeExporter commented 8 years ago
This is not an issue on Mac OS X 10.6 as it has both 32-bit and 64-bit Java6. 
This is still an issue on Mac OS X 10.5.

Original comment by eostrouk...@gmail.com on 1 Dec 2010 at 10:47