Bug 100829

Summary: [EFL] CMake shows ENABLE_3D_RENDERING and ENABLE_WEBGL is still OFF when AC is enabled
Product: WebKit Reporter: Joone Hur <joone>
Component: WebKit EFLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, halton.huo, lucas.de.marchi, paroga, rakuco, tmpsantos, webkit.review.bot, yael
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
patch
joone: review-, joone: commit-queue-
Patch
none
Patch none

Description Joone Hur 2012-10-31 01:34:02 PDT
When tiled backing store is turned on, graphics acceleration stuff is enabled accordingly as follows:

http://trac.webkit.org/changeset/132571
http://trac.webkit.org/changeset/132627

However, CMake still shows that ENABLE_3D_RENDERING and ENABLE_WEBGL is still OFF when running build-webkit with --tiled-backing-store.
Comment 1 Joone Hur 2012-10-31 01:58:01 PDT
Created attachment 171598 [details]
patch
Comment 2 Raphael Kubo da Costa (:rakuco) 2012-10-31 02:50:38 PDT
Comment on attachment 171598 [details]
patch

Simply moving the call to WEBKIT_OPTION_END() to the end of the file in OptionsEfl.cmake should solve the problem in a cleaner way.
Comment 3 Joone Hur 2012-10-31 23:06:14 PDT
(In reply to comment #2)
> (From update of attachment 171598 [details])
> Simply moving the call to WEBKIT_OPTION_END() to the end of the file in OptionsEfl.cmake should solve the problem in a cleaner way.

I tried, but it doesn't work.
Comment 4 Raphael Kubo da Costa (:rakuco) 2012-11-01 06:39:29 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 171598 [details] [details])
> > Simply moving the call to WEBKIT_OPTION_END() to the end of the file in OptionsEfl.cmake should solve the problem in a cleaner way.
> 
> I tried, but it doesn't work.

That's probably because of the interaction between normal variables and cache variables in CMake.

SET(ENABLE_WEBGL 1) in OptionsEfl.cmake sets a normal variable, and later the call to OPTION(ENABLE_WEBGL ...) will set a cache variable with the same name and remove the original normal variable.

I think the cleanest solution here is to move the call to WEBKIT_OPTION_END() to the end of OptionEfl.cmake and modify the macro so that the initial value it uses in the OPTION() call is the value of the variable itself if it is defined, and the initial value otherwise. This would still require you to clean up your cache nonetheless, as CMake by default won't change the options you already have in your cache when you call a command such as OPTION().

Patrick, what do you think?
Comment 5 Raphael Kubo da Costa (:rakuco) 2012-11-21 05:47:58 PST
*** Bug 102903 has been marked as a duplicate of this bug. ***
Comment 6 Halton Huo 2012-11-21 07:22:24 PST
How about my patch for the dup bug 102903? it works as wish. could somebody assign this bug to me so that i can add upload patch.
Comment 7 Halton Huo 2012-11-21 17:42:06 PST
Created attachment 175555 [details]
Patch
Comment 8 Halton Huo 2012-11-21 21:08:09 PST
Anyone could review? Joone, Gyuyoung?
Comment 9 Gyuyoung Kim 2012-11-21 22:12:57 PST
Comment on attachment 175555 [details]
Patch

Hmm, WebKit doesn't like that a bug lands multiple patches. I think you guys need to merge these patches into a patch.
Comment 10 Halton Huo 2012-11-22 01:21:53 PST
Kim, as joone said on comment #3, first patch does not work. The second one I uploaded works well, not need with first patch.
Comment 11 Gyuyoung Kim 2012-11-22 01:40:53 PST
Comment on attachment 175555 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=175555&action=review

I think rakuco might to take a final look.

> ChangeLog:4
> +        AC is enabled

It looks this is unneeded line change.
Comment 12 Halton Huo 2012-11-22 02:42:27 PST
Created attachment 175634 [details]
Patch
Comment 13 Halton Huo 2012-11-22 02:44:18 PST
(In reply to comment #12)
> Created an attachment (id=175634) [details]
> Patch
Update patch addressing Gyuyoung's comment at #11
Comment 14 WebKit Review Bot 2012-11-26 20:37:48 PST
Comment on attachment 175634 [details]
Patch

Clearing flags on attachment: 175634

Committed r135813: <http://trac.webkit.org/changeset/135813>
Comment 15 WebKit Review Bot 2012-11-26 20:37:53 PST
All reviewed patches have been landed.  Closing bug.