RESOLVED INVALID 90262
[Qt][WTR] Get rid of using DumpRenderTreeSupportQt
https://bugs.webkit.org/show_bug.cgi?id=90262
Summary [Qt][WTR] Get rid of using DumpRenderTreeSupportQt
Balazs Kelemen
Reported 2012-06-29 02:41:53 PDT
The only reason why we use it in some place is the v8 support. Now that we are over with it we can remove these.
Attachments
Patch (3.51 KB, patch)
2012-06-29 04:20 PDT, Balazs Kelemen
no flags
Patch (4.41 KB, patch)
2012-07-25 11:52 PDT, Balazs Kelemen
no flags
Patch (5.63 KB, patch)
2012-08-01 01:16 PDT, Balazs Kelemen
no flags
Balazs Kelemen
Comment 1 2012-06-29 04:20:02 PDT
Balazs Kelemen
Comment 2 2012-06-29 05:05:39 PDT
Comment on attachment 150134 [details] Patch Clearing flags on attachment: 150134 Committed r121550: <http://trac.webkit.org/changeset/121550>
Balazs Kelemen
Comment 3 2012-06-29 05:05:46 PDT
All reviewed patches have been landed. Closing bug.
Balazs Kelemen
Comment 4 2012-07-03 10:37:26 PDT
Reverted in r121788. Need to fix some issues of the Qt build system before relanding this.
Balazs Kelemen
Comment 5 2012-07-25 09:16:36 PDT
*** Bug 90346 has been marked as a duplicate of this bug. ***
Balazs Kelemen
Comment 6 2012-07-25 11:52:24 PDT
Early Warning System Bot
Comment 7 2012-07-25 13:09:00 PDT
Simon Hausmann
Comment 8 2012-07-26 07:56:27 PDT
Comment on attachment 154410 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=154410&action=review I think the patch is good, I like this approach. But it appears to break the Gtk build, the EWS failure seems valid. > Source/WebCore/ChangeLog:3 > + [Qt][WTR] Get rid of using DumpRenderTreeSupportQt I think the title is a bit generic, what about renaming it to "Get rid of DumpRenderTreeSupportQt::injectInternals wrapper"?
Balazs Kelemen
Comment 9 2012-08-01 01:16:50 PDT
Build Bot
Comment 10 2012-08-01 02:08:15 PDT
Build Bot
Comment 11 2012-08-01 02:59:20 PDT
Build Bot
Comment 12 2012-08-01 03:17:29 PDT
Balazs Kelemen
Comment 13 2012-08-01 05:11:17 PDT
Adding WEBCORE_TESTING to WebCoreTestSupport::injectInternalsObject and WebCoreTestSupport::resetInternalsObject breaks the win build. It needs the symbol "__declspec(dllimport) void __cdecl WebCoreTestSupport::injectInternalsObject(struct OpaqueJSContext const *)" in DRT but it doesn't find it. It seems strange to me because dllimport should be the right here to do from DRT. Idea?
Hajime Morrita
Comment 14 2012-08-01 17:41:01 PDT
(In reply to comment #13) > Adding WEBCORE_TESTING to WebCoreTestSupport::injectInternalsObject and WebCoreTestSupport::resetInternalsObject breaks the win build. It needs the symbol "__declspec(dllimport) void __cdecl WebCoreTestSupport::injectInternalsObject(struct OpaqueJSContext const *)" in DRT but it doesn't find it. It seems strange to me because dllimport should be the right here to do from DRT. Idea? WEBCORE_TESTING isn't ready yet. I'm sorry for the inconvenience,
Balazs Kelemen
Comment 15 2012-08-03 04:32:25 PDT
(In reply to comment #14) > (In reply to comment #13) > > Adding WEBCORE_TESTING to WebCoreTestSupport::injectInternalsObject and WebCoreTestSupport::resetInternalsObject breaks the win build. It needs the symbol "__declspec(dllimport) void __cdecl WebCoreTestSupport::injectInternalsObject(struct OpaqueJSContext const *)" in DRT but it doesn't find it. It seems strange to me because dllimport should be the right here to do from DRT. Idea? > > WEBCORE_TESTING isn't ready yet. I'm sorry for the inconvenience, I'm a bit confused how to proceed with this. I don't want to add a new platform specific define because WEBCORE_TESTING is supposed to serve for this Could you describe what is missing and how could those symbols note become exported?
Balazs Kelemen
Comment 16 2013-01-27 14:45:54 PST
I could not take care about this in the foreseeable future so feel free to pick it.
Note You need to log in before you can comment on or make changes to this bug.