Bug 102903 - [EFL] Wrong enabled features reported
Summary: [EFL] Wrong enabled features reported
Status: RESOLVED DUPLICATE of bug 100829
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Major
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-21 01:48 PST by Halton Huo
Modified: 2012-11-21 05:48 PST (History)
5 users (show)

See Also:


Attachments
Patch (4.42 KB, patch)
2012-11-21 02:14 PST, Halton Huo
no flags Details | Formatted Diff | Diff
Patch (4.66 KB, patch)
2012-11-21 04:45 PST, Halton Huo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Halton Huo 2012-11-21 01:48:18 PST
Build trunk@135354 on linux EFL port, it will dump me Enabled messages as following:
-- Enabled features:
--  ENABLE_API_TESTS ........................ ON
--  ENABLE_3D_RENDERING                       OFF
[...]
--  ENABLE_WEBGL                              OFF
[...]
--  WTF_USE_TILED_BACKING_STORE               OFF
--  WTF_USE_WTFURL .......................... OFF

But actually the EFL port is enable WTF_USE_TILED_BACKING_STORE by default in OpionsEfl.cmake. And ENABLE_3D_RENDERING and ENABLE_WEBGL is also enabled in when WTF_USE_TILED_BACKING_STORE is ON.
Comment 1 Halton Huo 2012-11-21 02:14:13 PST
Created attachment 175383 [details]
Patch
Comment 2 EFL EWS Bot 2012-11-21 02:33:13 PST
Comment on attachment 175383 [details]
Patch

Attachment 175383 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/14894924
Comment 3 Halton Huo 2012-11-21 02:47:07 PST
Bug #102910 is filed to track the compiling issue on CSSComputedStyleDeclaration.cpp. I do not think my patch cause this failure.
Comment 4 Halton Huo 2012-11-21 04:45:59 PST
Created attachment 175411 [details]
Patch
Comment 5 Halton Huo 2012-11-21 04:52:29 PST
Bug 102910 is a regression with my first patch, so close that bug. The problem in 1st patch is only enabled features will be added to FEATURE_DEFINES_WITH_SPACE_SEPARATOR, this cause makeprop.pl generate wrong CSSPropertyNames.[cpp|gperf|h].

The 2nd patch fix that problem, the message will be set by using value of ${_WEBKIT_AVAILABLE_OPTIONS_INITALVALUE_${_name}} and move out from  IF (${_name}) logic. The FEATURE_DEFINES_WITH_SPACE_SEPARATOR logic keep in the IF (${_name}) logic.

Please review.
Comment 6 Raphael Kubo da Costa (:rakuco) 2012-11-21 05:47:58 PST
While this could solve the problem, this would make the list of features we enable in OptionsEFL.cmake different from the list in FeaturesList.pm, since we don't enable WebGL by default and 3D Rendering is enabled as a side-effect of tiled backing store being on by default (this one is already correctly set in both OptionsEFL.cmake and FeaturesList.pm).

I still think that a good solution to the problem would be something along the lines of what I described in comment #4 of bug 100829.

*** This bug has been marked as a duplicate of bug 100829 ***