Summary: | [Qt] Patch to locate Java plugin DLL on Windows via its Registry... | ||
---|---|---|---|
Product: | WebKit | Reporter: | Dawit A. <adawit> |
Component: | Plug-ins | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Enhancement | CC: | abarth, ademar, andersca, ap, aroben, commit-queue, eric, hausmann, jam, jhoneycutt, kenneth, webkit.review.bot |
Priority: | P2 | Keywords: | Qt |
Version: | 528+ (Nightly build) | ||
Hardware: | PC | ||
OS: | All | ||
Bug Depends on: | |||
Bug Blocks: | 33044 | ||
Attachments: |
Description
Dawit A.
2010-05-11 08:42:41 PDT
Created attachment 55711 [details]
Patch to locate Windows Java plugin DLL via registry...
Created attachment 55841 [details]
Locate Java plugin dll via the Windows Registry...
Corrected the entry in WebCore/ChangLog...
Adam, can you look at the Windows part of this patch, or indicate a reviewer? Safari comes with its own Java plug-in. I don't think we'd want to use any other Java plug-in. Jon Honeycutt and Anders Carlsson are the right people to look at this. (In reply to comment #4) > Safari comes with its own Java plug-in. I don't think we'd want to use any other > Java plug-in. I can #ifdef the patch to make it only apply to QtWebKit in case others, e.g. chrome, do the similar things as Safari. That way everyone that needs this has to opt-in and activate it for their own port ? (In reply to comment #5) > (In reply to comment #4) > > Safari comes with its own Java plug-in. I don't think we'd want to use any other > > Java plug-in. > > I can #ifdef the patch to make it only apply to QtWebKit in case others, e.g. chrome, do the similar things as Safari. That way everyone that needs this has to opt-in and activate it for their own port ? Sounds like a good idea, just wrap the call to the function in #if PLATFORM(QT). Created attachment 56391 [details]
Locate Java plugin dll via the Windows Registry (Update III)
Wrapped the call to "addJavaPluginDirectory" from "defaultPluginDirectories" with #if PLATFORM(QT)...
Attachment 56391 [details] did not build on win: Build output: http://webkit-commit-queue.appspot.com/results/2291375 Comment on attachment 56391 [details]
Locate Java plugin dll via the Windows Registry (Update III)
The windows failure scares me. Probably best to land by hand.
(In reply to comment #9) > (From update of attachment 56391 [details]) > The windows failure scares me. Probably best to land by hand. The problem with the Windows build failure is that the log produced is worthless. It does not tell you anything so you cannot even begin to evaluate why it failed and provide a fix if necessary... Created attachment 65219 [details]
Locate Java plugin dll via the Windows Registry (Update IV)
Let us see if it compiles fine on Windows this time around...
Comment on attachment 56391 [details] Locate Java plugin dll via the Windows Registry (Update III) Cleared Simon Hausmann's review+ from obsolete attachment 56391 [details] so that this bug does not appear in http://webkit.org/pending-commit. Comment on attachment 65219 [details]
Locate Java plugin dll via the Windows Registry (Update IV)
r=me
Comment on attachment 65219 [details] Locate Java plugin dll via the Windows Registry (Update IV) Clearing flags on attachment: 65219 Committed r66234: <http://trac.webkit.org/changeset/66234> All reviewed patches have been landed. Closing bug. http://trac.webkit.org/changeset/66234 might have broken Qt Linux Release The following changes are on the blame list: http://trac.webkit.org/changeset/66233 http://trac.webkit.org/changeset/66234 Blocking QtWebKit 2.1 on this. See https://bugs.webkit.org/show_bug.cgi?id=33044#c30 for more information. Revision r66234 cherry-picked into qtwebkit-2.1 with commit 8c6f4ae <http://gitorious.org/webkit/qtwebkit/commit/8c6f4ae> |