RESOLVED FIXED 50071
[Qt] Fix compilation with !ENABLE(OFFLINE_WEB_APPLICATIONS) and !ENABLE(TILED_BACKING_STORE) for WebKit2 
https://bugs.webkit.org/show_bug.cgi?id=50071
Summary [Qt] Fix compilation with !ENABLE(OFFLINE_WEB_APPLICATIONS) and !ENABLE(TILED...
Laszlo Gombos
Reported Thursday, November 25, 2010 1:43:18 PM UTC
This is primary for consistency for the WebKit1 based offering.
Attachments
proposed patch (5.73 KB, patch)
2010-11-25 05:50 PST, Laszlo Gombos 		no flags
DetailsFormatted DiffDiff
2nd try, resolve conflict with trunk (4.27 KB, patch)
2010-11-25 06:48 PST, Laszlo Gombos 		ossy: review+
ossy: commit-queue-
DetailsFormatted DiffDiff
fixed ChangeLog (4.20 KB, patch)
2010-12-01 20:52 PST, Laszlo Gombos 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Laszlo Gombos
Comment 1 Thursday, November 25, 2010 1:50:14 PM UTC
Created attachment 74864 [details] proposed patch
Laszlo Gombos
Comment 2 Thursday, November 25, 2010 2:48:51 PM UTC
Created attachment 74870 [details] 2nd try, resolve conflict with trunk
Csaba Osztrogonác
Comment 3 Tuesday, November 30, 2010 9:12:08 AM UTC
Comment on attachment 74870 [details] 2nd try, resolve conflict with trunk View in context: https://bugs.webkit.org/attachment.cgi?id=74870&action=review > WebKit2/ChangeLog:6 > + [Qt] Fix compilation with !ENABLE(OFFLINE_WEB_APPLICATIONS) and !ENABLE(JAVASCRIPT_DEBUGGER) and > + !ENABLE(TILED_BACKING_STORE) for WebKit2 Please remove !ENABLE(JAVASCRIPT_DEBUGGER) from ChangeLog, because there isn't any ENABLE(JAVASCRIPT_DEBUGGER) guard in this patch. :) Otherwise LGTM, r=me.
Csaba Osztrogonác
Comment 4 Tuesday, November 30, 2010 9:14:32 AM UTC
(In reply to comment #3) > Please remove !ENABLE(JAVASCRIPT_DEBUGGER) from ChangeLog, because there isn't > any ENABLE(JAVASCRIPT_DEBUGGER) guard in this patch. :) Otherwise LGTM, r=me. Or add fix for !ENABLE(JAVASCRIPT_DEBUGGER) case if it is necessary.
Laszlo Gombos
Comment 5 Thursday, December 2, 2010 4:52:19 AM UTC
Created attachment 75349 [details] fixed ChangeLog
WebKit Commit Bot
Comment 6 Friday, December 3, 2010 12:39:16 AM UTC
Comment on attachment 75349 [details] fixed ChangeLog Clearing flags on attachment: 75349 Committed r73221: <http://trac.webkit.org/changeset/73221>
WebKit Commit Bot
Comment 7 Friday, December 3, 2010 12:39:21 AM UTC
All reviewed patches have been landed. Closing bug.
WebKit Commit Bot
Comment 8 Friday, December 3, 2010 2:28:24 AM UTC
The commit-queue encountered the following flaky tests while processing attachment 75349 [details]: animations/suspend-resume-animation.html fast/events/tabindex-focus-blur-all.html Please file bugs against the tests. These tests were authored by alice.liu@apple.com, cmarrin@apple.com, and pam@chromium.org. The commit-queue is continuing to process your patch.
Note You need to log in before you can comment on or make changes to this bug.