> On Jun 26, 2018, at 18:55, Michael Catanzaro <mcatanz...@igalia.com> wrote: > > > Hi, > > We're a lot closer after r233227! > > There is still IsSecureContextAttributeEnabled. Shall we switch the default > value of that one to use DEFAULT_EXPERIMENTAL_FEATURES_ENABLED instead of > true?
Not sure, waiting for other people to get back to me. There’s some indication that this one, indeed, should follow DEFAULT_EXPERIMENTAL_FEATURES_ENABLED, but I want to make sure :) > We should also move CSSAnimationTriggersEnabled to the experimental features > section of the file, and add the category tag. FullScreenEnabled and > EncryptedMediaAPIEnabled should also be moved down in the file. Will have to peek at these! > Then we can just update the comment to indicate that custom values like > DEFAULT_SERVICE_WORKERS_ENABLED are fine as long as they're thoughtful and > consider ENABLE(EXPERIMENTAL_FEATURES), and that will take care of the other > cases. Makes sense. > Michael > _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev