WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
14289
Spinneret builds but won't run.
https://bugs.webkit.org/show_bug.cgi?id=14289
Summary
Spinneret builds but won't run.
Jake
Reported
2007-06-21 12:26:32 PDT
Even though spinneret now builds with the new win32 code it doesn't run because CoreGraphics isn't getting initialized.
Attachments
Contains a fix for this bug.
(2.04 KB, patch)
2007-06-21 12:36 PDT
,
Jake
aroben
: review-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jake
Comment 1
2007-06-21 12:36:59 PDT
Created
attachment 15166
[details]
Contains a fix for this bug. This patch contains changes to Spinneret.cpp that initializes CoreGraphics. It uses the same method (InitializeCoreGraphics) as DumpRenderTree.
Adam Roben (:aroben)
Comment 2
2007-06-21 13:10:09 PDT
Comment on
attachment 15166
[details]
Contains a fix for this bug. I think instead of using LoadLibrary Spinneret should link against CoreGraphics[_debug].lib so it can just call InitializeCoreGraphics directly. This is what DRT does.
Adam Roben (:aroben)
Comment 3
2007-06-21 13:12:33 PDT
(In reply to
comment #2
)
> (From update of
attachment 15166
[details]
[edit]) > I think instead of using LoadLibrary Spinneret should link against > CoreGraphics[_debug].lib so it can just call InitializeCoreGraphics directly. > This is what DRT does.
Sorry, just CoreGraphics.lib.
Jake
Comment 4
2007-06-21 13:16:56 PDT
Well, I thought about that, but Spinneret uses only the COM interfaces and I'd like to keep it that way. In reality the call to InitializeCoreGraphics should be moved under the COM interfaces, in to WebKit. This is just a quick fix so that Spinneret will work, and w/o having additional link requirements imposed upon it. (In reply to
comment #3
)
> (In reply to
comment #2
) > > (From update of
attachment 15166
[details]
[edit] [edit]) > > I think instead of using LoadLibrary Spinneret should link against > > CoreGraphics[_debug].lib so it can just call InitializeCoreGraphics directly. > > This is what DRT does. > > Sorry, just CoreGraphics.lib. >
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug