[Webkit-unassigned] [Bug 135560] [EFL] Remove non Coordinated Graphics code path from cmake build system after r142169

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=135560





--- Comment #9 from Gyuyoung Kim <gyuyoung.kim at webkit.org>  2014-08-07 17:06:44 PST ---
(In reply to comment #6)
> (In reply to comment #5)
> > > > USE(TILED_BACKING_STORE) is still being used in source code. So, I'm not sure whether we can remove this here.
> > > 
> > > If we leave it in the code, the build will fail in case of minimal build. None of the other ports use it, so it's safe to remove it.
> > 
> > If so, you can just change isEfl() with 0. Are you sure whether other ports never enable it again ?
> 
> Let's see all options one by one:
> - default => isEfl() : It means it is enabled on EFL by default, but 
>   --minimal option would disable this mandatory feature. It isn't good.
> - default => 0 : It means it would be disabled always on EFL - on minimal
>   and on normal build too. It isn't good.
> - Deleting it from FeatureList.pm would cause problem if we would like
>   to make TILED_BACKING_STORE optional. But it isn't optional at all.
>   It is mandatory on EFL and not used / not work on other ports.

Ossy, I see. I wondered whether other ports will use it future. However, it looks GTK port won't use it. Looks good if you fix Ryuan's comments as well.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.