WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
35100
[Qt] ENABLE_3D_RENDERING should be optional
https://bugs.webkit.org/show_bug.cgi?id=35100
Summary
[Qt] ENABLE_3D_RENDERING should be optional
Noam Rosenthal
Reported
2010-02-18 07:31:11 PST
for example, building webkit with --minimal should disable them.
Attachments
ENABLE_3D_RENDERING moved to be a normal feature
(2.17 KB, patch)
2010-02-18 08:51 PST
,
Noam Rosenthal
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Noam Rosenthal
Comment 1
2010-02-18 08:51:34 PST
Created
attachment 49020
[details]
ENABLE_3D_RENDERING moved to be a normal feature This seems to rectify the problem for ENABLE_3D_RENDERING (moved to be a normal feature, rather than a forced define). However, I'm not sure how to do that for WTF_ENABLE_ACCELERATED_COMPOSITING, which doesn't appear as a "feature". Help would be appreciated.
Laszlo Gombos
Comment 2
2010-02-19 12:19:45 PST
The patch looks good to me, if we only try to restrict it to ENABLE_3D_RENDERING. Noam, can you change the bug title and ChangeLog accordingly ? I think we should open another bug for ACCELERATED_COMPOSITING as it will likely involve changes to other ports as well, not just for QtWebKit. In general USE(xx) flags are not meant to be turn on/off optionally at build time within a given WebKit port.
Laszlo Gombos
Comment 3
2010-02-20 12:10:23 PST
Comment on
attachment 49020
[details]
ENABLE_3D_RENDERING moved to be a normal feature LGTM, r+. I will update the title in the ChangeLogs as well during a manual commit.
Laszlo Gombos
Comment 4
2010-02-20 12:30:24 PST
Comment on
attachment 49020
[details]
ENABLE_3D_RENDERING moved to be a normal feature Committed as
http://trac.webkit.org/changeset/55056
.
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