<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add a RuntimeEnabledFeature for display: contents"
   href="https://bugs.webkit.org/show_bug.cgi?id=171984#c20">Comment # 20</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add a RuntimeEnabledFeature for display: contents"
   href="https://bugs.webkit.org/show_bug.cgi?id=171984">bug 171984</a>
              from <span class="vcard"><a class="email" href="mailto:ecobos@igalia.com" title="Emilio Cobos Álvarez <ecobos@igalia.com>"> <span class="fn">Emilio Cobos Álvarez</span></a>
</span></b>
        <pre>(In reply to Ryosuke Niwa from <a href="show_bug.cgi?id=171984#c13">comment #13</a>)
<span class="quote">> Comment on <span class="bz_obsolete"><a href="attachment.cgi?id=309797&action=diff" name="attach_309797" title="Patch">attachment 309797</a> <a href="attachment.cgi?id=309797&action=edit" title="Patch">[details]</a></span>
> Patch

> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=309797&action=review">https://bugs.webkit.org/attachment.cgi?id=309797&action=review</a>

> > Source/WebCore/ChangeLog:11
> > +        Add a RuntimeEnabledFeature for display: contents, defaulted to false.
> > +
> > +        The "defaulted to false" is not only because there are spec issues, but because
> > +        I ran the WPT suite, and there was a fair amount of crashes and messed render
> > +        trees.
> > +
> > +        <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add a RuntimeEnabledFeature for display: contents"
   href="show_bug.cgi?id=171984">https://bugs.webkit.org/show_bug.cgi?id=171984</a>
> > +
> > +        Reviewed by Antti Koivisto.

> Change log here is really messed up.

> The bug title should be followed by a bug URL, followed by a blank line,
> then a long description.</span >

Thanks for the heads-up, prepare-ChangeLog posted the whole git commit description before the url field, and no script complained about it, so I just assumed it was fine. I updated and amended it.

(In reply to Sam Weinig from <a href="show_bug.cgi?id=171984#c18">comment #18</a>)
<span class="quote">> Is there any reason this need to be a RuntimeEnabledFeature rather than a
> Setting?</span >

>From a quick look RuntimeEnabledFeatures it what's usually used for web features, but happy to change it if there's a good reason to.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>