Bug 150914 - Add runtime and compile time flags for enabling Web Animations API and model.
Summary: Add runtime and compile time flags for enabling Web Animations API and model.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Animations (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nikos Andronikos
URL:
Keywords:
Depends on:
Blocks: 122912
  Show dependency treegraph
 
Reported: 2015-11-04 17:10 PST by Nikos Andronikos
Modified: 2016-08-17 09:15 PDT (History)
5 users (show)

See Also:


Attachments
Patch (59.10 KB, patch)
2015-11-04 17:28 PST, Nikos Andronikos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nikos Andronikos 2015-11-04 17:10:11 PST
Add ENABLE_WEB_ANIMATIONS compile time flag, runtime flag webAnimationsEnabled and Expose WK2 preference for runtime flag for Web Animations implementation.
Comment 1 Nikos Andronikos 2015-11-04 17:28:59 PST
Created attachment 264831 [details]
Patch
Comment 2 Benjamin Poulain 2015-11-04 19:25:56 PST
Comment on attachment 264831 [details]
Patch

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

r- due to the lack of explanation why the flag is disabled.

> Source/WTF/wtf/FeatureDefines.h:783
> +#define ENABLE_WEB_ANIMATIONS 0

Why is the compile time flag disabled?

The compile time flags are important for shipping WebKit. If anyone needs to ship before a feature is finished and safe, its flag can be disabled.

For development, you should aim at having the flag enabled as soon as possible. That way you can have tests running on all platforms ASAP.
Comment 3 Nikos Andronikos 2015-11-04 19:58:17 PST
(In reply to comment #2)
> Comment on attachment 264831 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=264831&action=review
> 
> r- due to the lack of explanation why the flag is disabled.
> 
> > Source/WTF/wtf/FeatureDefines.h:783
> > +#define ENABLE_WEB_ANIMATIONS 0
> 
> Why is the compile time flag disabled?
> 
> The compile time flags are important for shipping WebKit. If anyone needs to
> ship before a feature is finished and safe, its flag can be disabled.
> 
> For development, you should aim at having the flag enabled as soon as
> possible. That way you can have tests running on all platforms ASAP.

My thinking was that we would enable it once there's something worthwhile to build. It looked to me like many features are disabled by default initially - e.g. ENABLE_ES6_GENERATORS and CSS_COMPOSITING.
I don't plan to wait too long to enable it, but would like to get some basics in first.
Comment 4 Benjamin Poulain 2015-11-05 13:36:37 PST
(In reply to comment #3)
> My thinking was that we would enable it once there's something worthwhile to
> build. It looked to me like many features are disabled by default initially
> - e.g. ENABLE_ES6_GENERATORS and CSS_COMPOSITING.
> I don't plan to wait too long to enable it, but would like to get some
> basics in first.

Ok, sounds good if you plan to enable it quickly.
Comment 5 WebKit Commit Bot 2015-11-05 14:27:52 PST
Comment on attachment 264831 [details]
Patch

Rejecting attachment 264831 [details] from commit-queue.

nikos.andronikos-webkit@cisra.canon.com.au does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 6 WebKit Commit Bot 2015-11-05 21:59:47 PST
Comment on attachment 264831 [details]
Patch

Clearing flags on attachment: 264831

Committed r192093: <http://trac.webkit.org/changeset/192093>
Comment 7 WebKit Commit Bot 2015-11-05 21:59:51 PST
All reviewed patches have been landed.  Closing bug.